Add trends to sidebar account list #697

Merged
JoshAntBrown merged 1 commits from feature/add-trends-to-sidebar-account-list into main 2024-04-29 21:56:38 +08:00
JoshAntBrown commented 2024-04-29 19:00:21 +08:00 (Migrated from github.com)

/claim #647
Resolves #647

This PR adds filterable series and trend information to the account list in the sidebar. In order to achieve this I have created a new route on the accounts controller that renders the account list in a turbo frame that can then be used to update the frame within the sidebar. The period select simply navigates to that url within the frame and sets the selected period.

As part of this I've also updated the designs to match those in the latest spec.

https://github.com/maybe-finance/maybe/assets/1793797/6b745d1d-0b9b-42f5-a2b7-caaa2ea500c6

/claim #647 Resolves #647 This PR adds filterable series and trend information to the account list in the sidebar. In order to achieve this I have created a new route on the accounts controller that renders the account list in a turbo frame that can then be used to update the frame within the sidebar. The period select simply navigates to that url within the frame and sets the selected period. As part of this I've also updated the designs to match those in the latest spec. https://github.com/maybe-finance/maybe/assets/1793797/6b745d1d-0b9b-42f5-a2b7-caaa2ea500c6
zachgoll (Migrated from github.com) approved these changes 2024-04-29 21:55:42 +08:00
zachgoll (Migrated from github.com) left a comment

Awesome stuff. Fun to see a perfect use case for turbo frames!

Awesome stuff. Fun to see a perfect use case for turbo frames!
Sign in to join this conversation.