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

[GH-993]: added support for subtasks in subscriptions #1095

Merged
merged 3 commits into from
Jul 25, 2024

Conversation

Kshitij-Katiyar
Copy link
Contributor

@Kshitij-Katiyar Kshitij-Katiyar commented Jul 4, 2024

Summary

Added functionality to support subtasks Jira subscription.

Screenshots

Old

Screenshot from 2024-07-04 11-26-52

New

Screenshot from 2024-06-28 18-04-05
Screenshot from 2024-06-28 17-16-43

Issues

Fixes #993
Fixes #859

@Kshitij-Katiyar
Copy link
Contributor Author

@mickmister Added the first part of the changes as per our discussion on #993. Please have a look

@hanzei hanzei added the 2: Dev Review Requires review by a core committer label Jul 4, 2024
Copy link
Member

@wiggin77 wiggin77 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Copy link

@AayushChaudhary0001 AayushChaudhary0001 left a comment

Choose a reason for hiding this comment

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

@raghavaggarwal2308 found a issue while testing this PR.
Issue: After creating the subtask, added subscriptions are not shown.

Step to reproduce:

  1. Create a new JIRA subscription for any channel
  2. Select the subtask for that subscription
  3. Create a new subtask
  4. Check the bot message on the channel

Expected: Subscription part is present in the bot message
Actual: Subscription part is missing in the bot message

Note: Mattermost Version: 8.1.0

@raghavaggarwal2308
Copy link
Contributor

@AayushChaudhary0001 You have to enable the below setting in system console to see that:
image

@raghavaggarwal2308 raghavaggarwal2308 added this to the v4.2.0 milestone Jul 18, 2024
Copy link

@AayushChaudhary0001 AayushChaudhary0001 left a comment

Choose a reason for hiding this comment

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

Working fine, Approved

@raghavaggarwal2308 raghavaggarwal2308 merged commit fe02ece into master Jul 25, 2024
9 checks passed
@raghavaggarwal2308 raghavaggarwal2308 deleted the GH-993 branch July 25, 2024 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2: Dev Review Requires review by a core committer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Subtasks are not supported in subscription modal Subtasks notification not working
6 participants