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

fix(slug): Remove trademark symbols from normalized strings #2447

Merged

Conversation

gdarchen
Copy link
Contributor

When importing a product, we resulted in a slug containing a ® trademark symbol, which is invalid.

Therefore, in this PR, I made sure to remove the trademark symbols from the normalized strings, such as ©, ® and .

@netlify
Copy link

netlify bot commented Oct 12, 2023

Deploy Preview for effervescent-donut-4977b2 canceled.

Name Link
🔨 Latest commit 33a3a49
🔍 Latest deploy log https://app.netlify.com/sites/effervescent-donut-4977b2/deploys/6527e542d1af9100077173e5

@michaelbromley michaelbromley merged commit 9aac191 into vendure-ecommerce:master Oct 12, 2023
13 checks passed
@michaelbromley
Copy link
Member

Thank you!

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