Skip to content
Snippets Groups Projects

Legal tag update event grid

Merged Akansha Rajput[Microsoft] requested to merge LegalTagUpdateEventGrid into master

All Submissions:


  • [YES] I have added an explanation of what changes in this merge do and why we should include it?
  • [YES] I have updated the documentation accordingly.
  • [YES] I have added tests to cover my changes.
  • [YES] All new and existing tests passed.
  • [YES] My code follows the code style of this project.
  • [YES] I ran lint checks locally prior to submission.

What is the current behavior?


Currently legal tag changes are publish on service bus.

What is the new behavior?


Enabling publishing of legal tag changed event to event grid.

  • Adding a event grid config.
  • Adding publishToEventGrid logic in LegalTagPublisherImpl.java
  • Adding a unit test to check event grid.

Does this introduce a breaking change?


  • [NO]

Any relevant logs, error output, etc?


(If it’s long, please paste to https://ghostbin.com/ and insert the link here.)

Other information


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
  • added 1 commit

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • added 4 commits

    Compare with previous version

  • added 1 commit

    • 076b9b69 - Make a publishToEventGrid private function.

    Compare with previous version

  • @arajput - This MR looks fine for me. See if you want to get the complete E2E working and then get this merged or merge with EG publish disabled and later iterate on for any issues during wiring with storage service flow and then enable by default?

  • resolved all threads

  • added 6 commits

    Compare with previous version

  • Krishna Nikhil Vedurumudi approved this merge request

    approved this merge request

  • Komal Makkar
  • Komal Makkar
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading