feat: add support for a technical support URL in the LTI-based provider download instructions #127
Conversation
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #127 +/- ##
==========================================
- Coverage 94.10% 93.95% -0.16%
==========================================
Files 71 71
Lines 1051 1058 +7
Branches 289 289
==========================================
+ Hits 989 994 +5
- Misses 57 59 +2
Partials 5 5 ☔ View full report in Codecov by Sentry. |
| <FormattedMessage | ||
| id="exam.DownloadSoftwareProctoredExamInstructions.supportText" | ||
| id="exam.DownloadSoftwareProctoredExamInstructions.LTI.supportText.URL" |
There was a problem hiding this comment.
I noticed that there is a FormattedMessage in RestProviderInstructions.jsx with the same original ID (exam.DownloadSoftwareProctoredExamInstructions.supportText). Does anyone know if that would cause a conflict with this original string? I added LTI to differentiate, but I'm not sure if the strings end up namespaced by component or something.
There was a problem hiding this comment.
also not sure but differentiating them seems wise
…er download instructions This commit adds support for displaying a technical support URL for LTI-based providers on the download instructions interstitial. The download instructions will display a technical support URL when it is returned from the proctoring settings backend endpoint. If the technical support URL is not available, then the technical support email and technical support phone number will be used instead.
MichaelRoytman
force-pushed
the
michaelroytman/COSMO-130-tech-support-site
branch
from
8a7e085
to
141ea62
Compare
| <FormattedMessage | ||
| id="exam.DownloadSoftwareProctoredExamInstructions.supportText" | ||
| id="exam.DownloadSoftwareProctoredExamInstructions.LTI.supportText.URL" |
There was a problem hiding this comment.
also not sure but differentiating them seems wise
MichaelRoytman
deleted the
michaelroytman/COSMO-130-tech-support-site
branch
Description
Jira: COSMO-130
Dependencies: edx/edx-exams#231
This commit adds support for displaying a technical support URL for LTI-based providers on the download instructions interstitial. The download instructions will display a technical support URL when it is returned from the proctoring settings backend endpoint. If the technical support URL is not available, then the technical support email and technical support phone number will be used instead.
Screenshots
supportURL
supportEmail, supportPhone
no supportURL, supportEmail, supportPhone