-
Notifications
You must be signed in to change notification settings - Fork 35
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
B-20438 and B-20440 #13572
B-20438 and B-20440 #13572
Conversation
-this is to include the new react-file-viewer v1.2.4 (made for milmove app) -all tests regarding FileViewer needed to be modified to workaround some dependency issues -updated Swagger, Yarn -with this commit, milmove Office users will be able to zoom on image uploads associated with documents. They will also be able to rotate images and save the image rotation so all users (including other users) will be able to see the updated rotation.
Bundle StatsHey there, this message comes from a GitHub action that helps you and reviewers to understand how these changes affect the size of this project's bundle. As this PR is updated, I'll keep you updated on how the bundle size is impacted. Total
Changeset
View detailed bundle breakdownAdded
Removed
Bigger No assets were bigger Smaller
Unchanged
|
At the end there had to do a make clean and re-generate the swagger files |
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.
LGTM
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.
LGTM!
Agility Tickets
B-20440
B-20438
Previous PRs (into INT) Associated with this PR
Verification that it all works
If you would like to verify that rotation/zoom still works on this branch, make sure you perform the following steps to ensure it will work as designed.