Skip to content

Conversation

@mrsaicharan1
Copy link
Member

@mrsaicharan1 mrsaicharan1 commented Aug 9, 2019

Fixes #6326

Short description of what this resolves:

  • Sends notifications to co-organizers as well

Checklist

  • I have read the Contribution & Best practices Guide and my PR follows them.
  • My branch is up-to-date with the Upstream development branch.
  • The unit tests pass locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • I have added necessary documentation (if appropriate)
  • All the functions created/modified in this PR contain relevant docstrings.

@auto-label auto-label bot added the feature label Aug 9, 2019
@mrsaicharan1 mrsaicharan1 changed the title feat: send notification to coorganizers as well feat: send notification to coorganizers - ticket sales Aug 9, 2019
@codecov
Copy link

codecov bot commented Aug 9, 2019

Codecov Report

Merging #6335 into development will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development    #6335      +/-   ##
===============================================
- Coverage        65.25%   65.24%   -0.01%     
===============================================
  Files              287      287              
  Lines            14729    14731       +2     
===============================================
  Hits              9611     9611              
- Misses            5118     5120       +2
Impacted Files Coverage Δ
app/api/orders.py 28.16% <0%> (-0.18%) ⬇️

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 2edccda...5687f66. Read the comment docs.

@iamareebjamal iamareebjamal merged commit 3f043f6 into fossasia:development Aug 10, 2019
@mrsaicharan1 mrsaicharan1 deleted the notif-others branch August 10, 2019 11:36
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.

Send notification to all organizers/co-organizers for ticket sales

5 participants