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
transactions: refactor #5369
transactions: refactor #5369
Conversation
Okay had to bump prettier version in this branch to play nicely with zustand. Forced a write which is why file amount is large. |
This reverts commit 0ead5a0.
// @ts-expect-error ts-migrate(7031) FIXME: Binding element 'title' implicitly has an 'any' ty... Remove this comment to see the full error message | ||
({ title }) => title === 'Pending' | ||
); | ||
const pendingSectionIndex = sectionedTransactions.findIndex(({ title }) => title === 'Pending'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@skylarbarrera not sure what the correct logic is here but seems like it should match https://github.com/rainbow-me/rainbow/pull/5369/files#diff-62d641fbc0fd2cd84a2d7e0b31ac36fcaa66ad4c5601819ca4a18546ff1015d2R98
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All issues in APP-1177 have been resolved ✅
Fixes APP-800
What changed (plus any additional context for devs)
Screen recordings / screenshots
What to test