#1809 Give pubsub_summary_templates access to the name/title of the node
Reporter
Stephen Paul Weber
Owner
Zash
Created
Updated
Stars
★ (1)
Tags
Type-Enhancement
Status-Fixed
Priority-Medium
Milestone-0.13
Stephen Paul Weber
on
Description of feature:
Allow a pubsub_summary_template to reference the title and/or name of the node.
Motivation: (Why?)
So that when getting notifications from the pubsub service I can easily see which node they came from.
Zash
on
https://hg.prosody.im/trunk/rev/c94989c557cd now makes it possible to use the node name as well as the title and description from the config via {@pubsub:node}, {@pubsub:title} and {@pubsub:description}.
Done by injecting these as namespaced attributes on the top level payload element. Other options considered would have involved larger changes to either the template library or making the templates more verbose, e.g. an inner or outer wrapper for the payloads and needing more of the {namespace}name syntax everywhere. Template library changes will have to wait for the 5th incarnation :)
Looking at other node config options, none seemed like something that warranted inclusions, but suggestions welcome.
Description of feature: Allow a pubsub_summary_template to reference the title and/or name of the node. Motivation: (Why?) So that when getting notifications from the pubsub service I can easily see which node they came from.
https://hg.prosody.im/trunk/rev/c94989c557cd now makes it possible to use the node name as well as the title and description from the config via {@pubsub:node}, {@pubsub:title} and {@pubsub:description}. Done by injecting these as namespaced attributes on the top level payload element. Other options considered would have involved larger changes to either the template library or making the templates more verbose, e.g. an inner or outer wrapper for the payloads and needing more of the {namespace}name syntax everywhere. Template library changes will have to wait for the 5th incarnation :) Looking at other node config options, none seemed like something that warranted inclusions, but suggestions welcome.
Changes