-
Notifications
You must be signed in to change notification settings - Fork 3
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
test(videos): video upload status and review front end tests #466
Conversation
643aac3
to
4fb3d49
Compare
This PR/issue depends on:
|
ping @linhnnk @nganphan123 |
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.
There have been a couple of changes to the video review component as we moved to the new upload logic, so you might want to double-check when you rebase.
any reason why this is still here and not merged? edit: oh duh this is pre-ui overhaul |
@MyStackOverflows yeah i think i'll close this one for now |
Welcome to PrivacyPal! 👋
Fixes: #461
Depends on #499
Description of the change:
Adds Cypress front end tests for
/upload/status
and/upload/review
Motivation for the change:
We need tests!