Skip to content

Conversation

@abhinavk96
Copy link
Contributor

Fixes #6268

Short description of what this resolves:

Adds a billing info mandatory field to the event model

@auto-label auto-label bot added the feature label Jul 30, 2019
@abhinavk96 abhinavk96 requested a review from iamareebjamal July 30, 2019 20:53
@abhinavk96
Copy link
Contributor Author

@iamareebjamal Fixed, please review again. Thanks!

@codecov
Copy link

codecov bot commented Jul 30, 2019

Codecov Report

❗ No coverage uploaded for pull request base (development@f2773e5). Click here to learn what that means.
The diff coverage is 100%.

Impacted file tree graph

@@              Coverage Diff               @@
##             development    #6269   +/-   ##
==============================================
  Coverage               ?   65.78%           
==============================================
  Files                  ?      286           
  Lines                  ?    14519           
  Branches               ?        0           
==============================================
  Hits                   ?     9552           
  Misses                 ?     4967           
  Partials               ?        0
Impacted Files Coverage Δ
app/api/schema/events.py 91.79% <100%> (ø)
app/models/event.py 79.19% <100%> (ø)

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 f2773e5...6c5bafd. Read the comment docs.

@abhinavk96 abhinavk96 merged commit e1a9061 into fossasia:development Jul 31, 2019
mrsaicharan1 pushed a commit to mrsaicharan1/open-event-server that referenced this pull request Aug 9, 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.

Give event organizer the ability to decide if billing info is mandatory

3 participants