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

When the replace video button is clicked, the page goes blank #1540

Open
mariajgrimaldi opened this issue Dec 2, 2024 · 5 comments
Open
Labels
release testing Affects the upcoming release (attention needed)

Comments

@mariajgrimaldi
Copy link
Member

When selecting replace video from the video editor the page goes blank without any error message in the console or any indication of what went wrong.

Expected behavior

It is expected that clicking the replace video button will redirect the page to the unit where the component is located.

Actual behavior

When you click the replace video button, the page appears blank because it is not redirecting properly.

Steps to reproduce

  1. Add a new video component.
  2. Edit the component
  3. Click the 'replace video' button.

Additional information

Screenshot From 2024-11-14 09-33-06

Originally reported here: openedx/wg-build-test-release#418

@mariajgrimaldi mariajgrimaldi added the release testing Affects the upcoming release (attention needed) label Dec 2, 2024
@mariajgrimaldi
Copy link
Member Author

@arbrandes @bradenmacdonald @openedx/2u-tnl, tagging you for visibility. Please, tag who you think can help us solve this issue. Thank you!

@bradenmacdonald
Copy link
Contributor

I'm not familiar with this, sorry. Why do we have a link back to the unit page anyways? It seems different than the other editors which should all use the same "Cancel" button to close the modal.

@mariajgrimaldi
Copy link
Member Author

mariajgrimaldi commented Dec 10, 2024

I completely forgot to update the issue with what @farhaanbukhsh found about this feature. Sorry. Here's the thread: https://openedx.slack.com/archives/C049JQZFR5E/p1733414487074569

@farhaanbukhsh
Copy link
Member

@mariajgrimaldi @bradenmacdonald you can also have a look at the Figma board

https://www.figma.com/design/cS42BX4OUsR8lOw6ppZHoz/Content-Libraries

Replace video should be redirected to the Gallery of videos and if this is not working it's possibly a bug.

@mariajgrimaldi
Copy link
Member Author

@crathbun428: Could you help us figure out whether this should be working?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release testing Affects the upcoming release (attention needed)
Projects
Status: No status
Development

No branches or pull requests

3 participants