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] Updated pathname regex to set the value of the slug passed. Fix required from strapi 4.17 >= #188

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

abhishek1020N
Copy link

updated as per new strapi version's path url

updated as per new strapi version's path url
@xaviermarchal
Copy link

@Baboo7 deeply need this fix, can you review and merge please????

collection-types|single-types|collectionType|singleType

fix to work on old and new strapi versions
@abhishek1020N
Copy link
Author

fix to work on old and new strapi versions

added below in the useSlug.js

image

@lichtteil
Copy link

Works for me!
@Baboo7 Maybe you find the 15 minutes to test and merge this. I tried to use a forked version in one project but it's a lot of work and not future proof.

@t-fritsch
Copy link

t-fritsch commented Aug 12, 2024

@Baboo7 thank you for the great plugin and sorry to bother you but is there anything someone can do to help you merge this fix ?

the code is really simple and without it your plugin simply doesn't work (and it's quite a critical need in admin dashboard)

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.

4 participants