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

maintain image order #56

Merged
merged 2 commits into from
Feb 9, 2018
Merged

maintain image order #56

merged 2 commits into from
Feb 9, 2018

Conversation

omzy
Copy link
Contributor

@omzy omzy commented Oct 31, 2017

No description provided.

@omzy
Copy link
Contributor Author

omzy commented Oct 31, 2017

The test failed, I think because now that the Gallery will maintain the order, the 6th image will be a different one to the one the test is looking for.

@omzy omzy mentioned this pull request Nov 26, 2017
@jgrossi
Copy link
Member

jgrossi commented Dec 1, 2017

@omzy83 then can you update the tests? can't us change the order just to make the test passing?

@omzy
Copy link
Contributor Author

omzy commented Jan 28, 2018

@jgrossi test updated and passes successfully, please can you check and merge?

@jgrossi jgrossi merged commit 981069b into corcel:develop Feb 9, 2018
@jgrossi
Copy link
Member

jgrossi commented Feb 9, 2018

sorry about being late 😎thanks for the PR!

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.

2 participants