-
Notifications
You must be signed in to change notification settings - Fork 313
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: Enable copying of publicly shared question code (if allowed by instructor) #9800
base: master
Are you sure you want to change the base?
Conversation
All images
|
...learn/src/pages/instructorCopyTemplateCourseQuestion/instructorCopyTemplateCourseQuestion.ts
Show resolved
Hide resolved
I am aware that this doesn't actually make a way to mark the Doing that is going to be a bunch of UI design/mess on the question settings page, I thought it would be nice to do that in a separate PR from this. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #9800 +/- ##
==========================================
- Coverage 67.01% 66.89% -0.13%
==========================================
Files 458 458
Lines 71353 71351 -2
Branches 5720 5703 -17
==========================================
- Hits 47820 47732 -88
- Misses 23104 23194 +90
+ Partials 429 425 -4 ☔ View full report in Codecov by Sentry. |
implements parts of #9397