Skip to content

Conversation

@Nityanand13
Copy link

No description provided.

raghavaggarwal2308 and others added 30 commits November 1, 2022 14:11
1. Updated API to support creating pipeline subscriptions.
2. Updated the UI for subscription modal to display pipeline event types.
3. Updated the UI of subscription card for for displaying pipeline subscriptions.
…eployment started" and "Release deployment completed"
…pipeline job request from requested user via MM post.
avas27JTG and others added 18 commits February 6, 2023 17:14
* [MI-2669_1]: Added a top level CSS class as a unique selector

* [MI-2669]: Added top level class selector for components and review fixes

---------

Co-authored-by: Abhishek Verma <[email protected]>
…e central login into it, formatted code (#32)

* [MI-2669]: Created a hook for intercepting redux changes and moved the central login into it, formatted code

* [MI-2669]: Modified comment

* [MI-2669]: Review fixes

---------

Co-authored-by: Abhishek Verma <[email protected]>
…ser for oAuth. (#35)

* [MI-2748]: Implemented 1:1 mapping between MM user and Azure DevOps user for oAuth.

* [MI-2748]: Added 1 test case and http response code

* [MI-2748]: Fixed condition

* [MI-2748]: Fixed token refresh logic

* [MI-2748]: Fixed CI

* [MI-2748]: Fixed CI

* [MI-2748]: Review fixes

* [MI-2748]: Fixed CI

* [MI-2748]: Review fixes

* [MI-2748]: Review fixes

---------

Co-authored-by: Abhishek Verma <[email protected]>
…om plugin's server. (#36)

* [MI-2750]: Implemented fetching channel list on webapp and remove from plugin's server.

* [MI-2750]: Review fixes

* [MI-2750]: Changed comment message

---------

Co-authored-by: Abhishek Verma <[email protected]>
…er (#37)

* [MI-2765]: Used @mattermost/client for making webapp calls to MM server

* [MI-2765]: Review fix

* [MI-2765]: Review fix

* [MI-2765]: Removed Mattermost API call RTK service

* [MI-2765]: Fixed CI

---------

Co-authored-by: Abhishek Verma <[email protected]>
1. Added isUserDeleted feature
1. Improved code quality
1. Improved code quality
1. Removed the boolean returned from DeleteUser
@hanzei
Copy link
Contributor

hanzei commented Apr 6, 2023

@Nityanand13 The diff in the PR seems off. Is it still valid?

@avas27JTG
Copy link
Contributor

@Nityanand13 The diff in the PR seems off. Is it still valid?

Yes @hanzei

@hanzei
Copy link
Contributor

hanzei commented Apr 18, 2023

@avas27JTG Could you please rebase onto master then and ensure the diff is somewhat clean?

@avas27JTG
Copy link
Contributor

avas27JTG commented Apr 20, 2023

@avas27JTG Could you please rebase onto master then and ensure the diff is somewhat clean?

Sure
also, there are too many commit differences between this and the master branch, will it be okay if we sync with the latest master by taking the pull to avoid complexities that may occur in rebasing?

@avas27JTG
Copy link
Contributor

@hanzei This branch is synced with latest master

@hanzei hanzei added 2: Dev Review Requires review by a core committer 3: QA Review Requires review by a QA tester and removed Work In Progress Not yet ready for review labels May 31, 2023
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 3: QA Review Requires review by a QA tester

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants