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

MM-15535 - UI updates for Jira 2 #71

Merged
merged 5 commits into from
May 24, 2019
Merged

MM-15535 - UI updates for Jira 2 #71

merged 5 commits into from
May 24, 2019

Conversation

asaadmahmood
Copy link
Contributor

@asaadmahmood asaadmahmood commented May 23, 2019

Some minor changes.

@jasonblais jasonblais added the 2: Dev Review Requires review by a core committer label May 23, 2019
@jasonblais jasonblais added this to the v2.0 milestone May 23, 2019
Copy link
Contributor

@levb levb left a comment

Choose a reason for hiding this comment

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

So how do we fix the lint issue? @crspeller @cpoile ideas?

webapp/src/components/input.jsx Outdated Show resolved Hide resolved
@hanzei hanzei requested a review from levb May 24, 2019 09:17
@levb levb added 4: Reviews Complete All reviewers have approved the pull request and removed 2: Dev Review Requires review by a core committer labels May 24, 2019
@levb levb merged commit 55a16f9 into jira2 May 24, 2019
@asaadmahmood asaadmahmood deleted the MM-15535 branch May 24, 2019 22:04
levb pushed a commit that referenced this pull request Aug 10, 2020
* Use connectInstances array for userConnected redux selector

* add null check
levb added a commit that referenced this pull request Aug 10, 2020
* iterate through instances when printing out the channel subscriptions
update tests for new printing output format

don't require --intance for subscribelist

* Update tests

* shorten field name

* several PR feedback fixes

* remove unused variable
rename map variable

* Disconnect users on instance uninstall (#41)

* [GH-4] add EE license check (#22)

* restrict plugin activation based on license

* run go mod tidy -v

* wip

* add enterprise package

* rename license naming to enterprise
if user does not have enterprise license, check number of instances
  installed before allowed to install another cloud or server instance

* remove function

* fix review feedback

* Add license checking tests

* wip

* wip by Lev

* wip

* wip

* remove comments

* remove function to return pointer to true bool value

* getMockInstanceStoreKV(0) should return initialized empty store
was returning instance store with one instance
correct all occurences call of getMockInstanceStoreKV(0) to call with 1
instanance. This fixes many failing tests
Add tests for InstallInstance function with extensive license checking

* fix linting

Co-authored-by: Lev Brouk <lev@mattermost.com>

* Fixed #31, #33 (#54)

- Use plugin URL as the "home" for app links
- Added extra logging for suspicious callbacks
- Use templates for `/jira instance install` output

* Fix test (#64)

* GH-21 Fixed websocket update on user disconnect, instance uninstall (#61)

* Fixed websocket update on user disconnect

* Revert "Fixed websocket update on user disconnect"

This reverts commit 84ca4b27ca2a3ed55408bf359786b3162dc1b29a.

* Fixed empty set refresh, defaulting in commands

* fixed instance status update

* GH-49: Fixed webhook, transition commands (#67)

* Use connectInstances array for userConnected redux selector (#71)

* Use connectInstances array for userConnected redux selector

* add null check

* [GH-45] Make sure frontend has up-to-date default instance data (#65)

* fetch instances on modal open

* Fix errors regarding redux update and missing channelId prop

* Get instances when subscribe modal opens

* fix tests

* Handle case where default instance value in frontend is stale (uninstalled)

* Make if statement more safe

* Make if statement more safe

* fix test

* lint

* GH-60: Fixed multi-work command alias parsing in webapp (#72)

* Fixed --instance

Co-authored-by: Lev <1187448+levb@users.noreply.github.com>
Co-authored-by: Lev Brouk <lev@mattermost.com>
Co-authored-by: Michael Kochell <6913320+mickmister@users.noreply.github.com>
mickmister pushed a commit that referenced this pull request Feb 9, 2024
* [MI-3483] Added issue status field in subscription modal (#71)

* [MI-3483] Added the status field in subscription modal

* [MI-3483] Code refactoring

* [MI-3483] Fix testcases

* [MI-3483] Added error handling and code refactoring

* [MI-3483] Review fixes

* [MI-3483] Review fixes

* [MI-3483] Review fixes

* [MI-3597] Moved the issue status field into filters section of modal (#72)

* [MI-3597] Moved the issue status field into filters section of modal

* [MI-3597] FIxed a minor issue

* [MI-3597] Review fix

* [MI-3628] Updated the code to return the status details in the meta data API (#74)

* [MI-3628] Updated the code to return the status details in the meta data API instead of making a separate API call.

* [MI-3628] Review fixes: Small code optimization

* [MI-3690] Review fixes
- Added testcases for status field.
- Fixed issue: create meta info API not working for Jira server instance

* [MM-4] Review fixes: Fixed some variable names

* [MM-4] Updated the test data files and test cases

* [MM-71] Fix issue: status filter not working with issue created event
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants