[Asterisk-code-review] res calendar: Add support for fetching calendars when reloading (asterisk[master])
Matt Jordan
asteriskteam at digium.com
Mon Oct 10 12:57:14 CDT 2016
Matt Jordan has posted comments on this change.
Change subject: res_calendar: Add support for fetching calendars when reloading
......................................................................
Patch Set 3:
Given:
- The lack of calendar tests that we have available
- Any tests that would be written would have to be written against a mock calendar backend, which would be fairly involved
- The new feature is extremely small and limited with low chance of any kind of regression
I'd vote to not require a test for this improvement.
--
To view, visit https://gerrit.asterisk.org/3998
To unsubscribe, visit https://gerrit.asterisk.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I384b02ebfa42b142bbbd5b7221458c7f4dee7077
Gerrit-PatchSet: 3
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Ludovic Gasc (Eyepea) <gmludo at gmail.com>
Gerrit-Reviewer: Anonymous Coward #1000019
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>
Gerrit-HasComments: No
More information about the asterisk-code-review
mailing list