Skip to content

fix: add docs link to build triggers card for functions and sites#3069

Merged
HarshMN2345 merged 1 commit into
mainfrom
fix-build-triggers-docs-link
May 29, 2026
Merged

fix: add docs link to build triggers card for functions and sites#3069
HarshMN2345 merged 1 commit into
mainfrom
fix-build-triggers-docs-link

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

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 29, 2026

Greptile Summary

Adds a "Learn more" external link to the Build Triggers settings card for both Functions and Sites, pointing to the respective Appwrite documentation pages. The description text is also wrapped in Typography.Text to align with the design system's text rendering.

  • updateBuildTriggers.svelte (functions): imports Typography and Link, wraps the card description in <Typography.Text>, and appends a link to appwrite.io/docs/products/functions/deploy-from-git#build-triggers.
  • updateBuildTriggers.svelte (sites): identical change pointing to appwrite.io/docs/products/sites/deploy-from-git#build-triggers.

Confidence Score: 5/5

Safe to merge — the change is additive and limited to UI copy and a docs link in two settings components.

Both files receive an identical, isolated addition: a Typography.Text wrapper and an external Link to the Appwrite documentation. The URLs are product-specific and correctly differentiated between functions and sites. No logic, state management, or API calls are affected.

No files require special attention.

Important Files Changed

Filename Overview
src/routes/(console)/project-[region]-[project]/functions/function-[function]/settings/updateBuildTriggers.svelte Adds a 'Learn more' link to the Build Triggers card pointing to the functions deploy-from-git docs, wrapping the description text in Typography.Text for consistent styling.
src/routes/(console)/project-[region]-[project]/sites/site-[site]/settings/updateBuildTriggers.svelte Adds a 'Learn more' link to the Build Triggers card pointing to the sites deploy-from-git docs, mirroring the same pattern as the functions variant.

Reviews (1): Last reviewed commit: "fix: add docs link to build triggers car..." | Re-trigger Greptile

Copy link
Copy Markdown
Contributor

@adityaoberai adityaoberai left a comment

Choose a reason for hiding this comment

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

Approving to unblock

@HarshMN2345 HarshMN2345 merged commit 2ebf49f into main May 29, 2026
4 checks passed
@HarshMN2345 HarshMN2345 deleted the fix-build-triggers-docs-link branch May 29, 2026 13:59
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