Conversation
scolapasta
left a comment
There was a problem hiding this comment.
See comment about parsing - it's not necessary currently, but in case we change this, for example, to be set at the collection level, someday. (and yes, hopefully by then we'd've retired the JSF, so maybe just being extra careful / paranoid.
100855f to
6cc6c83
Compare
|
Here is the setting update curl command
|
f6caf26 to
e98f790
Compare
807d3a3 to
2f6dbd7
Compare
|
@scolapasta I added the MarkupChecker |
8ed2607 to
b28c6c5
Compare

What this PR does / why we need it: Need to allow the setting :PublishDatasetDisclaimerText to be displayed with line feeds and embedded links
Which issue(s) this PR closes: IQSS/dataverse-pm#521
Special notes for your reviewer:
Suggestions on how to test this: use
for line feeds and <a href= for links. Maybe check for attacks/ embedded js code. This is an admin call so it shouldn't be an issue.
Example:
curl -sS -X PUT -d 'I agree to the following:
1. My submission has been fully anonymized (required for all humab subject'''s datasets).
2. My submission does not violate any known copyright laws.
3. I understand that I am liable for any and all violations of the Harvard Repository<a href="https://support.dataverse.harvard.edu/harvard-dataverse-general-terms-use">Terms of Use.' "http://localhost:8080/api/admin/settings/:PublishDatasetDisclaimerText"
Does this PR introduce a user interface change? If mockups are available, please link/include them here:

Is there a release notes update needed for this change?:
Additional documentation: