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

"General Terms of Use" field on "Sign Up" page is empty, even on dataverse.harvard.edu, with some Web browsers #7189

Closed
BPeuch opened this issue Aug 12, 2020 · 7 comments · Fixed by #7192

Comments

@BPeuch
Copy link
Contributor

BPeuch commented Aug 12, 2020

Version: 4.20

Hello everybody,

We have run into an issue at SODA, and it seems not just specific to our Dataverse installation.

My colleague @youssefOuahalou was trying to add text to the "General Terms of Use" field that shows up last on the "Sign Up" page:

CTU

After completing the steps to this end, several strange things came about:

  1. @youssefOuahalou did not see any change on his computer, even when changing Web browsers (the ones he tried: Firefox, Chrome, IE, Opera): the field still only contains the default sentence, "There are no Terms of Use for this dataset installation." Emptying the Web browser's cache or going into private navigation was not effective either.

  2. I, on the other hand, do see something new in the "General Terms of Use" field — but not in all Web browsers:
    ✔ ️Firefox
    ✔️ Google Chrome
    ✔️ Opera
    ❌ Internet Explorer
    ❌ Microsoft Edge

Here is what I see in Firefox:

sodhactu

  1. @youssefOuahalou further noticed that, most strangely, the "General Terms of Use" field appears to be "empty" on dataverse.harvard.edu as well (i.e. it only contains the default sentence, "There are no Terms of Use for this dataset installation.") on his side, regardless of the Web browser.

  2. That is not the case on my side, except for ❌ Microsoft Edge, in which this is what I see when trying to sign up on dataverse.harvard.edu:

harvgtu

Maybe Microsoft Edge is problematic in this instance because it is a young browser, so some of its technical specificities clash with Dataverse's code?

We are quite puzzled. 😐

@qqmyers
Copy link
Member

qqmyers commented Aug 12, 2020

@BPeuch - I don't see anything in the code that could be browser-specific - the text is added on the server. However, digging a bit I see that the terms are internationalized - could they be set for one language and not another? If it isn't set for a language, I think it defaults to whatever is set as system.app.terms in the main Bundle which is "There are no Terms of Use for this Dataverse installation."

If that's it, the workaround would be to set system.app.terms (and system.api.terms too probably). However, since there is now a setting (ApplicationTermsOfUse (and ApiTermsofUse)) it might be better for the code to default to the setting for the default language (if set) rather than the system.app.terms entry in in the Bundle.properties file.

@djbrooke
Copy link
Contributor

@BPeuch and @youssefOuahalou thanks for the thorough investigation and the report. This is a fairly important part of the application and we should prioritize getting the correct information displayed here.

@qqmyers thanks also. Yes, if ToU are not available in a specific language it should go with the default language. For example, in dataverse.harvard.edu I'd want everyone to get the English TOU.

@BPeuch
Copy link
Contributor Author

BPeuch commented Aug 12, 2020

Thank you for your replies, @qqmyers and @djbrooke. Indeed @djbrooke, we think this is something rather preoccupying if Dataverse is to be a program that can be implemented in other, non-English-speaking countries (and we know there are quite a few by now).

@qqmyers
Copy link
Member

qqmyers commented Aug 12, 2020

The PR will use the value set for the defaultLocale as the default for any locale unless those are specifically set (as described in http://guides.dataverse.org/en/latest/installation/config.html#id163).

Until then, setting the Bundle.properties system.app.terms value to match your terms, or adding language-specific values for the terms setting would be a work-around.

@BPeuch
Copy link
Contributor Author

BPeuch commented Aug 12, 2020

That's wonderful, @qqmyers. Thank you for working this out so quickly.

Do you think this PR will be integrated in an intermediary version of Dataverse, or is it planned for version 5?

@djbrooke
Copy link
Contributor

Hey @BPeuch - we plan to add this to Dataverse 5, which will be the next release.

@BPeuch
Copy link
Contributor Author

BPeuch commented Aug 12, 2020

Thanks, @djbrooke. And good luck! 🚀

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 a pull request may close this issue.

3 participants