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

[Test failure] TC_AUTHOR_36: Broken link in Studio, Course highlight emails, Learn more #432

Closed
angonz opened this issue Dec 4, 2024 · 4 comments
Labels
release testing Affects the upcoming release (attention needed) sumac

Comments

@angonz
Copy link

angonz commented Dec 4, 2024

Release

Sumac

Expected behavior

Studio, Course highlight emails, Learn more should lead to https://edx.readthedocs.io/projects/edx-partner-course-staff/en/latest/developing_course/course_sections.html#set-section-highlights-for-highlight-emails

Actual behavior

It leads to a broken link in https://edx.readthedocs.io/projects/open-edx-building-and-running-a-course/en/open-release-sumac.master/developing_course/course_sections.html#set-section-highlights-for-weekly-course-highlight-messages

Steps to reproduce

Go to Studio, open any course, in the course outline page click on "Learn more" under "Course highlight emails"
image

Additional information

No response

@angonz angonz added release testing Affects the upcoming release (attention needed) needs triage labels Dec 4, 2024
@magajh
Copy link

magajh commented Dec 5, 2024

Thank you for the report! I've confirmed that this issue is occurring

Seems like this link is read from an environment variable specific to the Course Authoring MFE that wasn’t set correctly during deployment? For a moment I thought the URL might be hardcoded, but I couldn’t find it anywhere in the code, and it also doesn’t seem to be returned by the MFE config API

@mariajgrimaldi, do you know who we could tag to help confirm this?

@mariajgrimaldi
Copy link
Member

mariajgrimaldi commented Dec 5, 2024

Thanks for tagging me!

@magajh: those help URLs come from this API /api/contentstore/v1/help_urls. From what I could find, the API generates the links depending on what's in this file and the release branch.

The help links don't work because the release is not available yet. You can find more details on this issue here: #163

Please, feel free to close this since links will work once the release is out.

@mariajgrimaldi
Copy link
Member

@angonz: Can we make a note of this on the test sheet?

@angonz
Copy link
Author

angonz commented Dec 5, 2024

Noted!

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) sumac
Projects
Development

No branches or pull requests

3 participants