Skip to content

Gate Gitea connect button to self-hosted, use real Gitea logo#3122

Merged
HarshMN2345 merged 1 commit into
mainfrom
fix/gitea-button-self-hosted-and-icon
Jul 15, 2026
Merged

Gate Gitea connect button to self-hosted, use real Gitea logo#3122
HarshMN2345 merged 1 commit into
mainfrom
fix/gitea-button-self-hosted-and-icon

Conversation

@HarshMN2345

Copy link
Copy Markdown
Member

Summary

Follow-up to #3121 (already merged) -- these two commits landed after that PR merged, so they're a separate PR:

  • Gitea is a self-hosted-only feature, not offered on Appwrite Cloud. The button now also checks isSelfHosted in addition to _APP_VCS_PROVIDERS containing gitea.
  • Swapped the placeholder teacup silhouette icon for Gitea's real mascot logo.

Test plan

  • prettier --check clean
  • Visual check of the new icon against GitHub's icon for size/weight consistency

🤖 Generated with Claude Code

Gitea is a self-hosted-only feature, not offered on Appwrite Cloud, so
the button now checks isSelfHosted in addition to _APP_VCS_PROVIDERS.
Swapped the placeholder teapot silhouette for Gitea's actual mascot
mark.
@appwrite

appwrite Bot commented Jul 15, 2026

Copy link
Copy Markdown

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

Sites (1)
Site Status Logs Preview QR
 console-stage
688b7cf6003b1842c9dc
Processing Processing View Logs Preview URL QR Code

Tip

Each function runs in its own isolated container with custom environment variables

@greptile-apps

greptile-apps Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR gates the Gitea connect button behind an isSelfHosted check (in addition to the existing _APP_VCS_PROVIDERS check) and replaces the placeholder teapot SVG with Gitea's real mascot logo.

  • connectGit.svelte: isGiteaEnabled now short-circuits to false on Appwrite Cloud by ANDing isSelfHosted before checking vcsProviders, which is the correct and minimal change.
  • IconGitea.svelte: The placeholder path is swapped for the real Gitea mascot mark. The new SVG uses viewBox="0 0 24 24" (up from 0 0 20 20) while the rendered width/height stay at 20 — the browser scales it proportionally, so no rendering breakage, though the coordinate space now differs from the old icon.

Confidence Score: 5/5

Safe to merge — both changes are small and self-contained with no side effects outside the git-connect UI.

The isSelfHosted guard is a one-liner using an existing, well-understood boolean constant, and it can only hide the button further (never show it incorrectly). The SVG swap is purely visual with currentcolor fill preserved, so it cannot break functionality.

No files require special attention.

Important Files Changed

Filename Overview
src/lib/components/git/connectGit.svelte Adds isSelfHosted guard to isGiteaEnabled, correctly restricting the Gitea connect button to self-hosted instances; logic is sound and consistent with the existing isVcsEnabled pattern.
src/lib/components/git/IconGitea.svelte Replaces placeholder teapot SVG with the real Gitea mascot path; swaps viewBox="0 0 20 20" for viewBox="0 0 24 24" while keeping width/height="20", which scales correctly but is a minor inconsistency with the previous icon's coordinate space.

Reviews (1): Last reviewed commit: "Gate Gitea button to self-hosted, use re..." | Re-trigger Greptile

@HarshMN2345
HarshMN2345 merged commit c067aa8 into main Jul 15, 2026
2 of 4 checks passed
@HarshMN2345
HarshMN2345 deleted the fix/gitea-button-self-hosted-and-icon branch July 15, 2026 13:21
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