Skip to content

chore: remove graphql from service list#2985

Merged
HarshMN2345 merged 1 commit intomainfrom
chore-remove-graphql-from-service-list
Apr 20, 2026
Merged

chore: remove graphql from service list#2985
HarshMN2345 merged 1 commit intomainfrom
chore-remove-graphql-from-service-list

Conversation

@HarshMN2345
Copy link
Copy Markdown
Member

What does this PR do?

image

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Apr 20, 2026

Greptile Summary

This PR removes the GraphQL entry from the project services list in project-services.ts, so it will no longer appear as a togglable service in the Appwrite Console settings UI. The remaining GraphQL references in the codebase (project-protocols.ts, constants.ts) refer to GraphQL as a protocol (REST/GraphQL/WebSocket), which is a distinct concept and is unaffected.

Confidence Score: 5/5

Safe to merge — straightforward removal with no side effects on other code paths.

Single-file change that removes one list entry; no logic, imports, or other files are impacted. All remaining GraphQL references in the repo are for a different concept (protocol, not service).

No files require special attention.

Important Files Changed

Filename Overview
src/lib/stores/project-services.ts Removes the GraphQL entry from the project services toggle list; no other logic is affected.

Reviews (1): Last reviewed commit: "chore: remove graphql from service list" | Re-trigger Greptile

@HarshMN2345 HarshMN2345 merged commit 145cab7 into main Apr 20, 2026
2 of 4 checks passed
@HarshMN2345 HarshMN2345 deleted the chore-remove-graphql-from-service-list branch April 20, 2026 13:05
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