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

Backport placeholder image #15890

Closed
wants to merge 16 commits into from
Closed

Backport placeholder image #15890

wants to merge 16 commits into from

Conversation

chirag-wagento
Copy link
Contributor

Description

Fixed Issues (if relevant)

  1. Product placeholder image resize issue #13636: Product placeholder image resize issue

Manual testing scenarios

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

chirag-wagento and others added 16 commits June 4, 2018 10:34
[Backport] Product placeholder image resize issue #13636
[Backport] Removed unused class from forms less file.
[Backport] [BUGFIX] Added row_id to the flat action indexer so the value isn't s…
[Backport] For MSI-377: fix test fixture and mark return processor as @api
…code of undefined Dynamic data rows were failing due to a read after delete condition Rows were removed just before the information update. An undefined javascript error was triggered.
[Backport] Fix for issue 911 found on MSI project - Cannot read property source_…
@chirag-wagento
Copy link
Contributor Author

@VladimirZaets Could you please check what cause failed travis?

Thanks,

@VladimirZaets
Copy link
Contributor

VladimirZaets commented Jun 12, 2018

Hi, @chirag-wagento, please fix failed unit and integration tests. Thanks

@orlangur
Copy link
Contributor

Commit history is incorrect. Please check it prior to PR creation.

@orlangur orlangur closed this Jun 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants