Skip to content

Add webhook migration wizard support#3045

Open
premtsd-code wants to merge 1 commit into
add-project-variable-migrationfrom
add-webhook-migration
Open

Add webhook migration wizard support#3045
premtsd-code wants to merge 1 commit into
add-project-variable-migrationfrom
add-webhook-migration

Conversation

@premtsd-code
Copy link
Copy Markdown
Contributor

Summary

Stacks on #3044. Adds webhook as its own top-level group in the migration wizard, parallel to platforms / api-keys / project-variables.

The description text warns operators that signing secrets don't survive the migration (SDK constraint — see utopia-php/migration#185).

Cross-repo

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 14, 2026

Greptile Summary

This PR adds webhook support to the migration wizard as a child option under the existing "Settings" group (alongside project variables), following the same parent/child pattern already used for integrations (platforms + api-keys). The signing-secret caveat is correctly surfaced via descriptionMap.settings.webhooks and renders as the description on the child checkbox inside the settings Accordion.

  • migration.ts adds 'webhook' to the resource union, MigrationResources, ResourcesFriendly, and providerResources.appwrite, and keeps both migrationFormToResources and resourcesToMigrationForm symmetric.
  • resource-form.svelte extends shouldRenderGroup('settings') to show the group when either ProjectVariable or Webhook is available.
  • importReport.svelte adds the webhook label and signing-secret description to the settings labelMap/descriptionMap; the description reaches users as the description prop on the child Selector.Checkbox inside the Accordion.

Confidence Score: 5/5

Safe to merge; changes are additive and follow established patterns throughout the migration wizard.

All three files make narrowly scoped, additive changes that mirror the existing api-key/platform nesting pattern. The form↔resource conversion helpers are symmetric, the provider guard correctly limits webhooks to the appwrite provider, and the signing-secret warning reaches users as the child-checkbox description inside the settings Accordion.

No files require special attention.

Important Files Changed

Filename Overview
src/lib/stores/migration.ts Adds 'webhook' to the MigrationResource union, MigrationResources constant, ResourcesFriendly map, providerResources.appwrite, and both form↔resource conversion helpers. Logic is symmetric and consistent with the existing api-key/platform pattern.
src/routes/(console)/(migration-wizard)/resource-form.svelte Extends shouldRenderGroup('settings') to include the Webhook resource check, correctly following the same OR pattern used for integrations (platform
src/routes/(console)/project-[region]-[project]/settings/migrations/(import)/importReport.svelte Adds webhooks label and description to the settings group; the signing-secret caveat in descriptionMap.settings.webhooks is correctly surfaced as the child-checkbox description prop inside the Accordion branch.

Reviews (2): Last reviewed commit: "Add webhook migration wizard support" | Re-trigger Greptile

@premtsd-code premtsd-code force-pushed the add-project-variable-migration branch from 4477797 to e53bbac Compare May 14, 2026 19:15
Adds webhook as a child toggle under the existing settings group,
joining project variables as the second project-level setting that
can be migrated.
@premtsd-code premtsd-code force-pushed the add-webhook-migration branch from f452174 to 09c4fc0 Compare May 14, 2026 19:34
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