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

Accept ToS #64

Closed
tobiasKaminsky opened this issue Oct 7, 2024 · 4 comments · Fixed by nextcloud/android#14159 or nextcloud/ios#3234
Closed

Accept ToS #64

tobiasKaminsky opened this issue Oct 7, 2024 · 4 comments · Fixed by nextcloud/android#14159 or nextcloud/ios#3234
Assignees
Labels
enhancement New feature or request 🍀 2025-Spring

Comments

@tobiasKaminsky
Copy link
Member

  • sysadmin adds/updates ToS
  • clients will not work
  • Mobile
    • receive push notification
    • show notification/info to redirect to web, to accept ToS
  • Desktop
    • show OS notification/info to redirect to web, to accept ToS
    • replace all info in activity stream with above notification + link
@tobiasKaminsky tobiasKaminsky added the enhancement New feature or request label Oct 7, 2024
@github-project-automation github-project-automation bot moved this to 🧭 Planning evaluation (don't pick) in 🤖 🍏 Mobile clients team Oct 7, 2024
@tobiasKaminsky tobiasKaminsky moved this from 🧭 Planning evaluation (don't pick) to 📄 To do (max 2 entries / member) in 🤖 🍏 Mobile clients team Oct 7, 2024
@AndyScherzinger
Copy link
Member

I tend to say launch a dialog on Windows since notifications might get ignored or overlooked since we are talking non-tech people who might not interaction with OS level notification or might have deactivated them (you can do so on Windows). So the only safe bet is "in-app" launching a dialog window.

@tobiasKaminsky tobiasKaminsky moved this from 📄 To do (max 2 entries / member) to 🏗️ In progress in 🤖 🍏 Mobile clients team Dec 9, 2024
@tobiasKaminsky
Copy link
Member Author

@marinofaggiana

  • webdav returns 403 with
OCA\\TermsOfService\\TermsNotSignedException
  • you can then retrieve text to display with /ocs/v2.php/apps/terms_of_service/terms
  • accept it with: POST to /ocs/v2.php/apps/terms_of_service/sign with termId: id of above

@marinofaggiana
Copy link
Member

Mobile

  • receive push notification
    (what's ) ?
  • show notification/info to redirect to web, to accept ToS

I have configured (only) the webdav returns 403

Note. Is it a good idea to also set up the accepted view (read only) in the settings/information, " Tems of service" ?

This was linked to pull requests Dec 11, 2024
@tobiasKaminsky
Copy link
Member Author

@Rello @mgallien Mobile is ready and working.
Can you double check it on Desktop?

There is a LTD test server, for which I can give you credentials via chat.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request 🍀 2025-Spring
Projects
Status: ☑️ Done
Development

Successfully merging a pull request may close this issue.

3 participants