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

Bedrock constraints #1826

Merged
merged 4 commits into from
Jun 26, 2023
Merged

Bedrock constraints #1826

merged 4 commits into from
Jun 26, 2023

Conversation

Tenari
Copy link
Contributor

@Tenari Tenari commented Jun 23, 2023

Ticket: RE-237

allow bedrock users to specify per-path and per-table constraints on various data types

this contains a functional but very limited initial version of constraints, but also includes various fixes to bedrock in general, mostly stuff that broke since the re-naming

@vercel
Copy link

vercel bot commented Jun 23, 2023

@Tenari is attempting to deploy a commit to the Holium Team on Vercel.

To accomplish this, @Tenari needs to request access to the Team.

Afterwards, an owner of the Team is required to accept their membership request.

If you're already a member of the respective Vercel Team, make sure that your Personal Vercel Account is connected to your GitHub account.

@ajlamarc
Copy link
Contributor

General design philosophy is we should be implementing constraints in the node rather than in Gall - to avoid complexity first. (Think of the node like the middleware, API Gateway, etc.). Eventually we should be able to drop the Urbit backend if necessary.

If this is still required, then go for it - if the only "user" of bedrock is the node, then we should aim for the constraints to be specified to the node instead.

@Tenari Tenari merged commit dfff4c2 into master Jun 26, 2023
@Tenari Tenari deleted the bedrock-constraints branch June 26, 2023 15:08
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.

2 participants