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

Add Privacy policy and Terms of service: #345

Merged
merged 4 commits into from
Apr 8, 2024
Merged

Conversation

MelissaAutumn
Copy link
Member

@MelissaAutumn MelissaAutumn commented Mar 27, 2024

Fixes #51

  • Uses Thunderbird's Privacy Policy.
  • Uses Website's Terms of Service.
  • Manual command to pull new legal updates as needed.
  • LegalView to pull down the html from the backend.

I was going to make this a plugin on the frontend to pull and render to html on vite build, but that would have taken longer lol. This is the simplest way, and when there's an update you just need to run run-command main download-legal.

If we're worried about io reads, I can hook the routes up to redis caching.

Screen Shot 2024-03-27 at 09 32 43-fullpage

Screen Shot 2024-03-27 at 09 32 22-fullpage

cc @Sancus @malini

* Uses Thunderbird's Privacy Policy.
* Uses Website's Terms of Service.
* Manual command to pull new legal updates as needed.
* LegalView to pull down the html from the backend.
@MelissaAutumn MelissaAutumn requested a review from devmount March 27, 2024 16:35
@MelissaAutumn MelissaAutumn self-assigned this Mar 27, 2024

extensions = ['markdown.extensions.attr_list']
# Only english for now. There's no german TB privacy policy?
locales = ['en']
Copy link
Member Author

Choose a reason for hiding this comment

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

I was going to pull based on locale, but legal doesn't have a german privacy policy for TB 🤔

Copy link
Contributor

Choose a reason for hiding this comment

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

@Sancus who do we prod to get privacy policy translated?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Well, if you have problems to find somebody 🤚🏻 I'm not exactly a professional translator but a native speaker at least 😅 I can provide a PR if you want.

Copy link
Contributor

Choose a reason for hiding this comment

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

@devmount thanks for the offer! I'm looking into getting this translated right now. We should setup a pipeline for how to do it without relying on SDEs 😅

Copy link
Collaborator

Choose a reason for hiding this comment

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

Of course, still wanted to offer that 😇

@malini
Copy link
Contributor

malini commented Mar 27, 2024

looks okay for now, I've added a ticket to automate this process in the post-mvp doc.

I'm proposing we make a scheduled task to run this command and check for changes. If there are changes, it'll open a PR with the change and notify us.

Copy link
Collaborator

@devmount devmount left a comment

Choose a reason for hiding this comment

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

Thanks for this, just a suggestion and a question.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Should we document all commands in the readme? I always appreciate having commands listed somewhere with some simple example call.

@@ -0,0 +1,53 @@
<template>
<div v-html="pageContents">
Copy link
Collaborator

Choose a reason for hiding this comment

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

Are we 100% sure, that pageContents does not contain any user manipulated content? Thinking about XSS here. Maybe we can at least filter the contents for some predefined tags, or is v-html safer now than in the past?

Copy link
Member Author

Choose a reason for hiding this comment

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

I can adjust this PR to generate a vue view component instead to resolve this. It's a good thing both projects are in the same repo.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes I think that would be much better 👍🏻

@MelissaAutumn
Copy link
Member Author

I kept the command, but just tell the person running it to copy the html files over to the frontend project lol. I then import it as raw text via vite, and use the v-html. Which is a-okay because it'll be compiled in, no network request needed.

@MelissaAutumn
Copy link
Member Author

Added backend readme. I'll add more details at a later date.

@MelissaAutumn MelissaAutumn requested a review from devmount April 4, 2024 17:31
Copy link
Collaborator

@devmount devmount left a comment

Choose a reason for hiding this comment

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

Looking good, thanks Mel!

backend/README.md Outdated Show resolved Hide resolved
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nice, thank you for this documentation 👏🏻

@MelissaAutumn
Copy link
Member Author

I'm going to merge this, we can look into German ToS/Privacy another time.

@MelissaAutumn MelissaAutumn merged commit 78929c4 into main Apr 8, 2024
2 checks passed
@MelissaAutumn MelissaAutumn deleted the tasks/51-add-legal branch April 8, 2024 15:31
jdbass pushed a commit that referenced this pull request May 17, 2024
* Add Privacy policy and Terms of service:
* Uses Thunderbird's Privacy Policy.
* Uses Website's Terms of Service.
* Manual command to pull new legal updates as needed.
* LegalView to pull down the html from the backend.

* Download to tmp, tell the individual to copy 'em over. Pull in local html files, and we should be good to go.

* Add backend readme.

* Update backend/README.md

Co-authored-by: Andreas <[email protected]>

---------

Co-authored-by: Andreas <[email protected]>
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.

Add Privacy Policy and Terms of Service
3 participants