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 infinite updates on empty slug #14

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Serhii-the-Dev
Copy link

resolves #11

@LaakJoonas
Copy link

LaakJoonas commented Sep 6, 2023

Thank you man! You saved my day. This extension is useful but crashes the UI when I erase and modify the slug enough...

EDIT: Didn't work :(

@Serhii-the-Dev
Copy link
Author

Hi @LaakJoonas
Could you please describe your case so I may try to reproduce? This PR should fix an issue when all slug sources are emptied manually, probably there are some extra steps in your case.

@LaakJoonas
Copy link

I figured it out. Your fix is good. My setup had another similar issue I fixed.

@albanallee
Copy link

Could you merge this pull request ?

@romanlex
Copy link

its alive?

@Serhii-the-Dev
Copy link
Author

@albanallee @romanlex the PR is alive, alas the repository itself doesn't seems so, and I'm not able nor to merge it without the owner's approval nor publish on npm. I ended up with copy-pasting this code into my Directus projects (anyway I usually have a plenty of custom extensions).

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.

Page hangs when the input is empty
4 participants