feat: add back to dashboard link on settings page#1016
Conversation
|
@TESTPERSONAL is attempting to deploy a commit to the PRIYANSHU DOSHI's projects Team on Vercel. A member of the Team first needs to authorize it. |
github-actions
Bot
added
gssoc26
GSSoC Label Checklist 🏷️@Priyanshu-byte-coder — please apply the appropriate labels before merging: Difficulty (pick one):
Quality (optional):
Validation (required to score):
|
The DashboardHeader renders the h1 as 'DASHBOARD' (all-caps via
design), but the Playwright selectors used an exact-case match
{ name: "Dashboard" } which never matched.
Changed all three heading assertions to /dashboard/i regex so they
match the actual DOM output and unblock every failing E2E test.
landing.spec.js: - Remove assertion for 'DevTrack' h1 (hero reads 'YOUR CODE HAS A PULSE') - Add .first() to 'Sign in with GitHub' link selectors (2 links on page, strict mode violation) - Replace 'View on GitHub' link with '★ Star on GitHub' (actual text) auth-bypass.spec.js: - Add .first() to 'Sign in with GitHub' link (strict mode violation) - Change heading assertions from exact 'Dashboard' to /dashboard/i regex dashboard-widgets.spec.js: - Add mock for /api/goals/sync so GoalTracker doesn't hang waiting for a real Supabase call (Goals heading was never rendering in CI) - Add mocks for additional API routes hit on dashboard load
|
Hi! This PR now has a merge conflict with main, likely on the shared e2e test files ( git fetch origin
git rebase origin/main
# resolve any conflicts
git push --force-with-leaseYour actual feature/fix changes are valuable — we'd love to get this merged once rebased! |
# Conflicts: # e2e/landing.spec.js # src/app/dashboard/settings/page.tsx
|
Just wanted to update you that this Pull Request has been fully rebased against the latest It is fully ready for your final review and merging. Thank you so much for your time and hard work in organizing GSSoC 2026! |
2 participants
Closes #970.
Summary of What Has Been Done:
Added a navigation back link to the dashboard from the Settings page.
Changes Made:
src/app/dashboard/settings/page.tsxto include an accessible, styled Next.js<Link>back to/dashboardfeaturing micro-interaction animations.