diff --git a/app/aws/mocks.py b/app/aws/mocks.py
index 62d6c21c0c..19aa7a4108 100644
--- a/app/aws/mocks.py
+++ b/app/aws/mocks.py
@@ -320,7 +320,6 @@ def _sns_callback(body):
     }
 
 
-# TODO: can we just use the _sns_callback() function instead of this one?
 def _pinpoint_callback(body):
     return {
         "Type": "Notification",
diff --git a/app/celery/process_pinpoint_receipts_tasks.py b/app/celery/process_pinpoint_receipts_tasks.py
index ca0cc92cd5..21410bdcc8 100644
--- a/app/celery/process_pinpoint_receipts_tasks.py
+++ b/app/celery/process_pinpoint_receipts_tasks.py
@@ -114,8 +114,6 @@ def process_pinpoint_results(self, response):
         current_app.logger.exception(f"Error processing Pinpoint results: {str(e)}")
         self.retry(queue=QueueNames.RETRY)
 
-    return True
-
 
 def determine_pinpoint_status(sns_status, provider_response):
     if sns_status == "DELIVERED":