Add schemaCheck:approve to the organization access token ui#7773
Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request addresses a functional gap by introducing the Highlights
Changelog
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request correctly adds the schemaCheck:approve permission to the available permissions for organization access tokens. My review includes a comment about adding an integration test to cover this new functionality, as per the repository's style guide.
| { | ||
| id: 'schemaCheck:approve', | ||
| title: 'Approve schema check', | ||
| description: 'Grant access to approve failed schema checks.', | ||
| dependsOn: 'project:describe', | ||
| }, |
There was a problem hiding this comment.
While this correctly adds the schemaCheck:approve permission, this new functionality should be covered by an integration test. Please add a test to verify that a token with this permission can approve a failed schema check, and a token without it cannot. This is required by the repository's style guide for new functionality.
References
- Adding new functionality to the GraphQL API or another service should come with a set of integration tests within
/integration-testsfor testing that new component/functionality. (link)
🚀 Snapshot Release (
|
| Package | Version | Info |
|---|---|---|
@graphql-hive/apollo |
0.48.0-alpha-20260305100508-68481c1a0f538a007d2c67c59749e22c58fac4a8 |
npm ↗︎ unpkg ↗︎ |
@graphql-hive/cli |
0.58.4-alpha-20260305100508-68481c1a0f538a007d2c67c59749e22c58fac4a8 |
npm ↗︎ unpkg ↗︎ |
@graphql-hive/core |
0.21.0-alpha-20260305100508-68481c1a0f538a007d2c67c59749e22c58fac4a8 |
npm ↗︎ unpkg ↗︎ |
@graphql-hive/envelop |
0.40.5-alpha-20260305100508-68481c1a0f538a007d2c67c59749e22c58fac4a8 |
npm ↗︎ unpkg ↗︎ |
@graphql-hive/yoga |
0.48.0-alpha-20260305100508-68481c1a0f538a007d2c67c59749e22c58fac4a8 |
npm ↗︎ unpkg ↗︎ |
hive |
9.6.0-alpha-20260305100508-68481c1a0f538a007d2c67c59749e22c58fac4a8 |
npm ↗︎ unpkg ↗︎ |
| id: 'schemaCheck:approve', | ||
| title: 'Approve schema check', | ||
| description: 'Grant access to approve failed schema checks.', | ||
| dependsOn: 'project:describe', |
There was a problem hiding this comment.
I'd like feedback on this--
I added the dependsOn to match the organization member permission UI, but admittedly I don't know why we have this dependsOn.
I assume because the resolvers make calls that require project:describe, but I'd be open to removing this restriction and adjusting the resolvers to avoid having permissions depend on other permissions.
There was a problem hiding this comment.
The depends on is mostly for user permissions, so you do not assign manage project without project describe. We could drop the dependency here.
|
🐋 This PR was built and pushed to the following Docker images: Targets: Platforms: Image Tag: |
💻 Website PreviewThe latest changes are available as preview in: https://pr-7773.hive-landing-page.pages.dev |
|
Can access tokens approve schema changes today? Also Okay..., nevermind, just realized we support approving failed schema checks via the CLI since #7193. 🤦 Okay, in that case this makes sense! |
Co-authored-by: Laurin <laurinquast@googlemail.com>
Background
Personal access tokens grant permission to
schemaCheck:approve, but this option is not available to organization access tokens.This permission is used on schema checks with the
--forceSafeargument.Description
Adds the permission.