Skip to content

Conversation

@uds5501
Copy link
Contributor

@uds5501 uds5501 commented Jun 5, 2019

Fixes #6003

Checklist

There was a typo in settings schema regarding omise activation

@auto-label auto-label bot added the fix label Jun 5, 2019
@uds5501
Copy link
Contributor Author

uds5501 commented Jun 5, 2019

@iamareebjamal Please review and close

mrsaicharan1
mrsaicharan1 previously approved these changes Jun 5, 2019
@uds5501
Copy link
Contributor Author

uds5501 commented Jun 5, 2019

Just a sec, let me add one more check which was missed here.

@uds5501 uds5501 changed the title fix: fix minor typo fix: fix minor typo and adding omise to payment-mode schema Jun 5, 2019
@codecov
Copy link

codecov bot commented Jun 5, 2019

Codecov Report

Merging #6002 into development will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##           development    #6002   +/-   ##
============================================
  Coverage        66.28%   66.28%           
============================================
  Files              285      285           
  Lines            13995    13995           
============================================
  Hits              9277     9277           
  Misses            4718     4718
Impacted Files Coverage Δ
app/api/schema/orders.py 88.88% <ø> (ø) ⬆️
app/api/schema/settings.py 100% <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 ab65174...7d230f9. Read the comment docs.

@uds5501
Copy link
Contributor Author

uds5501 commented Jun 5, 2019

@mrsaicharan1 @shreyanshdwivedi @iamareebjamal please review

@iamareebjamal iamareebjamal changed the title fix: fix minor typo and adding omise to payment-mode schema fix: minor typo and add omise to payment-mode schema Jun 5, 2019
@uds5501 uds5501 force-pushed the typo-omise-schema branch from 9a6b2db to 7d230f9 Compare June 5, 2019 17:40
@iamareebjamal iamareebjamal merged commit 3b66321 into fossasia:development Jun 6, 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.

minor typo in settings schema and add omise payment to order schema

6 participants