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-830]: Explained the jira subscription modal in the doc #904

Merged
merged 11 commits into from
Jan 18, 2024

Conversation

Nityanand13
Copy link
Contributor

Summary

  1. Explained the way to create and use the Jira subscription modal.
  2. Linked to the docs from the frontend

Ticket Link

Issue: #830

* [MI-2608]: Explained the jira subscription modal in the doc.

* [MI-2608]:Review fixes done
1. Modified the doc
2. Connected the doc from frontend

* [MI-2608]: Review fixes done
1. Modified the administrator guide doc
@mattermost-build
Copy link
Contributor

Hello @Nityanand13,

Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@hanzei hanzei added the 2: Dev Review Requires review by a core committer label Jan 24, 2023
@hanzei hanzei linked an issue Jan 24, 2023 that may be closed by this pull request
Comment on lines 64 to 67
6. To be more specific, you can add filters as well. You can either include or exclude the particular filter by adding its type or name. For now, we have 3 types of filters :
* Epic Link
* Labels
* Priority
Copy link
Contributor

Choose a reason for hiding this comment

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

For now, we have 3 types of filters :

There many other filter fields available, including custom fields. As mentioned in #583, "Basically all of them except date pickers." I know that Status is also not currently supported.

What fields have you tried to use that are not available in the feature?

Copy link
Contributor Author

@Nityanand13 Nityanand13 Jan 30, 2023

Choose a reason for hiding this comment

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

@mickmister
After exploring now I am getting total of 10 types of filters:

  1. Affects versions
  2. Epic Link
  3. Fix versions
  4. Labels
  5. Priority

And a few custom fields as well.

  1. Checkboxes
  2. Labels
  3. Radio Buttons
  4. Select List (multiple choices)
  5. Select List (single choice)

Nityanand13 added a commit to Brightscout/mattermost-plugin-jira that referenced this pull request Feb 1, 2023
* [MI-2698]: Review fixes done of PR mattermost#904

* [MI-2698]: Review fixes done
1. Improved the content of a administrative guide doc
@mattermost-build
Copy link
Contributor

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

@mickmister mickmister removed 2: Dev Review Requires review by a core committer Lifecycle/1:stale labels Feb 20, 2023
Copy link
Contributor

@mickmister mickmister left a comment

Choose a reason for hiding this comment

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

@Nityanand13 For some reason GitHub won't let me edit the files in the PR. Can you please update with these changes?

@hanzei hanzei added 2: Dev Review Requires review by a core committer Awaiting Submitter Action Blocked on the author labels Feb 21, 2023
Nityanand13 added a commit to Brightscout/mattermost-plugin-jira that referenced this pull request Feb 21, 2023
@codecov-commenter
Copy link

codecov-commenter commented Feb 22, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (249614e) 33.32% compared to head (3bd1923) 33.29%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #904      +/-   ##
==========================================
- Coverage   33.32%   33.29%   -0.03%     
==========================================
  Files          53       53              
  Lines        7944     7944              
==========================================
- Hits         2647     2645       -2     
- Misses       5070     5072       +2     
  Partials      227      227              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Nityanand13
Copy link
Contributor Author

@Nityanand13 For some reason GitHub won't let me edit the files in the PR. Can you please update with these changes?

Sure @mickmister I have updated these changes.

@mattermost-build
Copy link
Contributor

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

@hanzei hanzei requested review from mickmister and removed request for mickmister September 1, 2023 10:03
@hanzei
Copy link
Collaborator

hanzei commented Sep 1, 2023

@Nityanand13 Would you please merge master into your PR?

@mattermost-build
Copy link
Contributor

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

@hanzei
Copy link
Collaborator

hanzei commented Sep 27, 2023

@Nityanand13 Gentle reminder

@ayusht2810
Copy link
Contributor

@hanzei resolved the conflicts.

@ayusht2810
Copy link
Contributor

@hanzei fixed the review comment. Please re-review

@hanzei hanzei added 4: Reviews Complete All reviewers have approved the pull request 3: QA Review Requires review by a QA tester and removed 2: Dev Review Requires review by a core committer Awaiting Submitter Action Blocked on the author Lifecycle/1:stale 4: Reviews Complete All reviewers have approved the pull request labels Jan 4, 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.

Tested this PR and verified the "Learn more" link to redirect the user to the docs. Approved.

@hanzei hanzei added 4: Reviews Complete All reviewers have approved the pull request and removed 3: QA Review Requires review by a QA tester labels Jan 9, 2024
@mickmister mickmister merged commit ea9ab23 into mattermost:master Jan 18, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4: Reviews Complete All reviewers have approved the pull request Contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Jira subscriptions modal needs explanation in the docs
9 participants