Commit Graph

8 Commits

Author SHA1 Message Date
Martin Molinero f7665bf596 Address reviews
- Adding missing header file
- Adding string.Empty as default value for email data/message and subject. If null will use string.Empty
2019-07-10 18:44:28 -03:00
Michael Handschuh eeae2e291e BUG #3021: Validate email addresses in NotificationEmail
* Adds Validate class in util with accompanying regular expressions
* Provides tests for NotificationEmail and Validate.EmailAddress
* Provides default values for NotificationEmail Subject and Message
  These default values resolve the underlying issue of #3021 w/out
  needing to modify external sources (such as impl of IMessagingHandler)
2019-07-08 02:52:04 -04:00
Michael Handschuh b273487fcc BUG #3021: Refactor NotificationManager
* Moves _liveMode check into Allow method for stream-lined Email/Sms/Web
* Updates default _resetTime to default(DateTime) to make explicit the first
  invocation semantics
* Updates _resetTime increment to be hourly instead of rounded hours as this
  caused strange behavior in the units tests when too close to the hour
* Synchronizes Allow method to avoid potential race conditions with += on
  _count and setting of the _resetTime
2019-07-08 02:52:04 -04:00
QuantConnect fde0cbed3a Strip out block quote section header comments 2015-06-13 16:04:27 -04:00
snugs 89b6e98af8 Fixes warnings mostly relating to xml comments
Also cleans up some block comments

Via PR#72 - thanks @ammachado!
2015-05-13 10:56:47 -04:00
Jared Broad 921e166023 Added custom event handler for notification message types calling the virtual base Send method 2015-02-08 20:14:01 -03:00
Jared Broad 1f01abbe90 Removed type from base notification class to simplify notification object implementations and allow for future extensibility 2015-02-08 20:08:40 -03:00
Jared Broad d7b590f4c1 Closes #22, New notification manager implementation with notification classes, API and manager for rate limiting 2015-02-07 18:11:02 -03:00