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

[Request]: Support UUIDs #43

Open
Senexis opened this issue Nov 6, 2024 · 1 comment
Open

[Request]: Support UUIDs #43

Senexis opened this issue Nov 6, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@Senexis
Copy link
Contributor

Senexis commented Nov 6, 2024

As the discussions are disabled there isn't another avenue to post feature requests currently, so apologies for having to go through this channel.

Currently the views are not set up to support UUIDs. When extending the models and tables with the HasUuids trait, everything server-side works fine, however, since the $item->getKey() calls are just dumped into the views, Alpine will crash as it is trying to evaluate the keys as Javascript literals.

I would love to see support for any key type. I suspect but have not yet tried wrapping the $item->getKey() calls with @js() instead of {{ }} in the component would go a long way but, again, since I have not tried yet I'm not sure currently.

@Senexis Senexis added the bug Something isn't working label Nov 6, 2024
@iRaziul
Copy link
Contributor

iRaziul commented Dec 27, 2024

I don't think the current implementation supports UUID so everything works without any issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants