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

Broken link to privacy policy when accepting new terms of service #9272

Closed
2 tasks done
yvanzo opened this issue May 18, 2023 · 2 comments
Closed
2 tasks done

Broken link to privacy policy when accepting new terms of service #9272

yvanzo opened this issue May 18, 2023 · 2 comments
Assignees
Labels
bug Something is broken.
Milestone

Comments

@yvanzo
Copy link
Contributor

yvanzo commented May 18, 2023

Describe the issue

When logging into a Weblate instance where the legal terms have been changed, the user is appropriately redirected to the form /legal/confirm/ which includes the new Terms of Service in a pre-scrollable div.

In this context, the (new) link to the privacy policy from the section 5 of the ToS is not working: It resolves to None. It seems that the template {{ privacy_url }} doesn’t resolve correctly in this context.

In other contexts, this link works fine:

  • From /legal/terms/, it resolves to /legal/privacy/.
  • From the all-in-one /en/terms/, it resolves to #privacy.

Template for the link: https://github.com/WeblateOrg/wllegal/blob/2023.3/wllegal/templates/legal/documents/tos.html#LL101C211-L101C235

I’m reporting this bug to wllegal as it occurs since its latest version 2023.3.
However I don’t know if it has to be addressed in wllegal and/or weblate.
I’ve searched for similar issues in both repositories.

I already tried

  • I've read and searched the documentation.
  • I've searched for similar issues in this repository.

Steps to reproduce the behavior

  1. Go to /legal/confirm/?… (with an account having accepted old legal terms only, or using a new account?)
  2. Scroll to the section 5 about personal data
  3. Click on the link to the privacy policy
  4. Get page not found from /legal/confirm/None

Expected behavior

  1. Get /legal/privacy/, possibly in a new tab.

Screenshots

No response

Exception traceback

No response

Additional context

No response

@nijel nijel transferred this issue from WeblateOrg/wllegal May 19, 2023
@nijel nijel added the bug Something is broken. label May 19, 2023
@nijel nijel added this to the 4.18 milestone May 19, 2023
@nijel nijel self-assigned this May 19, 2023
@nijel
Copy link
Member

nijel commented May 19, 2023

Fixed in 45ddcee

@nijel nijel closed this as completed May 19, 2023
nijel added a commit to WeblateOrg/docker that referenced this issue May 19, 2023
@github-actions
Copy link

Thank you for your report; the issue you have reported has just been fixed.

  • In case you see a problem with the fix, please comment on this issue.
  • In case you see a similar problem, please open a separate issue.
  • If you are happy with the outcome, don’t hesitate to support Weblate by making a donation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is broken.
Projects
None yet
Development

No branches or pull requests

2 participants