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

feat: set links for CourseAuthoring dicussion alert #673

Conversation

CefBoud
Copy link

@CefBoud CefBoud commented Aug 26, 2024

Description

There is an alert in CourseAuthoring MFE to inform about the usage of an upgraded version of discussion. This PR sets the links used in that alert.

ref

BB-9079

Copy link
Member

@DanielVZ96 DanielVZ96 left a comment

Choose a reason for hiding this comment

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

@DanielVZ96 DanielVZ96 self-requested a review August 26, 2024 23:40
Copy link
Member

@DanielVZ96 DanielVZ96 left a comment

Choose a reason for hiding this comment

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

I think this should be done with a tutor plugin as this is not an upstreamable change
uhh maybe it should be? haha

if yes, please fix the test that is failing

@CefBoud CefBoud force-pushed the cef/BB-9079/set-links-for-course-authoring-alert branch 4 times, most recently from 41a16b4 to 0f22e79 Compare August 27, 2024 14:25
@CefBoud CefBoud force-pushed the cef/BB-9079/set-links-for-course-authoring-alert branch from 0f22e79 to 2337aa9 Compare August 27, 2024 15:03
@CefBoud CefBoud merged commit 09b2168 into opencraft-release/redwood.1 Aug 28, 2024
77 checks passed
@CefBoud CefBoud deleted the cef/BB-9079/set-links-for-course-authoring-alert branch August 28, 2024 10:33
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.

2 participants