Skip to content

Lychee/fix #1589

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

Merged
merged 6 commits into from
Dec 3, 2024
Merged

Lychee/fix #1589

merged 6 commits into from
Dec 3, 2024

Conversation

Isengo1989
Copy link
Contributor

No description provided.

@Isengo1989 Isengo1989 merged commit 5e7fdf0 into main Dec 3, 2024
8 checks passed
@Isengo1989 Isengo1989 deleted the lychee/fix branch December 3, 2024 14:42
@@ -9,11 +9,11 @@ nav:

## Overview

In this guide, you will see an example of how you can extend a Shopware migration profile of the [Shopware Migration Assistant](https://store.shopware.com/search?sSearch=Swag257162657297f). For this example, the Shopware 5 [SwagAdvDevBundle](https://github.com/shopwareLabs/SwagAdvDevBundle) plugin is migrated to the Shopware 6. For simplicity, only the local gateway is implemented.
Copy link
Member

Choose a reason for hiding this comment

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

@Isengo1989 now the link only goes to the store, which is not helpful, if you want to have the migration assistant 🤔 I guess, you can also link to GitHub?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@mitelg sry, missed that comment. So the link beforehand was also pointing to the store (but the old structure of search via SW5, I just adjusted the path to work with SW6). Feel free to add the Repo here if it makes more sense, sadly I was not involved yet in the migration assistant and can not make that decision.

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