Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(blocks): Add support for mutually exclusive input fields #8856

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from
Open

feat(blocks): Add support for mutually exclusive input fields #8856

wants to merge 4 commits into from
+149 −15

Conversation

Abhi1992002
Copy link
Contributor

@Abhi1992002 Abhi1992002 commented Dec 2, 2024 •
edited
Loading

Changes

  • Introduced mutually_exclusive parameter in SchemaField to manage input exclusivity.
  • Implemented logic in NodeGenericInputField to disable inputs based on mutual exclusivity.
  • Updated related components to support the new disabled state for inputs.
  • Enhanced BlockIOSubSchemaMeta to include mutually_exclusive property.

Currently, I’m disabling the input from the same group (I haven’t added any frontend validation to prevent users from bypassing it).

Screen.Recording.2024-12-02.at.5.08.42.PM.mov

@Abhi1992002
feat: Add mutually exclusive input handling in NodeGenericInputField
be8fd0f 
### Changes
- Introduced `mutually_exclusive` parameter in `SchemaField` to manage input exclusivity.
- Implemented logic in `NodeGenericInputField` to disable inputs based on mutual exclusivity.
- Updated related components to support the new `disabled` state for inputs.
- Enhanced `BlockIOSubSchemaMeta` to include `mutually_exclusive` property.
@Abhi1992002
feat: Enhance TestMutualExclusiveBlock with mutually exclusive input …
4b91a38 
…fields

### Changes
- Added `mutually_exclusive` parameter to `SchemaField` for `text_input_1`, `text_input_2`, and `text_input_3` to group them under "group_1".
- Updated `number_input_1` and `number_input_2` to belong to "group2" for mutual exclusivity.
@Abhi1992002 Abhi1992002 requested a review from a team as a code owner December 2, 2024 11:46
@Abhi1992002 Abhi1992002 requested review from Torantulino and majdyz and removed request for a team December 2, 2024 11:46
@netlify Netlify
Copy link

netlify bot commented Dec 2, 2024 •
edited
Loading

✅ Deploy Preview for auto-gpt-docs-dev ready!

Name Link
🔨 Latest commit 0484484
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs-dev/deploys/6756fbe41a0ec000082e77bb
😎 Deploy Preview https://deploy-preview-8856--auto-gpt-docs-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions github-actions bot added platform/frontend AutoGPT Platform - Front end platform/backend AutoGPT Platform - Back end platform/blocks size/l labels Dec 2, 2024
@Abhi1992002 Abhi1992002 changed the title Feat/single input validation lock feat(platform) : single input validation lock Dec 2, 2024
@netlify Netlify
Copy link

netlify bot commented Dec 2, 2024 •
edited
Loading

✅ Deploy Preview for auto-gpt-docs ready!

Name Link
🔨 Latest commit 0484484
🔍 Latest deploy log https://app.netlify.com/sites/auto-gpt-docs/deploys/6756fbe489714f0008fef552
😎 Deploy Preview https://deploy-preview-8856--auto-gpt-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@Abhi1992002 Abhi1992002 marked this pull request as draft December 2, 2024 11:51
@Abhi1992002 Abhi1992002 marked this pull request as ready for review December 2, 2024 12:58
aarushik93
aarushik93 reviewed Dec 3, 2024
View reviewed changes
autogpt_platform/backend/backend/blocks/test_mutual_exclusive.py
from backend.data.model import SchemaField


class TestMutualExclusiveBlock(Block):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is a good addition - for the tests however, I wonder if you could put the tests in one of the real blocks to show it working?

Pwuts
Pwuts reviewed Dec 5, 2024
View reviewed changes
Copy link
Member

@Pwuts Pwuts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a good feature, but I'm not sure about the implementation. JSON schema has oneOf keyword which seems perfect for this purpose. What do you think?

I always prefer using an existing mechanism or standard rather than inventing our own, unless that makes implementation unreasonably complex.

@Pwuts Pwuts changed the title feat(platform) : single input validation lock feat(blocks): Add support for mutually exclusive input fields Dec 5, 2024
@majdyz majdyz removed their request for review December 10, 2024 22:43
@ntindle
Copy link
Member

ntindle commented Dec 12, 2024 •
edited
Loading

This is a good feature, but I'm not sure about the implementation. JSON schema has oneOf keyword which seems perfect for this purpose. What do you think?

I always prefer using an existing mechanism or standard rather than inventing our own, unless that makes implementation unreasonably complex.

How would that look? I figure if we did that we'd need to implement the rest: allOf, anyOf, oneOf, not, if, then, etc

See also #8852

Imo this isn't worth blocking over and json schema is far more complex than meets the eye

@github-actions github-actions bot added the conflicts Automatically applied to PRs with merge conflicts label Dec 13, 2024
@github-actions GitHub Actions
Copy link
Contributor

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aarushik93 aarushik93 aarushik93 left review comments

@Pwuts Pwuts Pwuts left review comments

@Torantulino Torantulino Awaiting requested review from Torantulino Torantulino is a code owner automatically assigned from Significant-Gravitas/maintainers

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
conflicts Automatically applied to PRs with merge conflicts platform/backend AutoGPT Platform - Back end platform/blocks platform/frontend AutoGPT Platform - Front end size/l
Projects
Frontend
Status: No status
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Abhi1992002 @ntindle @Pwuts @aarushik93