Skip to content
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

ATT-1: Enable rotation of image attachments, adding the javascript cod… #28

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Wandji69
Copy link

…e and enabling the java code to call 'rotate instruction
https://issues.openmrs.org/browse/ATT-1

@HerbertYiga
Copy link
Contributor

did you see the failing ci build tests?

@Wandji69
Copy link
Author

Wandji69 commented Apr 29, 2019 via email

@mks-d mks-d self-requested a review October 23, 2019 08:23
@mks-d mks-d changed the title ATT-1:Enable rotation of image attachments, adding the javascript cod… ATT-1: Enable rotation of image attachments, adding the javascript cod… Oct 23, 2019
@mks-d
Copy link
Member

mks-d commented Oct 23, 2019

@Wandji69 this should probably be a draft PR while the ticket is still in design.

This will require to modify the UI as well, so that the thumbnail 'edit mode' shows a rotation icon/button, only for attachments of the image family.

@Wandji69
Copy link
Author

I have implemented the rotation of images and I have issues on the test part please can some help me look into this.

@mks-d mks-d marked this pull request as draft May 27, 2020 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants