Skip to content
Snippets Groups Projects

feat: thread google chat messages (fixes #27823)

Merged Chetan Sarva requested to merge csarva/gitlab:feature/thread-google-chat-messages into master

What does this MR do and why?

Threads messages sent to the Google Chat integration with the proper context. For example, notes and issue changes on a given issue will appear as a single chat thread rather than disparate messages.

Screenshots or screen recordings

Screen_Shot_2022-07-16_at_6.06.34_PM

How to set up and validate locally

  1. Enable Google Chat (hangouts) integration for a project
  2. Create a new issue within the project
  3. Add some comments to the issue, close it, reopen, etc
  4. Similarly for merge requests
  5. Push multiple changes to the default branch

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Bojan Marjanovic
  • Chetan Sarva added 2 commits

    added 2 commits

    • e2ec8c2a - feat: support threading for wiki and pipeline message types
    • b25e1b15 - fix: adopt style suggestions from review

    Compare with previous version

  • Chetan Sarva added 1 commit

    added 1 commit

    • edb2a15d - fix: use string concat (perf)

    Compare with previous version

  • Bojan Marjanovic resolved all threads

    resolved all threads

  • Bojan Marjanovic requested review from @.luke and removed review request for @bmarjanovic

    requested review from @.luke and removed review request for @bmarjanovic

  • Bojan Marjanovic approved this merge request

    approved this merge request

  • Luke Duncalfe
  • Luke Duncalfe
  • Luke Duncalfe
  • Luke Duncalfe removed review request for @.luke

    removed review request for @.luke

  • requested review from @kushalpandya

  • Kushal Pandya removed review request for @kushalpandya

    removed review request for @kushalpandya

  • @jannik_lehmann, this Community contribution is ready for review.

    • Do you have capacity and domain expertise to review this? We are mindful of your time, so if you are not able to take this on, please re-assign to one or more other reviewers.
    • Add the workflowin dev label if the merge request needs action from the author.
  • requested review from @jannik_lehmann

  • Jannik Lehmann requested review from @dblessing and removed review request for @jannik_lehmann

    requested review from @dblessing and removed review request for @jannik_lehmann

  • Drew Blessing removed review request for @dblessing

    removed review request for @dblessing

  • Chetan Sarva added 2 commits

    added 2 commits

    • ba1df785 - test: use more restrictive url regex
    • 2d5433e4 - test: added unit tests for google chat thread key param

    Compare with previous version

  • Author Contributor
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading