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

End-user agreement must be customized, but is undocumented and hard to find #1693

Open
mwoodiupui opened this issue Jun 17, 2022 · 4 comments

Comments

@mwoodiupui
Copy link
Member

Describe the bug
As delivered, DSpace has a hard-coded "lorem ipsum" end-user agreement. This is buried deeply inside the UI code in a component's template.

Expected behavior
A document which must be customized should be mentioned in the installation and configuration sections of the manual, and should be separated from the code in an easy-to-find location. One should not have to hack an obscure Angular component to provide a license.

@mwoodiupui
Copy link
Member Author

We should also consider app/info/privacy/privacy-content/privacy-content.component.html.

@tdonohue
Copy link
Member

tdonohue commented Jun 17, 2022

Likely instructions could be added to the hints/tips at https://wiki.lyrasis.org/display/DSDOC7x/User+Interface+Customization

I do agree it might be nice to make it easier to find, but at the same time, I think it needs to be an Angular Component, so there may be limitations to where it can be placed. In any case, I agree that at the very least it needs better docs, and maybe we can find ways to make it easier to find/modify.

@tdonohue tdonohue added help wanted Needs a volunteer to claim to move forward Estimate TBD and removed needs triage New issue needs triage and/or scheduling labels Jun 17, 2022
@tdonohue tdonohue added this to To Do in DSpace 7.4 release via automation Jun 17, 2022
@tdonohue tdonohue removed this from To Do in DSpace 7.4 release Oct 11, 2022
@mwoodiupui
Copy link
Member Author

I wonder if the text should be in a bitstream. Then the Agreement could be versioned, and we could just store the UUID of the version most recently accepted.

@tdonohue
Copy link
Member

@mwoodiupui : I'm not against it being converted into a Bitstream (or maybe even a metadata field on the Site object, since metadata fields can support markdown?). However, that seems like a larger design change.

For instance, we'd likely need to force the Bitstream to be a specific format (e.g. HTML or Markdown), if we want it to be displayed in the browser instead of downloaded. Even then, some HTML tags (like <script>) may not be allowed, as those may not play well with Angular (assuming we still want this to be displayed via the UI).

Basically, I think it's possible, but it'd require more testing & verification. I do think this could be a nice idea for the future though... as I see a great benefit to storing these sorts of files as Bitstreams / metadata (the license, privacy policy and other site-wide files could be stored similarly). But, I think we'd need to do some prototyping/testing to see how well this might work, etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: 📋 To Do
Development

No branches or pull requests

2 participants