-
Notifications
You must be signed in to change notification settings - Fork 5
[PortalDB][Attempt 2] POC PostgREST implementation with API Autogen #467
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
Conversation
fredteumer
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a few comments -- still struggling with the security model here.
I think we're mostly there I just want to wrap my head around the final details.
|
@fredteumer So here's where we at:
PTAL and lmk what you think. I'm still updating some of the docs & scripts.
|
|
@Olshansk Latest changes are 🔥
This should be a sound foundation to start iterating on top of. Approved ✔️ and left a simple typo comment to clean up. |
|
@fredteumer Merged. I'm pretty sure we're going to hit some issues along the way, but will figure it out along the way. Merging this will enable kicking off the effort here: https://github.com/buildwithgrove/infrastructure/pulls?q=sort%3Aupdated-desc+is%3Apr+is%3Aopen |


This is a stripped down version of #452 so it's easier to review & understand.
Reviewing - The key thing to review are the new
.sqlfiles.Testing -
cd ./portal-db; make quickstartand follow the instructions.Demo:
Screen.Recording.2025-10-01.at.9.55.49.PM.mov