Skip to content

Point the About tab Discord link at the no-expiry invite - #34

Merged
jatinkrmalik merged 1 commit into
mainfrom
jatin/about-discord-canonical
Aug 29, 2026
Merged

Point the About tab Discord link at the no-expiry invite#34
jatinkrmalik merged 1 commit into
mainfrom
jatin/about-discord-canonical

Conversation

@jatinkrmalik

Copy link
Copy Markdown
Member

The WebUI About tab still sent people to https://discord.gg/UMJduhcqn. That invite is stale. README and vocagateway.vocahq.com already use https://discord.gg/t6muquAJbm, Jatin's no-expiry invite.

This only changes the Discord tuple in app/fragments/about.py and the About tests that asserted the old URL. Family copy, gateway-mode status, and the other social links stay as they are. README and web/index.html already have the canonical invite so they are not in this PR.

@netlify

netlify Bot commented Aug 29, 2026

Copy link
Copy Markdown

Deploy Preview for voca-gateway ready!

Name Link
🔨 Latest commit a790769
🔍 Latest deploy log https://app.netlify.com/projects/voca-gateway/deploys/6a93623ae33f9100080d93e3
😎 Deploy Preview https://deploy-preview-34--voca-gateway.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@jatinkrmalik
jatinkrmalik marked this pull request as ready for review August 29, 2026 22:44
@greptile-apps

greptile-apps Bot commented Aug 29, 2026

Copy link
Copy Markdown

Greptile Summary

The PR updates the About tab to use the repository’s canonical no-expiry Discord invite and adjusts the corresponding assertions.

  • Replaces the stale Discord URL in the About fragment.
  • Updates fragment-level and admin-surface tests to expect the canonical invite.

Confidence Score: 5/5

The PR appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
app/fragments/about.py Replaces the Discord contact tuple’s stale invite with the canonical URL used elsewhere in the repository.
tests/test_about.py Adds a direct assertion that the rendered About fragment contains the canonical Discord invite.
tests/test_admin.py Updates the shared About-surface assertion to match the canonical Discord invite.

Reviews (4): Last reviewed commit: "Point the About tab Discord link at the ..." | Re-trigger Greptile

greptile-apps[bot]
greptile-apps Bot previously approved these changes Aug 29, 2026
@jatinkrmalik
jatinkrmalik marked this pull request as draft August 29, 2026 22:47
@jatinkrmalik
jatinkrmalik marked this pull request as ready for review August 29, 2026 22:48
The About social tuple still used https://discord.gg/UMJduhcqn.
README and the public site already use https://discord.gg/t6muquAJbm.
Swap the tuple and lock that URL in the About tests. Leave the rest
of the About copy alone, including gateway mode.
@jatinkrmalik
jatinkrmalik force-pushed the jatin/about-discord-canonical branch from 4676c90 to a790769 Compare August 29, 2026 22:50
@greptile-apps
greptile-apps Bot dismissed their stale review August 29, 2026 22:50

Dismissed because a newer commit was pushed; Greptile will re-review the current head.

@jatinkrmalik
jatinkrmalik merged commit 578579d into main Aug 29, 2026
6 checks passed
@Mr-Sunglasses
Mr-Sunglasses deleted the jatin/about-discord-canonical branch September 3, 2026 10:03
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