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.

Sign up for GitHub

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

Jump to bottom

Add TOS to start email to decrease accidental abuse #2073

Closed
wants to merge 2 commits into from
Closed

Add TOS to start email to decrease accidental abuse #2073

wants to merge 2 commits into from

Conversation

Geri680
Copy link

@Geri680 Geri680 commented Mar 16, 2024

Fix #2072

The same as #2071 but for email

@Geri680
Copy link
Author

Geri680 commented Mar 16, 2024

@nguyenkims can you please take a look? :)

nguyenkims
nguyenkims reviewed Mar 18, 2024
View reviewed changes
Copy link
Contributor

@nguyenkims nguyenkims left a comment

Choose a reason for hiding this comment

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

Hi thanks for making the PR. We try to keep the emails as short as possible, and the info here actually doesn't concern most users. (most users aren't using SL for mass signups or any abusive activity)

@Geri680 Geri680 closed this Mar 18, 2024
@Geri680 Geri680 deleted the policy-email branch March 18, 2024 11:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nguyenkims nguyenkims nguyenkims left review comments

@acasajus acasajus Awaiting requested review from acasajus acasajus is a code owner

@cquintana92 cquintana92 Awaiting requested review from cquintana92 cquintana92 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

del
2 participants
@Geri680 @nguyenkims