feat: frontend enhancement with page splits, CHANGELOG, and code quality rules#60
Merged
Sagargupta16 merged 4 commits intomainfrom Mar 14, 2026
Merged
Conversation
…GELOG - Add CHANGELOG.md covering full project history (v0.1.0 through v0.8.0) - Split SettingsPage (1733 lines) into 20 focused components under settings/ - Split SubscriptionTrackerPage (1109 lines) into 13 components - Split GoalsPage (1106 lines) into 13 components - Split ComparisonPage (1036 lines) into 13 components - Extract chart color constants (CHART_TEXT, CHART_SURFACE, CHART_INPUT) - Replace raw hex/rgba in chartDefaults and ChartTooltip with constants - Fix relative imports in useAnalyticsV2.ts to use @/ path alias - Update README with version badge, expanded project structure, CHANGELOG link
Remove 12 files that came back from main but conflict with actual project settings or serve no purpose: - .nvmrc (Node 19, project uses 22), .python-version (3.11, CI uses 3.12) - .prettierrc (tabWidth: 3, project uses ESLint not Prettier) - .editorconfig (indent_size: 3, project uses 2-space) - .dockerignore (no Docker setup in repo) - .gitattributes, .github/CODEOWNERS, .github/FUNDING.yml - .github/pull_request_template.md, SECURITY.md - renovate.json (configured at org level) - .maintenance (junk metadata file)
- Bump PyJWT from >=2.9.0 to >=2.12.0 (fixes unknown crit header extension vulnerability, CVE in PyJWT <=2.11.0) - Add pnpm override for undici >=7.24.0 (fixes 6 vulnerabilities: CRLF injection, WebSocket DoS, memory consumption, request smuggling)
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
CHART_TEXT,CHART_SURFACE,CHART_INPUT) to centralize ~129 repeated hex/rgba values across chart componentsTest plan
pnpm run type-check)pnpm run lint)pnpm run build)