Skip to content

Feat: integrate user account management and refine UI iconography with SVG presets - #62

Merged
Circuit-Overtime merged 3 commits into
mainfrom
feat/public-frontend-routes
Aug 15, 2026
Merged

Feat: integrate user account management and refine UI iconography with SVG presets#62
Circuit-Overtime merged 3 commits into
mainfrom
feat/public-frontend-routes

Conversation

@divyanshuj91

Copy link
Copy Markdown
Collaborator

Overview

This Pull Request completes the user account workflow by introducing core profile and authentication modules and upgrading the global iconography system. Emojis throughout key routes have been replaced with modern, scalable SVG presets to align with professional design standards.

Key Implementations

User Account & Authentication Views

  • Dynamic Profile Route (/profile/[username]): Renders contributor-specific metrics including merged pull requests, streak statistics, and specialized contribution badges.
  • Contributor Dashboard (/dashboard): Provides active tracking of claimed issues, repository integrations, and unified switches for managing contest workflows.
  • Financial Wallet Portal (/wallet): Displays payout tallies (earned, unpaid, and pending), linked payment methods (Wise), and past transaction wire ledgers.
  • Settings Dashboard (/settings): Facilitates updates to user metadata (username, email) and OAuth credentials configuration.
  • Gatekeeper Authentication Screens (/login, /register): Simplifies onboarding with simulated single sign-on (SSO) loops.

@elixpoo elixpoo added this to Dev Aug 7, 2026
@elixpoo elixpoo moved this to Todo in Dev Aug 7, 2026
@elixpoo elixpoo added the DEV Category: Dev label Aug 7, 2026
@github-advanced-security

Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@Circuit-Overtime Circuit-Overtime left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Blocking findings:

  1. components/navbar.tsx:38-64 initializes every visitor as logged in and treats a mutable localStorage flag as authentication. app/login/page.tsx:10-16 and app/register/page.tsx:10-16 set that flag without any Elixpo Login/session validation, while the UI labels the action GitHub sign-in. This exposes authenticated navigation and lets anyone impersonate the hardcoded user. Please integrate the actual Elixpo Login/session boundary, or keep these routes explicitly unauthenticated placeholders that cannot grant access.

  2. app/settings/page.tsx:8-16,71-76 hardcodes another person’s email/GitHub identity and reports successful saves/disconnect actions without persistence. app/profile/[username]/page.tsx:7-18 fabricates stats and contribution records for every arbitrary username. These user-facing pages must load authenticated/real data or present neutral, clearly labelled empty states.

  3. The added hierarchy conflicts with the agreed frontend route model (/contests, /contests/[slug]/projects, /contests/[slug]/leaderboard, /u/[handle], /dashboard/rewards) by adding top-level /projects, /leaderboard, /profile/[username], and /wallet. Please align the routes before these links become public contracts.

  4. The branch fails git diff --check in multiple added files. CI also currently fails because .github/workflows/biome.yml invokes a missing ./biome.sh; that repository-level CI defect must be resolved before this can be considered green.

I could not run typecheck/build in this workspace because the runner currently has no Node/npm executable, so those checks also remain unverified.

@Circuit-Overtime
Circuit-Overtime merged commit 69ab879 into main Aug 15, 2026
2 of 3 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in Dev Aug 15, 2026
@elixpoo

elixpoo commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

[CHANGE LOG] 📋 Changelog updated for PR #62. View changelog

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DEV Category: Dev

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants