-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Notify owners about organization deletion #4116
Conversation
c9945c7
to
6b4d0a2
Compare
def get_owners(self): | ||
from sentry.models import User | ||
return User.objects.filter( | ||
sentry_orgmember_set__role=roles.get_top_dog().id, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Favorite.
My stab at copy with one clear action that will work in both the app and email: Organization queued for deletionThe Sentry organization has been scheduled for deletion by:
This irreversible deletion will take place at Sept. 10, 2016, 11:46 p.m. and will permanently remove all associated data including events, projects, and team members. If this was unintentional and you would like to cancel the deletion:[Restore Organization] |
event=AuditLogEntryEvent.ORG_RESTORE, | ||
data=organization.get_audit_log_data(), | ||
) | ||
delete_logger.info('object.delete.canceled', extra={ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TIL it's normal to have both canceled
and cancelled
in English.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
YUP
Reviewed 8 of 12 files at r1. Comments from Reviewable |
683dd8e
to
d2f7156
Compare
d2f7156
to
7362368
Compare
Alright, tests have been written, and removed "WIP" tag. :) |
7362368
to
da3dd05
Compare
fix yo linters yo |
And prompt them to cancel and restore the organization.
da3dd05
to
d763823
Compare
@dcramer fixed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
And prompt them to cancel and restore the organization.
@getsentry/infrastructure @getsentry/ui
Help on messaging here would be nice since what I put is likely garbage.
Web UI:
Email:
TODO:
This change is