Enable passing of more Pushover API parameters. #570
Conversation
There was a problem hiding this comment.
Dear @jlrgraham,
thanks a stack for your contribution. I think it will be good to go and I am looking forward to cut another release after merging.
On the feedback side, I spotted a minor typo where I am humbly asking you for correction.
With kind regards,
Andreas.
Codecov Report
@@ Coverage Diff @@
## main #570 +/- ##
==========================================
+ Coverage 29.19% 29.25% +0.05%
==========================================
Files 82 82
Lines 3665 3668 +3
Branches 427 429 +2
==========================================
+ Hits 1070 1073 +3
Misses 2546 2546
Partials 49 49
Flags with carried forward coverage won't be shown. Click here to find out more.
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
|
Dear Justin, before merging, I've just added a corresponding test case to cover the added code. Thank you again. With kind regards, |
|
@amotl, fantastic, many thanks. I'll try to make sure that I get appropriate tests and coverage for future submissions. Best, -Justin |
Allow users to specify more of the parameters allowed by the Pushover API. In particular, add support for the embedded URL and HTML formatting options.