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.

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

Adding in functionality to return Subtasks and Components to search_issues #60

Merged

Conversation

dylan-bitovi
Copy link
Contributor

get_issue currently has the ability to enable flags to include an issue's Subtasks and Components.

However, this is also an available flag for search via the API, where you can include Subtasks and Components for issues which are returned by the JQL search query.

This PR adds the include_subtasks and include_components flags to the search_issue action. I've tested these changes locally by modifying the existing action, and there seems to be no impact on other functionality by adding these flags.

CHANGES.md has been updated, and pack.yaml version has been incremented to 2.4.1.

@dylan-bitovi dylan-bitovi requested review from Kami and removed request for a team August 21, 2023 19:38
Copy link
Collaborator

@floatingstatic floatingstatic left a comment

Choose a reason for hiding this comment

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

One minor thing to fix in the changelog, otherwise looks good to me. I think this change makes sense given get_issues supports these two additional parameters and to_issue_dict() is already wired to accept these flags. I tested in st2-docker installing the pack from your fork and every seems to work just fine. I can approve and merge this change after fixing the version in changelog. Thanks for the contribution!

CHANGES.md Outdated Show resolved Hide resolved
@dylan-bitovi
Copy link
Contributor Author

@floatingstatic Updated! Thanks.

@floatingstatic floatingstatic merged commit 3802b52 into StackStorm-Exchange:master Aug 22, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants