chore(member merge): clean up users without email addresses#101981
chore(member merge): clean up users without email addresses#101981
Conversation
github-actions
bot
added
the
Scope: Backend
mifu67
changed the base branch from
master
to
mifu67/member-merge/sentry-app-proxy-user-email-backfill-self-hosted
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
|
|
||
| @control_silo_test | ||
| @override_settings(SENTRY_SELF_HOSTED=True) | ||
| class BackfillSelfHostedSentryAppEmailsTest(TestMigrations): |
There was a problem hiding this comment.
Bug: Test Class Name Mismatch
The test class BackfillSelfHostedSentryAppEmailsTest has an incorrect name. This looks like a copy-paste from the 0997 migration test. The class is actually validating the 0998 migration, which deletes never-active users without emails, so the current name doesn't accurately reflect its purpose.
|
This pull request has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you add the label "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
|
This pull request has gone three weeks without activity. In another week, I will close it. But! If you comment or otherwise update it, I will reset the clock, and if you add the label "A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀 |
mifu67
changed the base branch from
mifu67/member-merge/sentry-app-proxy-user-email-backfill-self-hosted
to
master
mifu67
force-pushed
the
mifu67/member-merge/clean-up-users-without-emails-self-hosted
branch
from
38f4381 to
2a0b17e
Compare
mifu67
deleted the
mifu67/member-merge/clean-up-users-without-emails-self-hosted
branch
4 participants
Duplicate of https://github.com/getsentry/getsentry/pull/17549 using a migration for self-hosted users.