Skip to content

Conversation

@abhinavk96
Copy link
Contributor

No description provided.

iamareebjamal and others added 13 commits July 30, 2019 14:20
@auto-label auto-label bot added the chore label Jul 31, 2019

stripe_auth_code = fields.Str(load_only=True, required=True)


Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

blank line at end of file

@abhinavk96 abhinavk96 requested a review from iamareebjamal July 31, 2019 18:47
@iamareebjamal iamareebjamal merged commit d78bc1b into master Jul 31, 2019
@codecov
Copy link

codecov bot commented Jul 31, 2019

Codecov Report

Merging #6278 into master will decrease coverage by 0.3%.
The diff coverage is 23.72%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6278      +/-   ##
==========================================
- Coverage   65.84%   65.54%   -0.31%     
==========================================
  Files         286      286              
  Lines       14501    14588      +87     
==========================================
+ Hits         9548     9561      +13     
- Misses       4953     5027      +74
Impacted Files Coverage Δ
app/api/feedbacks.py 51.28% <ø> (ø) ⬆️
app/api/custom_forms.py 70.83% <ø> (ø) ⬆️
app/api/access_codes.py 51.8% <ø> (ø) ⬆️
app/api/custom_system_roles.py 67.74% <ø> (ø) ⬆️
app/api/helpers/notification.py 21.73% <ø> (ø) ⬆️
app/api/schema/users.py 100% <ø> (ø) ⬆️
app/api/events.py 22.16% <1.92%> (-2.98%) ⬇️
app/models/event.py 79.19% <100%> (+0.15%) ⬆️
app/api/schema/stripe_authorization.py 100% <100%> (ø) ⬆️
app/api/__init__.py 100% <100%> (ø) ⬆️
... and 3 more

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 5163c4c...1ef10f5. Read the comment docs.

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.

6 participants