Vi/mint mobile app #1
Merged
Conversation
This file contains 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
…rts. Turn off auth on FE for now. Add missing page routes.
…of loading spinner. Fix common color --var-primary. Open Goal Details page on a card click. Add Dashboard and Logout icons. Mock current user. Add routes to create a budget and a goal.
…a tracsaction. Fix Transactions page. Fix Spending Analysis section. Add page layout to Transactions and TransactionDetails pages. Fix displaying of dropdowns.
…investments. Mock Plaid token. Change color of buttons. Add Transactions to navigation menu. Fix infinite loop on Transactions page. Use page layout on Account Details and Transaction Details page. Add styles to Goal Details page. Fix alignment of column values in a few grids.
… a goal including displaying a validation message. Add styles to the Investments and Investment Details page. Display spinner while loading inverstment details. Add titles to summary sections on Dashboard. Start to fix Spending Trends section.
…stment Details pages. Reuse the Table component on the Investments page. Extend the Table component with logic to apply sorting by a column. Fix sorting of Budgets by the Remaining column. Improve UI representation of summary sections on the Dashboard page. Add crypto investments to mock data. Display spinner component instead of a loading text on a few pages.
…estment return value. Hide Transactions menu item from mobile navigation. Fix updating a transaction. Make layout of page content consistent across pages.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Brief description of changes:
Related Issue/Ticket:
Impact Assessment:
Target Environment:
Deployment Considerations:
Changes Made
Technical Changes
Architecture Changes
Database Changes
API Changes
UI/UX Changes
Security Considerations
Infrastructure Changes
Third-party Dependencies
Testing
Unit Tests
Integration Tests
E2E Tests
Performance Testing
Security Testing
Accessibility Testing
Review Checklist
Code Quality
Documentation
Security Review
Performance Review
Mobile Compatibility
Infrastructure
Monitoring/Logging
Rollback Plan
Tech Debt Assessment
Additional Notes
Screenshots/Videos