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

Product on Wishlist does not link to its variant when using URL Dispatcher #2987

Closed
3 tasks done
revlis-x opened this issue May 29, 2019 · 0 comments
Closed
3 tasks done
Labels
1: Easy bug Bug reports P3: Normal Priority mark - normal priority QA rejected Testers will add this label when something is still wrong
Milestone

Comments

@revlis-x
Copy link

Current behavior

With URL Dispatcher on, when saving a specific product variant to the wishlist, link is not generated correctly (e.g. with ?ChildSku).

Expected behavior

As before without URL Dispatcher - the wishlist product should link to its correct variant.

Steps to reproduce the issue

  1. Add non-default product-variation to wishlist
  2. Open wishlist and click product, which only links to the Configurable product without ChildSku

Can you handle fixing this bug by yourself?

Maybe, but not right now.

Which Release Cycle state this refers to? Info for developer.

Pick one option.

  • This is a bug report for test version on https://test.storefrontcloud.io - In this case Developer should create branch from develop branch and create Pull Request 2. Feature / Improvement back to develop.
  • This is a bug report for current Release Candidate version on https://next.storefrontcloud.io - In this case Developer should create branch from release branch and create Pull Request 3. Stabilisation fix back to release.
  • This is a bug report for current Stable version on https://demo.storefrontcloud.io and should be placed in next stable version hotfix - In this case Developer should create branch from hotfix or master branch and create Pull Request 4. Hotfix back to hotfix.
@revlis-x revlis-x added the bug Bug reports label May 29, 2019
@pkarw pkarw added 1: Easy P3: Normal Priority mark - normal priority labels May 29, 2019
@michasik michasik mentioned this issue Jun 10, 2019
8 tasks
@patzick patzick added this to the 1.10.0-rc.1 milestone Jun 10, 2019
@pkarw pkarw modified the milestones: 1.10.0-rc.1, 1.11.0-rc.1 Jun 11, 2019
pkarw added a commit that referenced this issue Jun 24, 2019
@pkarw pkarw added the QA - Ready for tests This is notification for testers, that improvement is ready to be tested and verified. label Jun 24, 2019
@pkarw pkarw closed this as completed Jun 24, 2019
@alinadivante alinadivante added QA approved after merge Testers will add this label after positive check on merged changes and removed QA - Ready for tests This is notification for testers, that improvement is ready to be tested and verified. labels Jun 25, 2019
@alinadivante alinadivante added QA rejected Testers will add this label when something is still wrong and removed QA approved after merge Testers will add this label after positive check on merged changes labels Jul 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1: Easy bug Bug reports P3: Normal Priority mark - normal priority QA rejected Testers will add this label when something is still wrong
Projects
None yet
Development

No branches or pull requests

4 participants