Skip to content

Sorting galery media by position #27593

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

Conversation

LucasCalazans
Copy link
Member

@LucasCalazans LucasCalazans commented Apr 4, 2020

Description

The gallery of configurable products is not showing the video correctly when the video position in the admin is changed.

Fixed Issues

  1. Fixes Product video doesn't play when it is not placed first in the sort order. #26709: Product video doesn't play when it is not placed first in the sort order.

Manual testing scenarios

  1. Create a configurable product with images in the parent product
  2. Add images and a video in the gallery of a child product
  3. Go to storefront, select the options of the child that has a video
  4. Click on the thumb of the video
  5. The video should appear in the gallery with a play button

https://www.loom.com/share/29ef24c26ea04a3d8cd2e03465f071e2

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 are green)

@m2-assistant
Copy link

m2-assistant bot commented Apr 4, 2020

Hi @LucasCalazans. Thank you for your contribution
Here is some useful tips how you can test your changes using Magento test environment.
Add the comment under your pull request to deploy test or vanilla Magento instance:

  • @magento give me test instance - deploy test instance based on PR changes
  • @magento give me 2.4-develop instance - deploy vanilla Magento instance

For more details, please, review the Magento Contributor Guide documentation.

@LucasCalazans
Copy link
Member Author

@magento give me test instance

@magento-engcom-team
Copy link
Contributor

Hi @LucasCalazans. Thank you for your request. I'm working on Magento instance for you

@magento-engcom-team
Copy link
Contributor

Hi @LucasCalazans, here is your new Magento instance.
Admin access: https://pr-27593.instances.magento-community.engineering/admin_9f69
Login: e7bd5e44 Password: 65d2e183d06a
Instance will be terminated in up to 3 hours.

@LucasCalazans
Copy link
Member Author

@magento give me 2.4-develop instance

@magento-engcom-team
Copy link
Contributor

Hi @LucasCalazans. Thank you for your request. I'm working on Magento 2.4-develop instance for you

@magento-engcom-team
Copy link
Contributor

Hi @LucasCalazans, here is your Magento instance.
Admin access: https://i-27593-2-4-develop.instances.magento-community.engineering/admin_4a29
Login: 4660f735 Password: de1157b0f96f
Instance will be terminated in up to 3 hours.

@miguelbalparda miguelbalparda self-assigned this Apr 6, 2020
@ghost ghost added Severity: S3 Affects non-critical data or functionality and does not force users to employ a workaround. Priority: P3 May be fixed according to the position in the backlog. labels May 6, 2020
@engcom-Alfa
Copy link
Contributor

✔️ QA Passed

Manual testing scenario:

  1. Edit the simple product of a configurable product and add a product video;
  2. Save product;
  3. Now, sort the video in the media gallery to make it seen anywhere but not the first position;
  4. Save the product again;
  5. View the configurable product in the storefront product details page.
  6. Click on the color swatch to view the simple product media gallery.
  7. Click the video thumbnail and play the video.

Before:
✖️ Only the thumbnail image of the youtube video is visible.
✖️ The video is not played if the video is not in the first position according to the sort order in admin panel.

2020-09-18_12-33

After:
✔️ The video plays when you click on the thumbnail.

2020-09-18_12-39

@engcom-Alfa engcom-Alfa added the Auto-Tests: Covered All changes in Pull Request is covered by auto-tests label Sep 18, 2020
@engcom-Alfa engcom-Alfa added the QA: Added to Regression Scope Scenario was analysed and added to Regression Testing Scope label Sep 18, 2020
@engcom-Echo engcom-Echo self-assigned this Sep 18, 2020
@engcom-Echo
Copy link
Contributor

@magento run all tests

@engcom-Echo engcom-Echo force-pushed the feature/gallery-position branch from b29bd16 to 4501b8a Compare September 21, 2020 11:00
@engcom-Echo
Copy link
Contributor

@magento run all tests

@engcom-Echo
Copy link
Contributor

engcom-Echo commented Sep 21, 2020

Need approval. Fixed type matching.
function getFormattedPrice() - app/code/Magento/Catalog/Model/Product.php

Copy link
Contributor

@engcom-Alfa engcom-Alfa left a comment

Choose a reason for hiding this comment

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

Hi @LucasCalazans .

Could you resolve merge conflicts?
Thanks!

@ishakhsuvarov
Copy link
Contributor

Hi @LucasCalazans

I am closing this PR for now as it seems like it has become stale and still needs work.

Please feel free to reopen if you wish to continue.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Auto-Tests: Covered All changes in Pull Request is covered by auto-tests Component: Catalog Event: Global-Contribution-Day Partner: Webjump partners-contribution Pull Request is created by Magento Partner Priority: P3 May be fixed according to the position in the backlog. QA: Added to Regression Scope Scenario was analysed and added to Regression Testing Scope Release Line: 2.4 Risk: low Severity: S3 Affects non-critical data or functionality and does not force users to employ a workaround.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Product video doesn't play when it is not placed first in the sort order.
8 participants