Skip to content

Conversation

@aspiers
Copy link
Collaborator

@aspiers aspiers commented Dec 9, 2025

Without this patch, the collaboration modal in sds-demo uses the old read/write permission model when granting access to collaborators.

This is a problem because commit d2e0b12 updated the grantAccess lexicon to require granular permissions (read, create, update, delete) instead of the write boolean. When users attempt to add collaborators via the UI, the API returns HTTP 400 with error Input/permissions must have the property "create".

This patch solves the problem by updating the collaboration modal to use the granular permission model. Role definitions (viewer, contributor, admin) now specify create, update, and delete permissions instead of a single write boolean. Also removes unused permissions state variable and RepositoryPermissions import that are no longer needed.

Co-authored-by: Composer noreply@cursor.com

Without this patch, the collaboration modal in sds-demo uses the old
read/write permission model when granting access to collaborators.

This is a problem because commit d2e0b12
updated the grantAccess lexicon to require granular permissions (read,
create, update, delete) instead of the write boolean. When users attempt
to add collaborators via the UI, the API returns HTTP 400 with error
"Input/permissions must have the property \"create\"".

This patch solves the problem by updating the collaboration modal to use
the granular permission model. Role definitions (viewer, contributor,
admin) now specify create, update, and delete permissions instead of a
single write boolean. Also removes unused permissions state variable and
RepositoryPermissions import that are no longer needed.

Co-authored-by: Composer <noreply@cursor.com>
@vercel
Copy link

vercel bot commented Dec 9, 2025

Someone is attempting to deploy a commit to the Hypercerts Foundation Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented Dec 9, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
sds-demo Ready Ready Preview Comment Dec 9, 2025 5:11pm

@aspiers aspiers merged commit 7b1c0ba into hypercerts-org:dev Dec 9, 2025
4 of 5 checks passed
@aspiers aspiers deleted the fix-sds-demo branch December 9, 2025 17:13
@aspiers aspiers requested a review from bitbeckers December 9, 2025 17:13
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.

1 participant