-
Notifications
You must be signed in to change notification settings - Fork 4
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
[nuxt] add nuxt module #204
base: development
Are you sure you want to change the base?
Conversation
aaef2ac
to
886557b
Compare
886557b
to
76d1056
Compare
This likely requires resolving #200 first |
If you're looking for a shortcut for User schema, this is what's minimally required for a PG-backed JWT auth workflow And here's a schema for authentication and authorization in Postgres |
b47cb3b
to
a9a7d45
Compare
I can see the Users UI briefly before it shows an error in the browser: "500 crypto.randomUUID is not a function" and a differetn one in the console:
|
This showed up in the FAB app before as well when using the ATable package, and I had helped Heather through the error here. If we assume secure contexts across the board, then I think this is a non-issue, but if we want to ease development as well in potentially insecure contexts, then I think we should revert to using the
These errors/warnings are coming from trying to render the Stonecrop instance object as is into the page ( |
@Alchez Turns out this is a known WSL problem. I was trying to access it via the WSL IP which the browser has every reason to think is an insecure website. It also only works on Brave and not Firefox. That said, I think the limitations of using the browser for this are clear. Before refactoring the UUID package I think it might make sense to look at alternatives that are more performant. |
@agritheory It seems like if we're looking at performance, then we might want to consider The differences in performance also only seem to show up when you scale up the number of IDs being generated, which is currently low since they're only being used for table stores. Should I move to |
I think nanoid is still overbuilt for our purposes and that a randomly generated 8-character base64 hash would be fine. |
Coverage Report for ./atable
File Coverage
|
@agritheory I've added the |
Yeah, I think that's what's most helpful, but having the components available wouldn't hurt either |
Closes #199.