Skip to content

fix: fix broken proctoring settings link in studio#35528

Merged
varshamenon4 merged 2 commits intomasterfrom
varshamenon4/fix-proctoring-settings-link
Sep 24, 2024
Merged

fix: fix broken proctoring settings link in studio#35528
varshamenon4 merged 2 commits intomasterfrom
varshamenon4/fix-proctoring-settings-link

Conversation

@varshamenon4
Copy link
Contributor

Description

This PR fixes a bug in studio. When there is a proctoring error (see screenshot) there is a page alert that includes a link. This link is currently broken due to an html formatting issue. This change will impact the course author since the bug is in studio.

Screenshot 2024-09-24 at 9 27 40 AM

Testing instructions

Deadline

This needs to be merged ASAP, by 9/27 to coincide with the removal of PSI as a proctoring provider. We expect this page alert to appear for users with software secure as their proctoring provider so it would be important to have this link fixed by then.

@zacharis278
Copy link
Contributor

thanks for catching this!

Copy link
Contributor

@schenedx schenedx left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@varshamenon4 varshamenon4 merged commit c396b52 into master Sep 24, 2024
@varshamenon4 varshamenon4 deleted the varshamenon4/fix-proctoring-settings-link branch September 24, 2024 14:58
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants