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

Convert ProductLink tab into web component #110

Merged
merged 2 commits into from
May 23, 2024
Merged

Convert ProductLink tab into web component #110

merged 2 commits into from
May 23, 2024

Conversation

tvdeyen
Copy link
Member

@tvdeyen tvdeyen commented May 18, 2024

Alchemy 7.2 converted the link dialog into a view component.
The current view partials and javascript does not work anymore.

Adding a alchemy-product-select web component.

@tvdeyen tvdeyen self-assigned this May 18, 2024
@tvdeyen tvdeyen requested a review from sascha-karnatz May 20, 2024 07:55
@tvdeyen tvdeyen force-pushed the fix-product-link branch 2 times, most recently from 588d0b9 to f6868c8 Compare May 22, 2024 17:40
@tvdeyen tvdeyen marked this pull request as ready for review May 22, 2024 17:40
@tvdeyen tvdeyen requested review from a team and removed request for sascha-karnatz May 22, 2024 17:40
@tvdeyen tvdeyen removed their assignment May 22, 2024
@mamhoff
Copy link
Contributor

mamhoff commented May 22, 2024

This looks all good to me, but the tests are very broken...

@tvdeyen
Copy link
Member Author

tvdeyen commented May 22, 2024

Yes, but totally unrelated.

@tvdeyen tvdeyen force-pushed the fix-product-link branch from f6868c8 to 3cf2ef8 Compare May 22, 2024 19:58
@tvdeyen tvdeyen force-pushed the fix-product-link branch from 3cf2ef8 to 8b756fc Compare May 23, 2024 07:55
Gemfile Outdated Show resolved Hide resolved
tvdeyen added 2 commits May 23, 2024 13:15
Alchemy 7.2 converted the link dialog into a view component.
The current view partials and javascript does not work anymore.

Adding a alchemy-product-select web component.
We need current main branch of Alchemy because
the Link Dialog has a breaking change and do not
support older Alchemy versions anymore.
@tvdeyen tvdeyen force-pushed the fix-product-link branch from 8b756fc to 5cb353d Compare May 23, 2024 11:15
@tvdeyen tvdeyen merged commit 12e4444 into main May 23, 2024
32 checks passed
@tvdeyen tvdeyen deleted the fix-product-link branch May 23, 2024 15:46
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