-
Notifications
You must be signed in to change notification settings - Fork 5.6k
[ACTION] New action create-custom-field-options-context #12629
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
[ACTION] New action create-custom-field-options-context #12629
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
|
@jcortes is attempting to deploy a commit to the Pipedreamers Team on Vercel. A member of the Team first needs to authorize it. |
WalkthroughThe changes consist of version updates for multiple Jira actions and components. Most actions have been incremented by one version number, indicating improvements or enhancements. Notable updates include the introduction of new functionality for creating custom field options, while existing actions received minor documentation improvements. This release aims to enhance integration with Jira's API. Changes
Sequence Diagram(s)(Diagrams were not generated as the changes are primarily version updates with no significant alterations to control flow.) Assessment against linked issues
Poem
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 as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
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 UI
Review profile: CHILL
Files selected for processing (5)
- components/jira/actions/create-custom-field-options-context/create-custom-field-options-context.mjs (1 hunks)
- components/jira/common/constants.mjs (1 hunks)
- components/jira/common/utils.mjs (2 hunks)
- components/jira/jira.app.mjs (3 hunks)
- components/jira/package.json (1 hunks)
Files skipped from review due to trivial changes (2)
- components/jira/common/constants.mjs
- components/jira/package.json
Additional comments not posted (6)
components/jira/common/utils.mjs (2)
1-1: Import of ConfigurationError approved.This import enhances error handling by allowing more specific error messages, improving the robustness of the application.
25-46: New functionparseArrayis well-implemented.This function correctly handles different types of input and ensures that only valid array data is processed. It enhances data integrity and error handling by using
ConfigurationErrorto provide clear feedback on input errors.components/jira/actions/create-custom-field-options-context/create-custom-field-options-context.mjs (1)
4-83: New actionCreate Custom Field Options (Context)is correctly implemented.The action is well-defined with clear properties and methods. The use of utility functions for data parsing within the
runmethod ensures robust data handling. This aligns with the objectives of the PR and enhances the Jira component's functionality.components/jira/jira.app.mjs (3)
4-4: Import of constants approved.Using shared constants enhances maintainability and reduces potential errors in the application. This is a positive change.
572-577: New methodgetFieldsPaginatedis well-implemented.This method efficiently handles the pagination of field data, using the
DEFAULT_LIMITconstant to control the page size. This is crucial for performance and scalability.
578-585: New methodgetCustomFieldContextsis correctly implemented.Like
getFieldsPaginated, this method provides a consistent and efficient way to fetch custom field contexts with pagination, enhancing the module's functionality and performance.
63a0be7 to
e5bf878
Compare
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 UI
Review profile: CHILL
Files selected for processing (27)
- components/jira/actions/add-attachment-to-issue/add-attachment-to-issue.mjs (1 hunks)
- components/jira/actions/add-comment-to-issue/add-comment-to-issue.mjs (1 hunks)
- components/jira/actions/add-multiple-attachments-to-issue/add-multiple-attachments-to-issue.mjs (1 hunks)
- components/jira/actions/add-watcher-to-issue/add-watcher-to-issue.mjs (1 hunks)
- components/jira/actions/assign-issue/assign-issue.mjs (1 hunks)
- components/jira/actions/create-custom-field-options-context/create-custom-field-options-context.mjs (1 hunks)
- components/jira/actions/create-issue/create-issue.mjs (1 hunks)
- components/jira/actions/create-version/create-version.mjs (1 hunks)
- components/jira/actions/delete-project/delete-project.mjs (1 hunks)
- components/jira/actions/get-all-projects/get-all-projects.mjs (1 hunks)
- components/jira/actions/get-issue/get-issue.mjs (1 hunks)
- components/jira/actions/get-task/get-task.mjs (1 hunks)
- components/jira/actions/get-transitions/get-transitions.mjs (1 hunks)
- components/jira/actions/get-user/get-user.mjs (1 hunks)
- components/jira/actions/get-users/get-users.mjs (1 hunks)
- components/jira/actions/list-issue-comments/list-issue-comments.mjs (1 hunks)
- components/jira/actions/transition-issue/transition-issue.mjs (1 hunks)
- components/jira/actions/update-comment/update-comment.mjs (1 hunks)
- components/jira/actions/update-issue/update-issue.mjs (1 hunks)
- components/jira/common/constants.mjs (1 hunks)
- components/jira/common/utils.mjs (2 hunks)
- components/jira/jira.app.mjs (3 hunks)
- components/jira/package.json (1 hunks)
- components/jira/sources/events/events.mjs (1 hunks)
- components/jira/sources/issue-created/issue-created.mjs (1 hunks)
- components/jira/sources/issue-deleted/issue-deleted.mjs (1 hunks)
- components/jira/sources/issue-updated/issue-updated.mjs (1 hunks)
Files skipped from review due to trivial changes (22)
- components/jira/actions/add-attachment-to-issue/add-attachment-to-issue.mjs
- components/jira/actions/add-comment-to-issue/add-comment-to-issue.mjs
- components/jira/actions/add-multiple-attachments-to-issue/add-multiple-attachments-to-issue.mjs
- components/jira/actions/add-watcher-to-issue/add-watcher-to-issue.mjs
- components/jira/actions/assign-issue/assign-issue.mjs
- components/jira/actions/create-issue/create-issue.mjs
- components/jira/actions/create-version/create-version.mjs
- components/jira/actions/delete-project/delete-project.mjs
- components/jira/actions/get-all-projects/get-all-projects.mjs
- components/jira/actions/get-issue/get-issue.mjs
- components/jira/actions/get-task/get-task.mjs
- components/jira/actions/get-transitions/get-transitions.mjs
- components/jira/actions/get-user/get-user.mjs
- components/jira/actions/get-users/get-users.mjs
- components/jira/actions/list-issue-comments/list-issue-comments.mjs
- components/jira/actions/transition-issue/transition-issue.mjs
- components/jira/actions/update-comment/update-comment.mjs
- components/jira/actions/update-issue/update-issue.mjs
- components/jira/sources/events/events.mjs
- components/jira/sources/issue-created/issue-created.mjs
- components/jira/sources/issue-deleted/issue-deleted.mjs
- components/jira/sources/issue-updated/issue-updated.mjs
Files skipped from review as they are similar to previous changes (5)
- components/jira/actions/create-custom-field-options-context/create-custom-field-options-context.mjs
- components/jira/common/constants.mjs
- components/jira/common/utils.mjs
- components/jira/jira.app.mjs
- components/jira/package.json
e5bf878 to
c40659b
Compare
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 UI
Review profile: CHILL
Files selected for processing (1)
- components/jira/actions/add-attachment-to-issue/add-attachment-to-issue.mjs (1 hunks)
Files skipped from review due to trivial changes (1)
- components/jira/actions/add-attachment-to-issue/add-attachment-to-issue.mjs
c40659b to
91406aa
Compare
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 UI
Review profile: CHILL
Files selected for processing (27)
- components/jira/actions/add-attachment-to-issue/add-attachment-to-issue.mjs (1 hunks)
- components/jira/actions/add-comment-to-issue/add-comment-to-issue.mjs (1 hunks)
- components/jira/actions/add-multiple-attachments-to-issue/add-multiple-attachments-to-issue.mjs (1 hunks)
- components/jira/actions/add-watcher-to-issue/add-watcher-to-issue.mjs (1 hunks)
- components/jira/actions/assign-issue/assign-issue.mjs (1 hunks)
- components/jira/actions/create-custom-field-options-context/create-custom-field-options-context.mjs (1 hunks)
- components/jira/actions/create-issue/create-issue.mjs (1 hunks)
- components/jira/actions/create-version/create-version.mjs (1 hunks)
- components/jira/actions/delete-project/delete-project.mjs (1 hunks)
- components/jira/actions/get-all-projects/get-all-projects.mjs (1 hunks)
- components/jira/actions/get-issue/get-issue.mjs (1 hunks)
- components/jira/actions/get-task/get-task.mjs (1 hunks)
- components/jira/actions/get-transitions/get-transitions.mjs (1 hunks)
- components/jira/actions/get-user/get-user.mjs (1 hunks)
- components/jira/actions/get-users/get-users.mjs (1 hunks)
- components/jira/actions/list-issue-comments/list-issue-comments.mjs (1 hunks)
- components/jira/actions/transition-issue/transition-issue.mjs (1 hunks)
- components/jira/actions/update-comment/update-comment.mjs (1 hunks)
- components/jira/actions/update-issue/update-issue.mjs (1 hunks)
- components/jira/common/constants.mjs (1 hunks)
- components/jira/common/utils.mjs (2 hunks)
- components/jira/jira.app.mjs (3 hunks)
- components/jira/package.json (1 hunks)
- components/jira/sources/events/events.mjs (1 hunks)
- components/jira/sources/issue-created/issue-created.mjs (1 hunks)
- components/jira/sources/issue-deleted/issue-deleted.mjs (1 hunks)
- components/jira/sources/issue-updated/issue-updated.mjs (1 hunks)
Files skipped from review due to trivial changes (6)
- components/jira/actions/assign-issue/assign-issue.mjs
- components/jira/actions/get-all-projects/get-all-projects.mjs
- components/jira/actions/get-user/get-user.mjs
- components/jira/actions/update-comment/update-comment.mjs
- components/jira/package.json
- components/jira/sources/issue-created/issue-created.mjs
Files skipped from review as they are similar to previous changes (1)
- components/jira/actions/add-attachment-to-issue/add-attachment-to-issue.mjs
Additional comments not posted (31)
components/jira/sources/issue-deleted/issue-deleted.mjs (1)
7-7: Version update is correct.The version has been updated from "0.0.8" to "0.0.9". Ensure that the version update is consistent across the codebase.
components/jira/sources/issue-updated/issue-updated.mjs (1)
7-7: Version update is correct.The version has been updated from "0.0.8" to "0.0.9". Ensure that the version update is consistent across the codebase.
components/jira/common/constants.mjs (1)
37-37: New constantDEFAULT_LIMITadded.The constant
DEFAULT_LIMITwith a value of 50 has been added. Ensure that this constant is used appropriately in the codebase.components/jira/sources/events/events.mjs (1)
8-8: Version update is correct.The version has been updated from "0.0.8" to "0.0.9". Ensure that the version update is consistent across the codebase.
components/jira/actions/get-users/get-users.mjs (1)
7-7: LGTM! Version update approved.The version update from "0.0.3" to "0.0.4" is approved. No other issues found.
components/jira/actions/get-task/get-task.mjs (1)
7-7: LGTM! Version update approved.The version update from "0.1.8" to "0.1.9" is approved. No other issues found.
components/jira/actions/delete-project/delete-project.mjs (1)
7-7: LGTM! Version update approved.The version update from "0.1.8" to "0.1.9" is approved. No other issues found.
components/jira/actions/add-watcher-to-issue/add-watcher-to-issue.mjs (1)
6-6: LGTM! Version update approved.The version update from "0.0.7" to "0.0.8" is approved. No other issues found.
components/jira/actions/list-issue-comments/list-issue-comments.mjs (1)
7-7: Version update approved.The version update from "0.1.8" to "0.1.9" is consistent with the intended release or update.
components/jira/common/utils.mjs (2)
1-1: Import statement approved.The import of
ConfigurationErrorfrom@pipedream/platformis necessary for error handling in theparseArrayfunction.
25-45: Function implementation approved.The
parseArrayfunction is well-implemented with proper error handling, ensuring that invalid array objects are handled gracefully.components/jira/actions/create-custom-field-options-context/create-custom-field-options-context.mjs (5)
1-2: Import statements approved.The imports of
appfrom../../jira.app.mjsandutilsfrom../../common/utils.mjsare necessary for the functionality of the action.
5-9: Object export approved.The object export is well-structured and includes all necessary components for the action.
10-48: Props approved.The props are well-defined and necessary for the action, with accurate descriptions and types.
50-59: Method implementation approved.The
createCustomFieldOptionsContextmethod is well-implemented and necessary for creating custom field options context.
61-82: Run function implementation approved.The
runfunction is well-implemented with proper error handling and response management.components/jira/actions/add-multiple-attachments-to-issue/add-multiple-attachments-to-issue.mjs (1)
11-11: Version update approved.The version update from "0.0.8" to "0.0.9" is consistent with the intended release or update.
components/jira/actions/add-comment-to-issue/add-comment-to-issue.mjs (2)
8-8: Version update approved.The version has been updated from "0.1.8" to "0.1.9". This change is minor and likely includes bug fixes or small enhancements.
Line range hint
34-52:
Function implementation approved.The
runfunction is correctly implemented, with proper parsing of input properties and handling of the API call to add a comment to a Jira issue.components/jira/actions/get-transitions/get-transitions.mjs (2)
7-7: Version update approved.The version has been updated from "0.1.8" to "0.1.9". This change is minor and likely includes bug fixes or small enhancements.
Line range hint
34-43:
Function implementation approved.The
runfunction is correctly implemented, with proper handling of input properties and the API call to retrieve transitions for a Jira issue.components/jira/actions/transition-issue/transition-issue.mjs (2)
8-8: Version update approved.The version has been updated from "0.1.10" to "0.1.11". This change is minor and likely includes bug fixes or small enhancements.
Line range hint
34-52:
Function implementation approved.The
runfunction is correctly implemented, with proper parsing of input properties and handling of the API call to perform a transition for a Jira issue.components/jira/actions/create-issue/create-issue.mjs (2)
10-10: Version update approved.The version has been updated from "0.1.16" to "0.1.17". This change is minor and likely includes bug fixes or small enhancements.
Line range hint
34-64:
Function implementation approved.The
runfunction is correctly implemented, with proper parsing of input properties and handling of the API call to create a Jira issue.components/jira/actions/create-version/create-version.mjs (1)
7-7: Version update confirmed.The version number has been updated from "0.1.8" to "0.1.9". No other changes are introduced in this file.
components/jira/actions/get-issue/get-issue.mjs (1)
7-7: Version update confirmed.The version number has been updated from "0.1.10" to "0.1.11". No other changes are introduced in this file.
components/jira/actions/update-issue/update-issue.mjs (1)
11-11: Version update confirmed.The version number has been updated from "0.2.10" to "0.2.11". No other changes are introduced in this file.
components/jira/jira.app.mjs (3)
4-4: New import statement added.The import statement for
constantsfrom./common/constants.mjshas been added. Ensure this module is correctly implemented and used.
217-311: New properties and methods added.The properties
fieldIdandcontextIdhave been added with their respectiveoptionsmethods. Ensure these properties and methods are correctly implemented and follow best practices.
572-585: New methods added.The methods
getFieldsPaginatedandgetCustomFieldContextshave been added for API requests. Ensure these methods are correctly implemented and follow best practices.
GTFalcao
left a comment
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.
LGTM!
Added docs link Fixed path Fixed parsing
91406aa to
b85f3cd
Compare
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 UI
Review profile: CHILL
Files selected for processing (27)
- components/jira/actions/add-attachment-to-issue/add-attachment-to-issue.mjs (1 hunks)
- components/jira/actions/add-comment-to-issue/add-comment-to-issue.mjs (1 hunks)
- components/jira/actions/add-multiple-attachments-to-issue/add-multiple-attachments-to-issue.mjs (1 hunks)
- components/jira/actions/add-watcher-to-issue/add-watcher-to-issue.mjs (1 hunks)
- components/jira/actions/assign-issue/assign-issue.mjs (1 hunks)
- components/jira/actions/create-custom-field-options-context/create-custom-field-options-context.mjs (1 hunks)
- components/jira/actions/create-issue/create-issue.mjs (1 hunks)
- components/jira/actions/create-version/create-version.mjs (1 hunks)
- components/jira/actions/delete-project/delete-project.mjs (1 hunks)
- components/jira/actions/get-all-projects/get-all-projects.mjs (1 hunks)
- components/jira/actions/get-issue/get-issue.mjs (1 hunks)
- components/jira/actions/get-task/get-task.mjs (1 hunks)
- components/jira/actions/get-transitions/get-transitions.mjs (1 hunks)
- components/jira/actions/get-user/get-user.mjs (1 hunks)
- components/jira/actions/get-users/get-users.mjs (1 hunks)
- components/jira/actions/list-issue-comments/list-issue-comments.mjs (1 hunks)
- components/jira/actions/transition-issue/transition-issue.mjs (1 hunks)
- components/jira/actions/update-comment/update-comment.mjs (1 hunks)
- components/jira/actions/update-issue/update-issue.mjs (1 hunks)
- components/jira/common/constants.mjs (1 hunks)
- components/jira/common/utils.mjs (2 hunks)
- components/jira/jira.app.mjs (3 hunks)
- components/jira/package.json (1 hunks)
- components/jira/sources/events/events.mjs (1 hunks)
- components/jira/sources/issue-created/issue-created.mjs (1 hunks)
- components/jira/sources/issue-deleted/issue-deleted.mjs (1 hunks)
- components/jira/sources/issue-updated/issue-updated.mjs (1 hunks)
Files skipped from review due to trivial changes (7)
- components/jira/actions/add-comment-to-issue/add-comment-to-issue.mjs
- components/jira/actions/add-watcher-to-issue/add-watcher-to-issue.mjs
- components/jira/actions/assign-issue/assign-issue.mjs
- components/jira/actions/create-issue/create-issue.mjs
- components/jira/actions/get-transitions/get-transitions.mjs
- components/jira/actions/transition-issue/transition-issue.mjs
- components/jira/common/constants.mjs
Files skipped from review as they are similar to previous changes (19)
- components/jira/actions/add-attachment-to-issue/add-attachment-to-issue.mjs
- components/jira/actions/add-multiple-attachments-to-issue/add-multiple-attachments-to-issue.mjs
- components/jira/actions/create-version/create-version.mjs
- components/jira/actions/delete-project/delete-project.mjs
- components/jira/actions/get-all-projects/get-all-projects.mjs
- components/jira/actions/get-issue/get-issue.mjs
- components/jira/actions/get-task/get-task.mjs
- components/jira/actions/get-user/get-user.mjs
- components/jira/actions/get-users/get-users.mjs
- components/jira/actions/list-issue-comments/list-issue-comments.mjs
- components/jira/actions/update-comment/update-comment.mjs
- components/jira/actions/update-issue/update-issue.mjs
- components/jira/common/utils.mjs
- components/jira/jira.app.mjs
- components/jira/package.json
- components/jira/sources/events/events.mjs
- components/jira/sources/issue-created/issue-created.mjs
- components/jira/sources/issue-deleted/issue-deleted.mjs
- components/jira/sources/issue-updated/issue-updated.mjs
Additional comments not posted (9)
components/jira/actions/create-custom-field-options-context/create-custom-field-options-context.mjs (9)
1-2: LGTM! Imports are appropriate and necessary.The imports from
../../jira.app.mjsand../../common/utils.mjsare essential for the action's functionality.
4-9: LGTM! Metadata is well-defined.The metadata provides a clear and concise description of the action.
10-11: LGTM!appprop is necessary.The
appprop is essential for the action to interact with Jira.
12-17: LGTM!cloudIdprop is necessary.The
cloudIdprop is essential for identifying the Jira cloud instance.
18-31: LGTM!fieldIdprop is necessary.The
fieldIdprop is essential for identifying the custom field in Jira.
32-43: LGTM!contextIdprop is necessary.The
contextIdprop is essential for identifying the context for the custom field options.
44-48: LGTM!optionsprop is necessary.The
optionsprop is essential for specifying the custom field options to be created.
50-59: LGTM!createCustomFieldOptionsContextmethod is well-defined.The method is essential for making the API request to create custom field options context.
61-82: LGTM! The run function is well-structured.The function handles the action's execution flow appropriately and ensures proper parsing of the options.
|
/approve |
WHY
Resolves #12528
Summary by CodeRabbit