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

download a user's transactions to csv #130

Merged
merged 4 commits into from
Apr 15, 2022
Merged

download a user's transactions to csv #130

merged 4 commits into from
Apr 15, 2022

Conversation

06linux
Copy link
Contributor

@06linux 06linux commented Apr 14, 2022

@wk3368
Copy link
Contributor

wk3368 commented Apr 14, 2022

06linux

这个pr的标题也没有任何意义,跟主题毫不相干
你可以参考一下其他pr的标题,看看都是怎么描述的

@06linux 06linux changed the title 06linux download a user's transactions to csv Apr 14, 2022
@uvd
Copy link
Collaborator

uvd commented Apr 15, 2022

wait me upgrade v5 and react18

Copy link
Contributor

@wk3368 wk3368 left a comment

Choose a reason for hiding this comment

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

LGTM

@wk3368 wk3368 merged commit 8808f71 into main Apr 15, 2022
@wk3368
Copy link
Contributor

wk3368 commented Apr 15, 2022

你的那个pr改动太大,review需要时间,不能因此block别的pr

@ssyuan ssyuan deleted the 06linux branch May 18, 2022 07:50
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.

4 participants