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

[v4.3] Use `spree` routing proxy in theme selection partial #5603

Merged
merged 1 commit into from
Jan 18, 2024

Conversation

github-actions[bot]
Copy link

Backport

This will backport the following commits from main to v4.3:

Questions ?

Please refer to the Backport tool documentation

When using the navigation from a gem that isolates its namespace in such
a way that the default route helper is not `spree`, this partial fails
to render with an undefined method error.

This can easily be fixes by prefixing the route helper with Solidus'
routing proxy.

(cherry picked from commit e2b1257)
@github-actions github-actions bot requested a review from a team as a code owner January 18, 2024 12:55
@github-actions github-actions bot added the changelog:solidus_backend Changes to the solidus_backend gem label Jan 18, 2024
@tvdeyen tvdeyen merged commit f01ce97 into v4.3 Jan 18, 2024
9 checks passed
@tvdeyen tvdeyen deleted the backport/v4.3/pr-5599 branch January 18, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:solidus_backend Changes to the solidus_backend gem
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants