Edgewall Software

Changes between Initial Version and Version 1 of Ticket #11928, comment 11


Ignore:
Timestamp:
Mar 13, 2017, 1:10:08 PM (7 years ago)
Author:
Jun Omae

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #11928, comment 11

    initial v1  
    99 * [c455774f9/jomae.git] applied T11928_save_notification_format_session_pref.diff
    1010
    11 Adding `[notification] default_format.email` option would be able to edit the value via [th:TracIniAdminPanelPlugin] and [th:IniAdminPlugin]. I think it is not easy to edit `[notification-subscriber]` section.
    12 
    13 I consider it might be good to use `notification.format.<transport>` rather than `notification.default_format.<transport>'. I think the `notification.default_format.<transport>` is verbose a bit.
     11Comments:
     12 * I don't think it it good to use `get_preferred_format()` in each subscriber.
     13 * Adding `[notification] default_format.email` option would be able to edit the value via [th:TracIniAdminPanelPlugin] and [th:IniAdminPlugin]. I think it is not easy to edit `[notification-subscriber]` section.
     14 * I consider it might be good to use `notification.format.<transport>` rather than `notification.default_format.<transport>'. I think the `notification.default_format.<transport>` is verbose a bit.