-
Notifications
You must be signed in to change notification settings - Fork 699
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
API and CSV fixes #9523
API and CSV fixes #9523
Conversation
bd8e646
to
95e336d
Compare
No sign of UI issues and/or errors in the console, good to go! 👍🏽 |
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.
Manual QA is looking good, !
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.
lThere's one point I don't quite understand so it would be great if you can comment it.
For the most part, loving these changes
Have addressed all comments - as long as tests pass, this should be good to merge. |
Summary
Testing checklist
PR process
Reviewer checklist
yarn
andpip
)