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

Soften shame message #3541

Merged
merged 4 commits into from
Mar 1, 2021
Merged

Soften shame message #3541

merged 4 commits into from
Mar 1, 2021

Conversation

timgl
Copy link
Collaborator

@timgl timgl commented Mar 1, 2021

Changes

Please describe.
If this affects the frontend, include screenshots.

Checklist

  • All querysets/queries filter by Organization, by Team, and by User
  • Django backend tests
  • Jest frontend tests
  • Cypress end-to-end tests

@timgl
Copy link
Collaborator Author

timgl commented Mar 1, 2021

I do think the message is funny but external people/potential candidates will read it too and will think we are a horrible place to work at

@timgl timgl temporarily deployed to posthog-soften-shame-me-lpnljz March 1, 2021 18:37 Inactive
@posthog-bot
Copy link
Contributor

Hello!

👋

This pull request seems to contain no description. Please add useful context, rationale, and/or any other information that will help make sense of this change now and in the distant Mars-based future.

@Twixes
Copy link
Collaborator

Twixes commented Mar 1, 2021

LOL
That's fair.
In this case the is-collaborator step is obsolete and can be removed.

@Twixes Twixes temporarily deployed to posthog-soften-shame-me-lpnljz March 1, 2021 18:44 Inactive
@Twixes Twixes temporarily deployed to posthog-soften-shame-me-lpnljz March 1, 2021 18:45 Inactive
@Twixes Twixes temporarily deployed to posthog-soften-shame-me-lpnljz March 1, 2021 19:28 Inactive
@Twixes Twixes merged commit 6215f77 into master Mar 1, 2021
@Twixes Twixes deleted the soften-shame-message branch March 1, 2021 20:28
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