Skip to content

maintain image order#56

Merged
jgrossi merged 2 commits into
corcel:developfrom
omzy:ACF-50
Feb 9, 2018
Merged

maintain image order#56
jgrossi merged 2 commits into
corcel:developfrom
omzy:ACF-50

Conversation

@omzy

@omzy omzy commented Oct 31, 2017

Copy link
Copy Markdown
Contributor

No description provided.

@omzy

omzy commented Oct 31, 2017

Copy link
Copy Markdown
Contributor Author

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

jgrossi commented Dec 1, 2017

Copy link
Copy Markdown
Member

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

@omzy

omzy commented Jan 28, 2018

Copy link
Copy Markdown
Contributor Author

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

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

jgrossi commented Feb 9, 2018

Copy link
Copy Markdown
Member

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