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

[TASK-114] TOS form #4711

Merged
merged 38 commits into from Nov 16, 2023
Merged

[TASK-114] TOS form #4711

merged 38 commits into from Nov 16, 2023

Conversation

magicznyleszek
Copy link
Member

@magicznyleszek magicznyleszek commented Oct 31, 2023

Checklist

  1. If you've added code that should be tested, add tests
  2. If you've changed APIs, update (or create!) the documentation
  3. Ensure the tests pass
  4. Make sure that your code lints and that you've followed our coding style
  5. Write a title and, if necessary, a description of your work suitable for publishing in our release notes
  6. Mention any related issues in this repository (as #ISSUE) and in other repositories (as kobotoolbox/other#ISSUE)
  7. Open an issue in the docs if there are UI/UX changes

Description

Displays a TOS form content for users who didn't accept TOS yet. Will include a form for updating all required user metadata fields.

Notes

Things changed here:

  • New component TOSForm that displays the TOS text with the form for updating required user fields
  • Added fetchGetUrl to api.ts
  • Added one helper function to envStore
  • Added logOut function to session store

It requires to update the Sitewide Messages from the /admin page. It needs to have at least terms_of_service (the slug of the sitewide message) added. It also handles translated TOS Announcement message text with e.g. terms_of_service_fr for francais.

@magicznyleszek magicznyleszek changed the title Feature tos tos form [TASK-114] TOS form Oct 31, 2023
@notion-workspace
Copy link

TOS Form

@magicznyleszek magicznyleszek mentioned this pull request Nov 1, 2023
@magicznyleszek magicznyleszek marked this pull request as ready for review November 8, 2023 15:25
@p2edwards p2edwards self-assigned this Nov 8, 2023
@p2edwards p2edwards self-requested a review November 8, 2023 22:32
@p2edwards p2edwards merged commit 7c38074 into feature/tos Nov 16, 2023
4 checks passed
@p2edwards p2edwards deleted the feature-tos--tos-form branch November 16, 2023 22:18
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 this pull request may close these issues.

None yet

3 participants