Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: sitescripts/management/bin/generateNotifications.py

Issue 6582297721569280: Issue 2275 - Parse notification groups (Closed)
Patch Set: Rebased Created April 15, 2015, 9:26 a.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | sitescripts/notifications/parser.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: sitescripts/management/bin/generateNotifications.py
===================================================================
--- a/sitescripts/management/bin/generateNotifications.py
+++ b/sitescripts/management/bin/generateNotifications.py
@@ -24,6 +24,9 @@
def generate_notifications(path):
notifications = load_notifications()
+ # Ignoring notifications with variants here - we can only process those in a
+ # URL handler.
+ notifications = [x for x in notifications if "variants" in x]
output = {
"notifications": notifications,
"version": time.strftime("%Y%m%d%H%M", time.gmtime())
« no previous file with comments | « no previous file | sitescripts/notifications/parser.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld