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

Back button does not work from Discussion #703

Open
mariajgrimaldi opened this issue May 20, 2024 · 7 comments
Open

Back button does not work from Discussion #703

mariajgrimaldi opened this issue May 20, 2024 · 7 comments
Assignees
Labels
release testing Affects the upcoming release (attention needed)

Comments

@mariajgrimaldi
Copy link
Member

Expected behavior

Clicking your browser's "back" button from the Discussion tab should take you to the previous page.

Steps to reproduce

  1. Start on any course page, such as https://apps.redwood.demo.edly.io/learning/course/course-v1:HinaX+HX101+2024_T2/home
  2. Click on "Discussion".
  3. Use your browser's back button.
  4. Witness a brief moment of "All posts sorted by recent activity" on the left.

The URL changes briefly from
https://apps.redwood.demo.edly.io/discussions/course-v1:HinaX+HX101+2024_T2/posts
to
https://apps.redwood.demo.edly.io/discussions/course-v1:HinaX+HX101+2024_T2/
and then back to
https://apps.redwood.demo.edly.io/discussions/course-v1:HinaX+HX101+2024_T2/posts

The original issue was reported on the BTR board as part of the redwood testing effort: openedx/wg-build-test-release#346

@mariajgrimaldi
Copy link
Member Author

FYI @arbrandes @brian-smith-tcril

@mariajgrimaldi
Copy link
Member Author

Given the catalog-info.yml, I'll also tag you folks @openedx/2u-infinity here. Thanks!

@mariajgrimaldi mariajgrimaldi added the release testing Affects the upcoming release (attention needed) label May 20, 2024
@arbrandes arbrandes self-assigned this May 20, 2024
@ayesha-waris
Copy link
Contributor

@mariajgrimaldi I don't have access to the URLs mentioned in the issue report, but I tested on the edX production environment and couldn't reproduce the problem.
The URL changes are from
https://learning.edx.org/course/course-v1:edX+Inf101+2022/home
To
https://discussions.edx.org/course-v1:edX+Inf101+2022/posts
Back to
https://learning.edx.org/course/course-v1:edX+Inf101+2022/home
Would you like me to try anything else?

@mariajgrimaldi
Copy link
Member Author

mariajgrimaldi commented Jun 12, 2024

@ayesha-waris: you can test here (redwood sandbox):
https://redwood.demo.edly.io/
Here are the credentials: https://github.com/overhangio/openedx-release-demo?tab=readme-ov-file#open-edx-release-demo-platform-cd
I'm not sure what's wrong, though. I tried this: /home -> /posts, but I can't get out of /posts in that installation. What version is edX using? Maybe it got fixed after the redwood cut.

@ayesha-waris
Copy link
Contributor

@mariajgrimaldi tested on https://apps.redwood.demo.edly.io/learning/course/course-v1:OpenedX+DemoX+DemoCourse/home and it's working as expected. Can you please try clearing your chache.
attaching GIF for reference
chrome-capture-2024-6-12

@mariajgrimaldi
Copy link
Member Author

mariajgrimaldi commented Aug 6, 2024

I tested it again in a private window:

Screencast.from.06-08-24.09.31.30.mp4

Can you use firefox? In Chrome, it seems to work.

@mariajgrimaldi
Copy link
Member Author

Hi @ayesha-waris! Could you let me know if you could reproduce this using Firefox?

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: Backlog
Development

No branches or pull requests

3 participants