New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

#160126500 Include message payload in push notifications #43

Merged

olusoladavid merged 1 commit into develop from ch-refactor-reminder-160126500

Aug 29, 2018

Owner

olusoladavid commented Aug 29, 2018

What does this PR do?

Adds a specific message payload to the reminder push notification to improve flexibility of future notifications

How should this be manually tested?

Pull branch into local machine
Run npm test

What are the relevant pivotal tracker stories?


          chore(Notification): Include message payload in push notifications

39c044e

[Delivers #160126500]

coveralls commented Aug 29, 2018

Pull Request Test Coverage Report for Build 144

2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals
Change from base Build 142:	0.0%
Covered Lines:	182
Relevant Lines:	182

💛 - Coveralls

olusoladavid merged commit a306c19 into develop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment