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.

Sign up for GitHub

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

Jump to bottom

(feature) New tax report #816

Merged
merged 117 commits into from
Jul 15, 2024
Merged

(feature) New tax report #816

merged 117 commits into from
Jul 15, 2024

Conversation

alexstotsky
Copy link
Contributor

@alexstotsky alexstotsky commented May 16, 2024 •
edited
Loading

Tasks:
https://app.asana.com/0/1163495710802945/1207173834368490/f
https://app.asana.com/0/1163495710802945/1206668729891587/f

Description:

  • Implements representation for the new Tax Report according to the latest requirements
  • Implements exporting to CSV and PDF support for this report


Preview:

new_tax_report_prev

 
new_tax_raport_export_prev

new-tax-report+export-flow.mov

@alexstotsky alexstotsky changed the title [WIP] (feature) New tax report (feature) New tax report May 16, 2024
@alexstotsky alexstotsky marked this pull request as ready for review May 16, 2024 10:22
@alexstotsky alexstotsky marked this pull request as draft May 20, 2024 09:31
@alexstotsky alexstotsky marked this pull request as ready for review June 20, 2024 15:50
@alexstotsky alexstotsky marked this pull request as draft June 20, 2024 16:25
alexstotsky added 24 commits June 24, 2024 13:53
@alexstotsky
Extend tax report sections constants
88059de
@alexstotsky
Add transactions section url getter
5d0a2fa
@alexstotsky
Add tax transactions key/description
4443a08
@alexstotsky
Implement transactions tab routing
1aa82a4
@alexstotsky
[wip]Tax transactions report
6b01f38
@alexstotsky
Update tax_transactions component
017f9f4
@alexstotsky
Add tax transactions to the main
9556706
@alexstotsky
Cleanup
e99d693
@alexstotsky
Actualize components naming
be2d29e
@alexstotsky
Add tax transacrions columns descriptions
34d88a2
@alexstotsky
Add transactions constants
5fdab16
@alexstotsky
Add tax transactions initial state
ee1b556
@alexstotsky
Implement tax transactions actions
2fe9bd2
@alexstotsky
Add getReqTaxTransactions handler
d042a70
@alexstotsky
Fix params providing
6d1f0bb
@alexstotsky
Implement fetchTransactions saga
4d8c5e0
@alexstotsky
lint fixes
5a43b23
@alexstotsky
Update tax reducers handling cases
6a687ed
@alexstotsky
Actualize tax report selectors
42dd81c
@alexstotsky
Add getTransactionsData selector
aeebb3e
@alexstotsky
Actualize transactions mapped methods and props
ede9775
@alexstotsky
Add refersh transactions handling
b5a576c
@alexstotsky
Fix refresh transactions handling
fc61a70
@alexstotsky
Update locales
dbb71c6
@alexstotsky
Copy link
Contributor Author

Note: rebased against the latest staging without changes from the last review

@alexstotsky alexstotsky marked this pull request as ready for review June 24, 2024 11:00
@alexstotsky alexstotsky mentioned this pull request Jul 2, 2024
Merged
1 task
ezewer
ezewer approved these changes Jul 15, 2024
View reviewed changes
Copy link
Contributor

@ezewer ezewer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@ezewer ezewer merged commit 12bc4c7 into bitfinexcom:staging Jul 15, 2024
@alexstotsky alexstotsky deleted the new-tax-report branch July 20, 2024 10:29
This was referenced Jul 31, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ezewer ezewer ezewer approved these changes

@elvis-poggi elvis-poggi elvis-poggi approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@alexstotsky @ezewer @elvis-poggi