WCM-3: add include logic to webhooks #862
Conversation
| @@ -43,6 +43,7 @@ def notify_after_checkin(context, event): | |||
| @grok.subscribe(zeit.cms.interfaces.ICMSContent, zeit.cms.workflow.interfaces.IPublishedEvent) | |||
| def notify_after_publish(context, event): | |||
| create_webhook_job('publish', context, countdown=5) | |||
| create_webhook_job('publish-epaper', context, countdown=5) | |||
There was a problem hiding this comment.
Kann eigentlich weg, da es einfach mehrere <webhook> mit der gleichen ID aber anderer url
Sinnaj94
force-pushed
the
WCM-3
branch
2 times, most recently
from
eb34fba
to
8c7a652
Compare
|
Muss wohl nochmal an die Tests. |
Sinnaj94
changed the title
| <exclude> | ||
| <type>testcontenttype</type> | ||
| </exclude> | ||
| </webhook> |
There was a problem hiding this comment.
dies soll der Fall exclude gewinnt gegen include sein, so wie in Zeile 244ff, richtig? Das ist sehr implizit. Ich schlage vor, einmal einen Test test_exclude_takes_precedence_over_include hinzuzufügen. Es gibt bisher auch keinen simplen <include> Test, analog zu den Tests, die in der WebhookExcludeTest Klasse drin sind.
There was a problem hiding this comment.
benötigen wir diesen Erweiterung der Config noch, nachdem wir die WebhookExcludeStrongerThanIncludeTest oben haben? Der Context hier sind ja die Events, ob irgendwas wegen der In-/Exclude-Bedingung rausfliegt, ist nebensächlich.
| <exclude> | ||
| <type>testcontenttype</type> | ||
| </exclude> | ||
| </webhook> |
There was a problem hiding this comment.
benötigen wir diesen Erweiterung der Config noch, nachdem wir die WebhookExcludeStrongerThanIncludeTest oben haben? Der Context hier sind ja die Events, ob irgendwas wegen der In-/Exclude-Bedingung rausfliegt, ist nebensächlich.
Es wurde ein Webhook für E-Papers hinzugefügt. Um die Konfiguration zu vereinfachen, wurde neben der vorhandenen
<exclude>-Logik zum Ausschließen von Artikeln eine<include>-Logik hinzugefügt. Bei der Implementierung wird zunächst auf die<include>und danach auf die<exclude>-Logik geschaut.Ticket: WCM-3
Checklist
gif