-
Notifications
You must be signed in to change notification settings - Fork 3
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
MDCT-1993 api updates #4180
MDCT-1993 api updates #4180
Conversation
get: jest | ||
.fn() | ||
.mockImplementationOnce(() => ({ | ||
Item: undefined, | ||
})) | ||
.mockImplementation(() => ({ | ||
Item: mockBannerResponse, | ||
})), |
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.
Note: I'm planning on opening a stacked branch to improve this
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.
Stacked PR here: #4181
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.
nice ✨ I was able to use the Postman Collection too!
Description
MDCT-1993 modify api to match standards
How to test
Try out the mcr postman collection from confluence! Check that:
ping me if you want help!
Changed Dependencies
none
Code author checklist
Reviewer checklist (two different people)