Legal tag update event grid
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
Activity
- Resolved by Akansha Rajput[Microsoft]
- Resolved by Akansha Rajput[Microsoft]
- Resolved by Akansha Rajput[Microsoft]
- Resolved by Akansha Rajput[Microsoft]
- Resolved by Akansha Rajput[Microsoft]
- Resolved by Akansha Rajput[Microsoft]
- Resolved by Akansha Rajput[Microsoft]
added 4 commits
-
c6b145d2...272e40e5 - 2 commits from branch
master
- 0f1bb87f - Merge branch 'master' of...
- 5de11ef9 - Update event grid message format.
-
c6b145d2...272e40e5 - 2 commits from branch
@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?
changed milestone to %M7 - Release 0.10
added Azure label
added 6 commits
-
076b9b69...2c62b2bc - 4 commits from branch
master
- c8644b31 - Merge branch 'master' of...
- bef813b8 - Setting publishing to EG disabled
-
076b9b69...2c62b2bc - 4 commits from branch
- Resolved by Akansha Rajput[Microsoft]
- Resolved by Akansha Rajput[Microsoft]
Please register or sign in to reply