@diosmosis opened this Pull Request on December 12th 2018 Member

Simple solution that does not support range periods.

Refs #13819

@tsteur commented on December 13th 2018 Member

Can we list the schedule first and then report period? and add some inline help for report period? also I would maybe automatically set report period to the same value when changing the email schedule but not the other way around. By default they should always match unless someone changes report period. If they changed report period, and then change the schedule, I would again also set report period to the same as email schedule.(no need to remember whether it was different I think, but could maybe... not sure which is better).

@diosmosis commented on December 13th 2018 Member

👍 will do

@diosmosis commented on December 17th 2018 Member

Updated the PR. @tsteur let me know if this can go in 3.8

Powered by GitHub Issue Mirror