Added Groups for Notification Service - Githubissues

Azure / osdu-infrastructure

OSDU Azure Infrastructure as Code

Apache License 2.0

42 stars 39 forks source link

Added Groups for Notification Service #196

Closed komakkar closed 4 years ago

komakkar commented 4 years ago

The following are added to UserInfo and TenantInfo.

Pubsub
Ops
Cron

All Submissions:

[YES/NO] Have you added an explanation of what your changes do and why you'd like us to include them?
[YES/NO] I have updated the documentation accordingly.
[YES/NO/NA] I have added tests to cover my changes.
[YES/NO/NA] All new and existing tests passed.
[YES/NO/NA] I have formatted the terraform code. (terraform fmt -recursive && go fmt ./...)

Current Behavior or Linked Issues

https://github.com/Azure/osdu-infrastructure/issues/197

Does this introduce a breaking change?

[YES/NO]

Other information

ghost commented 4 years ago

All CLA requirements met.

danielscholl commented 4 years ago

Can you rebase in the changes that already exist to resolve the conflicts?

komakkar commented 4 years ago

Needs an Issue in Gitlab and then determine the proper place for the changes.

I created an issue in github itself. I will make the issue in Gitlab and close the github ones.

komakkar commented 4 years ago

As the repo is migrating to Gitlab, moved the changes to Gitlab