fix(console): regional Projects API for service toggles + full ApiSer… by HarshMN2345 · Pull Request #2976 · appwrite/console

HarshMN2345 · 2026-04-13T15:04:06Z

…vice list

What does this PR do?

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.)

…vice list

greptile-apps · 2026-04-13T15:07:37Z

Greptile Summary

This PR migrates service/webhook SDK calls to sdk.forProject(region, projectId) for regional routing, reverts project-services.ts to an explicit list (fixing the GraphQL label regression and adding Sites, TablesDB, and Migrations entries), adds a new Protocols UI card with enable/disable-all, and shows the signing secret on webhook creation.

P1: updateSecurity.svelte line 79 passes the literal string \"Placeholder\" as the description prop of the TLS checkbox — this text will be visible to users in the Settings page.
P2: updateProtocols.svelte uses Submit.ProjectService for all protocol toggle events (lines 59 and 97), conflating protocol and service analytics since no Submit.ProjectProtocol constant exists.

Confidence Score: 4/5

Safe to merge after fixing the "Placeholder" description text in updateSecurity.svelte, which will render as literal UI copy.

One P1 finding (literal "Placeholder" text exposed in UI) blocks a clean merge; the P2 analytics misattribution in updateProtocols.svelte should also be addressed but is not a functional breakage.

src/routes/(console)/project-[region]-[project]/settings/webhooks/[webhook]/updateSecurity.svelte (placeholder description text) and src/routes/(console)/project-[region]-[project]/settings/updateProtocols.svelte (analytics event name).

Important Files Changed

Filename	Overview
src/lib/stores/project-services.ts	Reverted to explicit hand-written list (fixes the GraphQL label regression); adds Sites, TablesDB, and Migrations entries; correctly sorted.
src/routes/(console)/project-[region]-[project]/settings/updateProtocols.svelte	New protocols UI card with enable/disable-all; uses Submit.ProjectService for analytics tracking instead of a protocol-specific event — protocol events will be misattributed to services.
src/routes/(console)/project-[region]-[project]/settings/updateServices.svelte	Migrated to sdk.forProject regional call; enable/disable-all UI with confirmation dialog; correct implementation.
src/routes/(console)/project-[region]-[project]/settings/webhooks/[webhook]/updateSecurity.svelte	Migrated to regional sdk.forProject; contains literal "Placeholder" as the description prop on the TLS Selector.Checkbox, which will surface in production UI.
src/routes/(console)/project-[region]-[project]/settings/webhooks/create/+page.svelte	Adds show-secret modal on webhook creation; correctly reads webhook.secret from the creation response and conditionally shows the modal.

_{Reviews (8): Last reviewed commit: "add enable and disable all to protocol u..." | Re-trigger Greptile}

src/routes/(console)/project-[region]-[project]/+layout.ts

src/lib/stores/project-services.ts

src/routes/(console)/project-[region]-[project]/+layout.ts

Made-with: Cursor

src/lib/stores/project-services.ts

Co-authored-by: Matej Bačo <matejbaco2000@gmail.com>

Meldiron · 2026-04-13T15:23:12Z

src/lib/stores/project-services.ts

+            { label: 'Functions', method: ApiService.Functions, value: null },
+            { label: 'GraphQL', method: ApiService.Graphql, value: null },
+            { label: 'Messaging', method: ApiService.Messaging, value: null }
+        ] satisfies Service[]


We are missing Migrations

Let's also ask AI to sort the list alphabetically

Meldiron · 2026-04-13T15:24:00Z

src/lib/stores/project-services.ts

+                {
+                    label: 'Messaging',
+                    method: ApiService.Messaging,
+                    value: project.serviceStatusForMessaging


Missing migrations I think

Meldiron · 2026-04-13T15:24:21Z

src/routes/(console)/project-[region]-[project]/settings/updateServices.svelte


        try {
-            await sdk.forConsole.projects.updateServiceStatusAll({
+            await sdk.forConsoleIn($project.region).projects.updateServiceStatusAll({


We no longer have updateServiceStatusAll. We need to loop them, and do "updateServiceStatus".

Meldiron · 2026-04-13T15:24:37Z

src/routes/(console)/project-[region]-[project]/settings/updateServices.svelte


        try {
-            await sdk.forConsole.projects.updateServiceStatus({
+            await sdk.forConsoleIn($project.region).projects.updateServiceStatus({


We need new SDK and use project SDK, not projects.

src/lib/stores/project-services.ts

src/routes/(console)/project-[region]-[project]/settings/webhooks/[webhook]/regenerate.svelte

Meldiron · 2026-04-14T12:25:47Z

...utes/(console)/project-[region]-[project]/settings/webhooks/[webhook]/updateSignature.svelte

    <svelte:fragment slot="aside">
        <div>
-            <Secret label="Key" copyEvent="signature" bind:value={$webhook.signatureKey} />
+            <Secret label="Key" copyEvent="signature" bind:value={$webhook.secret} />


You can no longer read secret - you only see it once after creating it, or after updating it

src/lib/stores/project-services.ts

src/routes/(console)/project-[region]-[project]/settings/webhooks/[webhook]/regenerate.svelte

fix(console): regional Projects API for service toggles + full ApiSer…

9cb4209

…vice list

greptile-apps bot reviewed Apr 13, 2026

View reviewed changes

src/routes/(console)/project-[region]-[project]/+layout.ts Outdated Show resolved Hide resolved

src/lib/stores/project-services.ts Outdated Show resolved Hide resolved

src/routes/(console)/project-[region]-[project]/+layout.ts Outdated Show resolved Hide resolved

HarshMN2345 added 3 commits April 13, 2026 20:39

revert(project): restore +layout.ts and updateServices.svelte from main

763c09a

Made-with: Cursor

fix

cb537b5

fix sdk

69ae22a

HarshMN2345 requested a review from Meldiron April 13, 2026 15:16

Meldiron reviewed Apr 13, 2026

View reviewed changes

src/lib/stores/project-services.ts Outdated Show resolved Hide resolved

HarshMN2345 and others added 2 commits April 13, 2026 20:50

Update src/lib/stores/project-services.ts

b98fdd2

Co-authored-by: Matej Bačo <matejbaco2000@gmail.com>

Update project-services.ts

7c26748

Meldiron reviewed Apr 13, 2026

View reviewed changes

fix: esolve merge conflucts

35808ed

HarshMN2345 requested a review from Meldiron April 14, 2026 08:57

Meldiron reviewed Apr 14, 2026

View reviewed changes

src/lib/stores/project-services.ts Outdated Show resolved Hide resolved

Meldiron reviewed Apr 14, 2026

View reviewed changes

src/routes/(console)/project-[region]-[project]/settings/webhooks/[webhook]/regenerate.svelte Outdated Show resolved Hide resolved

Meldiron reviewed Apr 14, 2026

View reviewed changes

src/lib/stores/project-services.ts Outdated Show resolved Hide resolved

HarshMN2345 added 2 commits April 15, 2026 16:38

done some ui changes and addressed comments

56a5901

feat new protocols ui card

f98b287

greptile-apps bot reviewed Apr 15, 2026

View reviewed changes

src/routes/(console)/project-[region]-[project]/settings/webhooks/[webhook]/regenerate.svelte Show resolved Hide resolved

HarshMN2345 added 2 commits April 15, 2026 17:06

adding show key flow to create webhook

5885084

add enable and disable all to protocol ui for consistency

bd6a99f

HarshMN2345 self-assigned this Apr 15, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(console): regional Projects API for service toggles + full ApiSer…#2976

fix(console): regional Projects API for service toggles + full ApiSer…#2976
HarshMN2345 wants to merge 11 commits intomainfrom
fix-project-services-region-and-sdk-list

HarshMN2345 commented Apr 13, 2026 •

edited

Loading

Uh oh!

greptile-apps bot commented Apr 13, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Meldiron Apr 13, 2026

Uh oh!

Meldiron Apr 13, 2026

Uh oh!

Meldiron Apr 13, 2026

Uh oh!

Meldiron Apr 13, 2026

Uh oh!

Uh oh!

Uh oh!

Meldiron Apr 14, 2026

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

HarshMN2345 commented Apr 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What does this PR do?

Test Plan

Related PRs and Issues

Have you read the Contributing Guidelines on issues?

Uh oh!

greptile-apps bot commented Apr 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Greptile Summary

Confidence Score: 4/5

Important Files Changed

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Meldiron Apr 13, 2026

Choose a reason for hiding this comment

Uh oh!

Meldiron Apr 13, 2026

Choose a reason for hiding this comment

Uh oh!

Meldiron Apr 13, 2026

Choose a reason for hiding this comment

Uh oh!

Meldiron Apr 13, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Meldiron Apr 14, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

HarshMN2345 commented Apr 13, 2026 •

edited

Loading

greptile-apps bot commented Apr 13, 2026 •

edited

Loading