Index: sitescripts/notifications/test/notification.py
===================================================================
--- a/sitescripts/notifications/test/notification.py
+++ b/sitescripts/notifications/test/notification.py
@@ -361,5 +361,14 @@
     self.assertEqual(result["notifications"][0]["id"], "b")
     self.assertEqual(result["notifications"][1]["id"], "c")
 
+  def test_inactive_notification_variant_not_returned(self):
+    self.load_notifications_mock.return_value = [
+      {"id": "a", "inactive": True}
+    ]
+    result = json.loads(notification.notification({
+      "QUERY_STRING": "lastVersion=197001010000-a/1"
+    }, lambda *args: None))
+    self.assertEqual(len(result["notifications"]), 0)
+
 if __name__ == '__main__':
   unittest.main()
Index: sitescripts/notifications/web/notification.py
===================================================================
--- a/sitescripts/notifications/web/notification.py
+++ b/sitescripts/notifications/web/notification.py
@@ -59,7 +59,9 @@
     variant = group["variant"]
     if variant == 0:
       continue
-    notification = next(x for x in notifications if x["id"] == group_id)
+    notification = next((x for x in notifications if x["id"] == group_id), None)
+    if not notification:
+      continue
     notification = copy.deepcopy(notification)
     notification.update(notification["variants"][variant - 1])
     for key_to_remove in ("sample", "variants"):