Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

FOC-11-fix-cropperjs-usage-for-image-cropping #20

Merged
merged 11 commits into from
Feb 6, 2022

Conversation

cjativa
Copy link
Contributor

@cjativa cjativa commented Feb 6, 2022

This pull request does the following

  • Upgrades react to a non-breaking version so we can use hooks. Mainly useRef
  • Replaces our manual usage of cropperjs with react-cropperjs so that we can more easily use cropperjs in a more react-friendly way
  • Consolidates a couple files for redundancy
  • Fixes the issue where photos captured on a mobile device are oriented horizontally (see this issue)

This latest version is deployed on our Lambda function for the app.

@cjativa cjativa self-assigned this Feb 6, 2022
@cjativa cjativa merged commit d4eb809 into master Feb 6, 2022
@cjativa cjativa deleted the FOC-11-fix-cropperjs-usage-for-image-cropping branch February 6, 2022 00:43
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant