-
Notifications
You must be signed in to change notification settings - Fork 7
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
AUT-2393: Axios upgrade from 0.25.0 to 0.1.6.8 #1555
Conversation
6cd8480
to
bf459a7
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.
Could we reword the commit to remove the WIP comment? Also, do we need a particular rollout plan? Have we tested this in some environment?
7de290f
to
6ce7bed
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.
Looking good. Just trying to assess if any more changes are required / we're not going to have hidden issues around. I've one suggestion for the moment.
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.
Just ran through some smoke testing and seen the flows work when targeting the build env. Approved.
2a65cdd
to
7733f14
Compare
- Resolve ts error enforcing to implement other missing properties of AxiosHeaders
What
Bumps axios from 0.25.0 to 1.6.8.
Release notes:
Bug Fixes