Closed shreyagupta30 closed 4 years ago
Merging #17 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #17 +/- ##
====================================
Coverage 0% 0%
====================================
Files 3 3
Lines 214 214
====================================
Misses 214 214
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 69b4a2d...5221245. Read the comment docs.
@shreyagupta30 Look into the pep8 issues listed above.
Add a readme file in same folder explaining its usage.
@satyammittal okay, sir. On it. :+1:
Also Steps to run it and where to get and setup the keys.
@shreyagupta30 Can we get integrated in our main chatbot code as well.
@satyammittal Sir, I searched for it a lot but didn't get much about it. The SDK available is just for a single POST request to create expense is available. Please give me 2 more days. I'll try doing it again and will get back to you.
@shreyagupta30 I am merging this. let me know if you get the new approach.
Hello @shreyagupta30! Thanks for updating this PR.
There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:
Comment last updated at 2019-07-18 08:29:14 UTC