Skip to content

Add notifications for overdue cards and board sharing - #309

Merged
juliusknorr merged 12 commits into
masterfrom
notifications-duedate
Oct 3, 2017
Merged

juliusknorr merged 12 commits into
masterfrom
notifications-duedate

Conversation

@juliusknorr

@juliusknorr juliusknorr commented Sep 28, 2017 •

Copy link
Copy Markdown
Member

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

codecov Bot commented Sep 28, 2017 •

Copy link
Copy Markdown

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

@juliusknorr juliusknorr mentioned this pull request Oct 1, 2017
4 tasks done
@juliusknorr
juliusknorr force-pushed the notifications-duedate branch from e387778 to aadc4fb Compare October 2, 2017 10:29
@juliusknorr
juliusknorr force-pushed the notifications-duedate branch from aadc4fb to 8b210d4 Compare October 3, 2017 16:20
Signed-off-by: Julius H盲rtl <[email protected]>
@juliusknorr
juliusknorr force-pushed the notifications-duedate branch from 2305bd7 to afae40e Compare October 3, 2017 17:12
@juliusknorr
juliusknorr force-pushed the notifications-duedate branch from eeb7e50 to 2a23987 Compare October 3, 2017 20:12
@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
2 of 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