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

✨ Privacy Policy & EULA #5378

Merged
merged 6 commits into from
Feb 27, 2024
Merged

✨ Privacy Policy & EULA #5378

merged 6 commits into from
Feb 27, 2024

Conversation

ignapas
Copy link
Contributor

@ignapas ignapas commented Feb 26, 2024

What do these changes do?

Adds the Privacy Policy & EULA to the request account form, with links to the documents.
image

Related issue/s

How to test

Can it be tested? (send the form)

Dev Checklist

DevOps Checklist

@ignapas ignapas added t:enhancement Improvement or request on an existing feature a:frontend issue affecting the front-end (area group) labels Feb 26, 2024
@ignapas ignapas self-assigned this Feb 26, 2024
@ignapas ignapas marked this pull request as ready for review February 26, 2024 16:56
@ignapas ignapas requested a review from jsaq007 as a code owner February 26, 2024 16:56
Copy link
Member

@mrnicegyu11 mrnicegyu11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I am going to be annoying I think I need to double check with what the laywer said and adjust. ill let you know!

@pcrespov
Copy link
Member

pcrespov commented Feb 26, 2024

Sorry I am going to be annoying I think I need to double check with what the laywer said and adjust. ill let you know!

@mrnicegyu11 please add the links, as agreed many many times, in ITISFoundation/osparc-issues#1030
@ignapas please only use the links provided by @mrnicegyu11 in that issue

Copy link
Member

@pcrespov pcrespov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comments above.

@mrnicegyu11
Copy link
Member

here is the verbatim text that we need to adhere to, for the record:

When creating the account, the user is informed that their data will be processed in accordance with the linked privacy policy and that they are authorized to use the web product in accordance with the linked EULA. The user should confirm both by ticking the box before he can create the account. The user must only have the opportunity to take note of the two documents by clicking on the link. However, they do not have to do this. However, they must check the box.

@mrnicegyu11
Copy link
Member

@ignapas I am not sure if "please read" is legally sufficient (I really dont know if it is. How I understood it the EULA is formally a contract, so i guess reading it is not sufficient you have to accept the agreement as well). The wording could be along the lines of what was provided by the legal counsel, which is more long, annoying and boring, but maybe safer. Such as:
"Data will be processed in accordance with the linked privacy policy. Users are authorized to use the web product in accordance with the linked EULA."

Potentially also Nik, our GDPR officer, should decide this

@ignapas ignapas requested a review from mrnicegyu11 February 27, 2024 09:52
@ignapas
Copy link
Contributor Author

ignapas commented Feb 27, 2024

Text has been changed according to the one in this comment. The link for the privacy policy has the staging. part stripped out for when the prod website is finally available.

Copy link
Member

@mrnicegyu11 mrnicegyu11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thx 🎉

Copy link
Contributor

@jsaq007 jsaq007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

@ignapas ignapas enabled auto-merge (squash) February 27, 2024 16:05
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@ignapas ignapas merged commit 8977ee2 into ITISFoundation:master Feb 27, 2024
49 checks passed
jsaq007 pushed a commit to jsaq007/osparc-simcore that referenced this pull request Feb 27, 2024
jsaq007 pushed a commit to jsaq007/osparc-simcore that referenced this pull request Feb 27, 2024
jsaq007 pushed a commit to jsaq007/osparc-simcore that referenced this pull request Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:frontend issue affecting the front-end (area group) t:enhancement Improvement or request on an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants