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

Added the test cases for the updated LHS of gitlab. #157

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

arush-vashishtha
Copy link
Contributor

Summary

This PR consists the test cases for the following scenarios,

  • Assigned merge request instead of open merge request in the LHS of Gitlsb on MM.
  • Updated icons in the LHS of Gitlab.
  • Updated names in the tooltip for the icons in the LHS of Gitlab.

* [MM-86]: added the test cases for the updated LHS of gitlab.

* [MM-86]: Fixed the comments by Aayush.
@arush-vashishtha arush-vashishtha deleted the LHS_gitlab branch January 16, 2024 13:23
@arush-vashishtha arush-vashishtha restored the LHS_gitlab branch January 16, 2024 13:24
@lindalumitchell lindalumitchell added the 2: QA Review Requires review by a QA tester label Apr 30, 2024
@DHaussermann DHaussermann self-requested a review May 3, 2024 16:55
Copy link
Contributor

@DHaussermann DHaussermann left a comment

Choose a reason for hiding this comment

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

Looks like 2 minor updates are need here as the wrong plugin is referenced.


**Step 1**

1. Connect the MM account to your Gitlab account by running the slash command `/zoom connect`.
Copy link
Contributor

Choose a reason for hiding this comment

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

0/5 Maybe I'm misunderstanding but this looks like the command should be updated for the GitLab plugin.


**Step 2**

1. Connect the MM account to your Gitlab account by running the slash command `/zoom connect`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Same issue as above. I looks like the wrong plugin is referenced.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2: QA Review Requires review by a QA tester
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants