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

Subtasks are not supported in subscription modal #993

Closed
mickmister opened this issue Oct 20, 2023 · 4 comments · Fixed by #1095
Closed

Subtasks are not supported in subscription modal #993

mickmister opened this issue Oct 20, 2023 · 4 comments · Fixed by #1095
Milestone

Comments

@mickmister
Copy link
Contributor

At the moment, we don't support selecting "subtask types" as "issue types" in the subscription modal. It seems this is on purpose based on this line:

return project.issuetypes.filter((i) => !i.subtask);

The task here is to look into the feasibility of supporting subtasks in the subscriptions feature. This would also affect the create issue modal if we were to change the line shown above.

@Kshitij-Katiyar
Copy link
Contributor

Kshitij-Katiyar commented Jun 28, 2024

Hi @mickmister,
I explored this further and found that removing the filter from this line will have the following effects:

Subscriptions

  • Subtasks will be available in the Issue Type dropdown of the Add Jira Subscription modal.
    Screenshot from 2024-06-28 18-04-05
  • The subscriptions feature works correctly, and notifications are received as expected.
    Screenshot from 2024-06-28 17-16-43

Create Issue

  • Subtasks will be available in the Issue Type dropdown of the Create Jira Issue modal.
  • With the current flow, attempting to create an issue with a subtask will result in the following error:
    Please create your Jira issue manually. The project you tried to create an issue for has required fields this plugin does not yet support: Parent
    Screenshot from 2024-06-28 17-16-19
  • To allow the creation of a Jira issue for a subtask, we will need to:
  1. Add Parent Issue in the createJiraIssue API call payload.
  2. Include Parent Issue as an input field in the Create Jira Issue modal.

@mickmister
Copy link
Contributor Author

@Kshitij-Katiyar Okay so subscriptions are working correctly with subtasks after that small change 👍 Though it leaves the "create issue" modal in a partially broken state with no support for subtasks. How much effort do you think it would take to support subtasks in the "create issue" modal? We already have an issue searching thing per project, so maybe we could just reuse that code/component for searching for the parent issue.

Maybe we can split this up into 2 PRs:

  • One that makes it so the !i.subtask line only runs for the "create issue" modal. This way subscriptions can immediately support subtasks
  • One that implements support for the "create issue" modal. I think we can make this one HW

What do you think @Kshitij-Katiyar?

@raghavaggarwal2308
Copy link
Contributor

@Kshitij-Katiyar Okay so subscriptions are working correctly with subtasks after that small change 👍 Though it leaves the "create issue" modal in a partially broken state with no support for subtasks. How much effort do you think it would take to support subtasks in the "create issue" modal? We already have an issue searching thing per project, so maybe we could just reuse that code/component for searching for the parent issue.

@mickmister I think it will take about 10-12 hrs for adding support for sub-tasks in create issue modal.

Maybe we can split this up into 2 PRs:

One that makes it so the !i.subtask line only runs for the "create issue" modal. This way subscriptions can immediately support subtasks
One that implements support for the "create issue" modal. I think we can make this one HW
What do you think @Kshitij-Katiyar?

@mickmister That sounds good. We can pick up the second task if there are any requests for that.

@Kshitij-Katiyar
Copy link
Contributor

@mickmister Created an issue for the second task. #1093

@raghavaggarwal2308 raghavaggarwal2308 moved this from Backlog to Submitted / In Review in Brightscout Plugin Maintenance Jul 17, 2024
@raghavaggarwal2308 raghavaggarwal2308 added this to the v4.2.0 milestone Jul 17, 2024
raghavaggarwal2308 pushed a commit that referenced this issue Jul 25, 2024
* [GH-993]: added support for subtasks in subscriptions

* [GH-993]: updated an additional condition of getting issueTypes

* [GH-993]: Added types for the subtask parameter in getIssueType function
@github-project-automation github-project-automation bot moved this from Submitted / In Review to Done in Brightscout Plugin Maintenance Jul 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging a pull request may close this issue.

3 participants