Skip to content

Conversation

@abhinavk96
Copy link
Contributor

Fixes #6090

Instead of the email, the actual user object needs to be passed for notification helper to work.

@auto-label auto-label bot added the fix label Jun 22, 2019
@codecov
Copy link

codecov bot commented Jun 22, 2019

Codecov Report

Merging #6096 into development will increase coverage by 0.02%.
The diff coverage is 0%.

Impacted file tree graph

@@              Coverage Diff               @@
##           development   #6096      +/-   ##
==============================================
+ Coverage        66.17%   66.2%   +0.02%     
==============================================
  Files              285     285              
  Lines            14133   14133              
==============================================
+ Hits              9353    9357       +4     
+ Misses            4780    4776       -4
Impacted Files Coverage Δ
app/api/helpers/scheduled_jobs.py 26.59% <0%> (+4.25%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6d69104...6110b09. Read the comment docs.

@iamareebjamal iamareebjamal merged commit 1400605 into fossasia:development Jun 22, 2019
iamareebjamal pushed a commit to iamareebjamal/open-event-server that referenced this pull request Aug 2, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

After event mail cron job fails

2 participants