From 476e14fcb765b7c903f2a90a6666d52fbe50c66c Mon Sep 17 00:00:00 2001 From: Jason Robbins Date: Fri, 10 Jan 2025 01:48:41 +0000 Subject: [PATCH] =?UTF-8?q?Revert=20"Add=20check=20that=20user=20can=20vie?= =?UTF-8?q?w=20feature=20before=20sending=20change=20notificati=E2=80=A6"?= =?UTF-8?q?=20(#4687)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This reverts commit acab78e9c4517521b82556904689fd1c8d829f8c. --- internals/notifier.py | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/internals/notifier.py b/internals/notifier.py index e14ec30835f1..30add6f98e57 100644 --- a/internals/notifier.py +++ b/internals/notifier.py @@ -454,10 +454,7 @@ def process_post_data(self, **kwargs): # actually changes to it. # Load feature directly from NDB so as to never get a stale cached copy. fe = FeatureEntry.get_by_id(feature['id']) - user = users.User(email=triggering_user_email) - can_view_feature = permissions.can_view_feature(user, fe) - - if fe and (is_update and len(changes) or not is_update) and can_view_feature: + if fe and (is_update and len(changes) or not is_update): email_tasks = make_feature_changes_email( fe, is_update=is_update, changes=changes, triggering_user_email=triggering_user_email)