Skip to content

Add updateProject GraphQL mutation#3438

Merged
williamjallen merged 1 commit intoKitware:masterfrom
williamjallen:update-project-mutation
Feb 5, 2026
Merged

Add updateProject GraphQL mutation#3438
williamjallen merged 1 commit intoKitware:masterfrom
williamjallen:update-project-mutation

Conversation

@williamjallen
Copy link
Collaborator

This commit adds an updateProject mutation which will form the basis for the overhaul of the project settings page. Note that our infrastructure doesn't support multi-part GraphQL requests yet, so I plan to create a separate dedicated route for project logo uploads, outside the GraphQL API.

Copy link
Member

@josephsnyder josephsnyder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Most things look and respond great to the mutation running in GraphQL Explorer. My only question is:

Should we allow name in the mutation? If they had access to the settings page for the project, they wouldn't be able to change the name. It is validating properly for name collisions still so they can't collide.

@williamjallen
Copy link
Collaborator Author

@josephsnyder Based on the feedback we got last year when projects wanted to change names due to the case-insensitivity change with the MySQL->Postgres transition, I figured that it's an important thing to have. I wouldn't be opposed to disallowing name changes if you have a good argument for that though.

The tests for name changes are here.

@josephsnyder
Copy link
Member

if you have a good argument for that though.

I wouldn't say its a good argument, just noticing it's something we don't currently allow. The case-insensitivity makes sense. Thanks @williamjallen

This commit adds an `updateProject` mutation which will form the basis for the overhaul of the project settings page.  Note that our infrastructure doesn't support multi-part GraphQL requests yet, so I plan to create a separate dedicated route for project logo uploads, outside the GraphQL API.
@williamjallen williamjallen force-pushed the update-project-mutation branch from 0075370 to b9174f0 Compare February 5, 2026 16:42
@williamjallen williamjallen added this pull request to the merge queue Feb 5, 2026
Merged via the queue into Kitware:master with commit 5738de2 Feb 5, 2026
7 checks passed
@williamjallen williamjallen deleted the update-project-mutation branch February 5, 2026 18:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants