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

Fix image orientation in SALSAH 1 (moved from #768) #817

Merged
merged 2 commits into from
Apr 10, 2018

Conversation

benjamingeer
Copy link

@benjamingeer benjamingeer commented Apr 10, 2018

This is PR #768, adapted for the current develop.

The Knora-Sipi integration tests pass.

@loicjaouen Can you check that this does what you want? If so, I think we can merge it.

fixes: #726

@benjamingeer benjamingeer requested a review from subotic April 10, 2018 13:25
@benjamingeer benjamingeer added this to the v1.4.0 milestone Apr 10, 2018
@benjamingeer benjamingeer added the bug something isn't working label Apr 10, 2018
@loicjaouen
Copy link
Contributor

Thanks for caring @benjamingeer , yes, that's the code change that I wanted, if the tests pass, let's move forward.

@benjamingeer
Copy link
Author

You're welcome! It's always great to get bug fixes. :) Could you just update the release notes? I forgot to do that.

Copy link
Collaborator

@subotic subotic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks :-)

@benjamingeer benjamingeer merged commit 831a8b2 into develop Apr 10, 2018
@benjamingeer benjamingeer deleted the fix/img-orientation-2 branch April 10, 2018 15:06
@loicjaouen
Copy link
Contributor

Could you just update the release notes?

oups, sorry @benjamingeer , I am catching up too late

@benjamingeer
Copy link
Author

No worries. :)

@benjamingeer benjamingeer mentioned this pull request Jul 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Salsah 1.5 image viewer doesn't respect image orientation
3 participants