-
Notifications
You must be signed in to change notification settings - Fork 2
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
changes for heal admin portal #87
Conversation
@@ -54,7 +54,7 @@ const DirectPayForm = (props) => { | |||
var data = { | |||
"queryStringParameters": { | |||
"method": "confirmBillingID", | |||
"brh_data": { | |||
"pp_data": { |
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.
will this change affects the instance that currently deployed in BRH?
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.
Yes. I wanted to go with the generic name for both BRH and Heal so Changes are needed on both. Currently, this is on Dev, once we move Heal to Production, These changes should be made on BRH and Heal Production Workspace Admin Env
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.
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.
and @urvi-occ just to confirm, even though this change will affect BRH in the future, it will not break BRH now right? Since I assume the prod backend that BRH is pointing to is still expecting the old data format?
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.
It would not break the BRH Production currently, but Dev BRH would not work without these changes
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.
Thanks for clarfying, I'll merge this and deploy to qa-heal momentarily
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.
Thanks
New Features
Changes for Heal and BMH admin portal
brh_data
topp_data
Bug Fixes
response.body[0]["Message"]["statusCode"]
wasn't able to findstatusCode
Improvements
updateRedirectHome
for Direct Pay Form