-
Notifications
You must be signed in to change notification settings - Fork 8
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
BCMOHAD-23189 LRA EMPI API Changes #1408
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved @Nikhiljeenru @RanadheerRG
cc @NataliaNikishina
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Second review completed. approved @Nikhiljeenru @RanadheerRG
95b9d06
to
319deae
Compare
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some changes will be required later -- this LRA credential will likely be the SAT LRA credential with another one for EDRD LRA credential. Please proceed with testing and demonstration to the business team (Megan/Karen). Approved @Nikhiljeenru @RanadheerRG @NataliaNikishina
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved @Nikhiljeenru
|
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # BCMOHAD-23189
Type of change
List high-level changes as bullet items
LRA EMPI API Changes and its dependent components and required permission sets added.
Below are some examples
Deployment Tracker
List all the metadata that is pushed in this commit/PR. Full URL should be fine.
#1408
Checklist: