Skip to content

feat: add GitHub Sponsors dashboard link (#50) - #464

Merged
lewisgoddard merged 3 commits into
elementary:mainfrom
Pavankalyan32:add-github-sponsors-dashboard-link
Aug 14, 2026
Merged

feat: add GitHub Sponsors dashboard link (#50)#464
lewisgoddard merged 3 commits into
elementary:mainfrom
Pavankalyan32:add-github-sponsors-dashboard-link

Conversation

@Pavankalyan32

Copy link
Copy Markdown
Contributor

Addresses #50 — adds a 'Manage Sponsorship' link/button so sponsors can easily access their GitHub Sponsors dashboard to manage payment methods, tier, etc.

Changes

  • Navbar: Added 'Manage Sponsorship' link in the authenticated user dropdown (next to Log Out)
  • Sponsor page: Added 'Manage Sponsorship' button alongside existing 'Sponsor elementary' and 'Visit elementary.io' buttons
  • Both link to (opens in new tab with )

Testing

  • All CI checks pass locally: ✅, ✅, ✅, ✅
  • Changes follow existing code patterns (Vue 3, Nuxt 3, Pinia, TypeScript)
  • No new dependencies added

Fixes #50

- Add 'Manage Sponsorship' link to navbar for authenticated users
- Add 'Manage Sponsorship' button to sponsor page
- Both link to https://github.com/sponsors/elementary/dashboard

Fixes elementary#50
@danirabbit
danirabbit requested a review from a team August 12, 2026 16:59
@lewisgoddard

Copy link
Copy Markdown
Member

The https://github.com/sponsors/elementary/dashboard link 404s if you are not a sponsor, so maybe we should link to https://github.com/sponsors/elementary instead. This is how it looks:
image
image

@danirabbit Code is fine, thoughts on the link?

Comment thread pages/sponsor.vue
>
♡ Sponsor elementary
</a>
<a

@Pavankalyan32 Pavankalyan32 Aug 13, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

💡 Suggestion (non-blocking) — updated after lewisgoddard's screenshots: this button is visible to all visitors, and the /dashboard URL 404s for anyone who isn't a sponsor, so non-sponsors get a dead end rather than an empty state. Fixed in the latest commit: both links now point to the public https://github.com/sponsors/elementary page (works for everyone; sponsors still reach their manage options from there). Gating the navbar variant behind the existing useAuthStore.loggedIn remains an option if you want a sponsor-only affordance later.

@Pavankalyan32

Copy link
Copy Markdown
Contributor Author

💡 Suggestion (non-blocking): the PR description markdown got mangled — "Both link to (opens in new tab with )" and "All CI checks pass locally: ✅, ✅, ✅, ✅" are missing the actual URLs and check names, so the rendered description doesn't show where the links point or which checks ran. A quick edit to the description body would make it read properly. The code itself is clean — nice, small PR.

@Pavankalyan32

Copy link
Copy Markdown
Contributor Author

Thanks @lewisgoddard — good catch, I can reproduce that. I've pushed a fix (572d9cd): both links now point to the public https://github.com/sponsors/elementary page, which serves everyone and shows manage options for existing sponsors. Appreciate the screenshots.

@lewisgoddard
lewisgoddard merged commit fc39dd2 into elementary:main Aug 14, 2026
4 checks passed
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.

Link to GitHub Sponsors management page

2 participants