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

feat: Add GitLab Reaction Emojis on MR Comments #3456

Merged
merged 6 commits into from
Jun 13, 2023

Conversation

X-Guardian
Copy link
Contributor

what

Adds reaction emojis to Atlantis command comments on GitLab merge requests.

  • Replace stub GitLab ReactToComment function with a call to g.Client.AwardEmoji.CreateMergeRequestAwardEmojiOnNote
  • GitLab requires the MergeRequestIID (pullNum) and NoteID (commentID) properties for the call. pullNum was not currently being passed to the ReactToComment function so this has been added to all the VCS ReactToComment functions and calls. commentID has been added as an output of ParseGitlabMergeRequestCommentEvent.

Missing documentation for the emoji-reaction property has also been added.

why

Bring Atlantis GitLab VCS support more inline with the GitHub VCS support.

tests

No emoji-reaction property

image

thumbsup emoji-reaction property

image

references

@X-Guardian X-Guardian requested a review from a team as a code owner May 27, 2023 13:22
@X-Guardian
Copy link
Contributor Author

This PR is ready for review.

Copy link
Member

@nitrocode nitrocode left a comment

Choose a reason for hiding this comment

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

Thank you!

@nitrocode nitrocode added this to the v0.24.3 milestone Jun 3, 2023
@nitrocode nitrocode merged commit 86607d8 into runatlantis:main Jun 13, 2023
mtavaresmedeiros pushed a commit to mtavaresmedeiros/atlantis that referenced this pull request Jul 3, 2023
* Add GitLab Reaction Emoji

* Update docs

* update docs

* Update event_parser_test
ijames-gc pushed a commit to gocardless/atlantis that referenced this pull request Feb 13, 2024
* Add GitLab Reaction Emoji

* Update docs

* update docs

* Update event_parser_test
ijames-gc pushed a commit to gocardless/atlantis that referenced this pull request Feb 13, 2024
* Add GitLab Reaction Emoji

* Update docs

* update docs

* Update event_parser_test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: GitLab Reaction Emojis on MR Comments
2 participants