-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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.
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
feat: action redesign, UQI upgrade AWS plugin config to dual zone format #36092
Conversation
WalkthroughThe changes involve updates to the Changes
Assessment against linked issues
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
/build-deploy-preview skip-tests=true |
Deploying Your Preview: https://github.com/appsmithorg/appsmith/actions/runs/10685253834. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files selected for processing (3)
- app/server/appsmith-plugins/awsLambdaPlugin/src/main/resources/editor/invoke.json (2 hunks)
- app/server/appsmith-plugins/awsLambdaPlugin/src/main/resources/editor/list.json (1 hunks)
- app/server/appsmith-plugins/awsLambdaPlugin/src/main/resources/editor/root.json (1 hunks)
Additional comments not posted (5)
app/server/appsmith-plugins/awsLambdaPlugin/src/main/resources/editor/list.json (1)
3-3
: The change in thecontrolType
property is approved.The update from
"SECTION"
to"SECTION_V2"
suggests an evolution in the configuration structure, likely aimed at improving the functionality or user experience of the application. This change is in line with the PR objectives of upgrading the AWS Lambda plugin configuration to a new format.Here are a few insights:
The change is isolated to the
controlType
property, and the rest of the configuration remains intact, minimizing the risk of unintended side effects.The
conditionals
object is unchanged, maintaining the existing logic for showing the control based on the specified condition.The new
"SECTION_V2"
value may indicate enhanced functionality or a new version of the control, potentially offering improved user experience or additional features.It's important to ensure that the frontend components and any associated backend logic are updated to handle the new
"SECTION_V2"
control type correctly.Consider updating the documentation or release notes to inform users about the changes and any new features or improvements associated with the
"SECTION_V2"
control type.Overall, the change appears to be a step towards enhancing the AWS Lambda plugin configuration and aligns with the PR objectives. Great work!
app/server/appsmith-plugins/awsLambdaPlugin/src/main/resources/editor/root.json (1)
4-4
: Please provide more context about the"SECTION_V2"
control type.It would be helpful to understand the following:
- What new features or improvements does the
"SECTION_V2"
control type introduce compared to the previous"SECTION"
control type?- How does this change align with the overall goals and roadmap of the project?
- Are there any potential risks or backward compatibility issues that need to be considered with this update?
Providing more context will help in assessing the impact and necessity of this change.
app/server/appsmith-plugins/awsLambdaPlugin/src/main/resources/editor/invoke.json (3)
3-3
: Verify compatibility with the new control type version.The
controlType
for the main section has been updated to"SECTION_V2"
. Please ensure that:
- The UI components rendering this section are compatible with the new control type version.
- The backend logic processing this configuration handles the new version correctly.
To verify the compatibility, you can run the following script:
70-71
: Verify the integration of the new input field.A new control type,
"SINGLE_COLUMN_ZONE"
, has been introduced, encapsulating a new input field for the "Post body". Please ensure that:
- The new input field is properly rendered in the user interface.
- The data entered in this field is correctly processed by the backend logic.
- The data is included in the request payload when invoking the AWS Lambda function.
To verify the integration, you can run the following script:
9-9
: Verify the UI rendering of the new layout.The
controlType
for the details section has been updated to"DOUBLE_COLUMN_ZONE"
, indicating a change in the layout. Please ensure that the UI components are updated to render this new layout correctly.To verify the UI rendering, you can run the following script:
Deploy-Preview-URL: https://ce-36092.dp.appsmith.com |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: .coderabbit.yaml
Review profile: CHILL
Files selected for processing (1)
- app/server/appsmith-plugins/awsLambdaPlugin/src/main/resources/editor/root.json (1 hunks)
Files skipped from review as they are similar to previous changes (1)
- app/server/appsmith-plugins/awsLambdaPlugin/src/main/resources/editor/root.json
Failed server tests
|
This PR has been tested on CE & EE (modules, workflows). |
Description
Upgrade AWS lambda plugin config to new format using SECTION_V2, SINGLE_COLUMN_ZONE, and DOUBLE_COLUMN_ZONE.
Fixes #35488
Automation
/ok-to-test tags="@tag.All"
🔍 Cypress test results
Tip
🟢 🟢 🟢 All cypress tests have passed! 🎉 🎉 🎉
Workflow run: https://github.com/appsmithorg/appsmith/actions/runs/10716532467
Commit: 7af86d6
Cypress dashboard.
Tags:
@tag.All
Spec:
Thu, 05 Sep 2024 09:39:20 UTC
Communication
Should the DevRel and Marketing teams inform users about this change?
Summary by CodeRabbit
New Features
Bug Fixes