Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Transaction section #121

Merged
merged 1 commit into from
Nov 29, 2023
Merged

Transaction section #121

merged 1 commit into from
Nov 29, 2023

Conversation

0xbaggi
Copy link
Contributor

@0xbaggi 0xbaggi commented Nov 18, 2023

I've implemented and fixed the 'Transactions' page, particularly the 'Categories' and 'Accounts' tabs.

Categories:
- Adjusted the color scheme for both the list and the graph (fixed in #103 '[Categories Tab] Graph should take colors by category').
- Fixed the list (space between items and borderRadius).

Accounts:
- Implemented the page according to the Figma design.
- Enabled backend graph functionality with accounts.
- Enabled backend list functionality with accounts and transactions.

TODO:
- (As in the categories tab) 'Income' and 'Expenses' filters do not change the view."

demo.mp4

I've implemented and fixed the 'Transactions' page, particularly the 'Categories' and 'Accounts' tabs.

Categories:
	- Adjusted the color scheme for both the list and the graph (fixed in RIP-Comm#103 '[Categories Tab] Graph should take colors by category').
	- Fixed the list (space between items and borderRadius).
Accounts:
	- Implemented the page according to the Figma design.
	- Enabled backend graph functionality with accounts.
	- Enabled backend list functionality with accounts and transactions.
TODO:
	- (As in the categories tab) 'Income' and 'Expenses' filters do not change the view."
@mikev-cw
Copy link
Collaborator

mikev-cw commented Nov 29, 2023

Very well done! I'm merging this, thank you!!!

PS: Failing checks are known issues, we'll fix that

@mikev-cw mikev-cw merged commit 3653b6c into RIP-Comm:main Nov 29, 2023
1 check failed
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.

2 participants