Skip to content
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.

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

Add notifications for overdue cards and board sharing #309

Merged
merged 12 commits into from
Oct 3, 2017

Conversation

juliusknorr
Copy link
Member

@juliusknorr juliusknorr commented Sep 28, 2017

Send users a notification if a board is shared with them and if a card on one of their boards is overdue.

  • Use either a proper type to highlight board/card in notifications or just show the text
  • Remove pending notifications when duedate is updated

fix for #18

@codecov
Copy link

codecov bot commented Sep 28, 2017

Codecov Report

Merging #309 into master will decrease coverage by 0.42%.
The diff coverage is 74.05%.

@@            Coverage Diff             @@
##           master     #309      +/-   ##
==========================================
- Coverage   76.86%   76.43%   -0.43%     
==========================================
  Files          33       36       +3     
  Lines         981     1133     +152     
==========================================
+ Hits          754      866     +112     
- Misses        227      267      +40

Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliusknorr juliusknorr merged commit 7c6e48d into master Oct 3, 2017
@juliusknorr juliusknorr deleted the notifications-duedate branch October 3, 2017 21:00
@juliusknorr juliusknorr mentioned this pull request Oct 16, 2017
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant