feat: Adds transaction initiation logic & Payments Manager - PayTM #6342

mrsaicharan1 · 2019-08-10T13:44:34Z

Fixes #6341
Response Image

Short description of what this resolves:

This PR initiates the transaction to obtain the txn token to process the payments. This txn token will be utilized for further processing and confirmation of the transaction in the later stages.

Changes proposed in this pull request:

Adds PayTM payments manager Class
Adds a route which initiates the transaction by hitting the transaction API with the generated checksum
Adds checksum module from PayTM's official repo.

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.

app/api/orders.py

houndci-bot · 2019-08-10T13:44:40Z

app/api/orders.py

+    order = safe_query(db, Order, 'identifier', order_identifier, 'identifier')
+    paytmParams = dict()
+    # body parameters
+    paytm_params["body"] = {


undefined name 'paytm_params'

app/api/orders.py

houndci-bot · 2019-08-10T13:47:47Z

app/api/orders.py

+        "signature"	: checksum
+    }
+
+    post_data = json.dumps(paytm_params)


undefined name 'json'
undefined name 'paytm_params'

app/api/orders.py

app/api/helpers/payment.py

app/api/orders.py

app/api/helpers/checksum.py

app/api/orders.py

app/api/helpers/checksum.py

codecov · 2019-08-12T09:23:09Z

Codecov Report

Merging #6342 into development will decrease coverage by 0.26%.
The diff coverage is 27.45%.

@@              Coverage Diff               @@
##           development   #6342      +/-   ##
==============================================
- Coverage        65.16%   64.9%   -0.27%     
==============================================
  Files              287     288       +1     
  Lines            14810   14911     +101     
==============================================
+ Hits              9651    9678      +27     
- Misses            5159    5233      +74

Impacted Files	Coverage Δ
app/api/schema/orders.py	`89.06% <ø> (ø)`	⬆️
app/api/helpers/checksum.py	`24.67% <24.67%> (ø)`
app/api/orders.py	`27.77% <29.41%> (-0.14%)`	⬇️
app/api/helpers/payment.py	`25.17% <50%> (+1.47%)`	⬆️

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 fcf5275...7a0c904. Read the comment docs.

app/api/helpers/checksum.py

app/api/orders.py

shreyanshdwivedi

@mrsaicharan1 add paytm as a valid payment_mode here -

open-event-server/app/api/schema/orders.py

Line 64 in af9e8c7

"cheque", "onsite", "omise", "alipay"]),

feat: added checksum file from paytm Added condition for sandbox & live added paytm in schema choices

mrsaicharan1 · 2019-08-20T03:31:41Z

@iamareebjamal Please review.

auto-label bot added the feature label Aug 10, 2019

houndci-bot reviewed Aug 10, 2019

View reviewed changes

mrsaicharan1 changed the title ~~feat: Adds transaction initiation logic & Payments Manager - PayTM~~ [WIP]feat: Adds transaction initiation logic & Payments Manager - PayTM Aug 10, 2019

auto-label bot removed the feature label Aug 10, 2019

mrsaicharan1 force-pushed the paytm-workflow branch from 00e0a21 to ecac5e5 Compare August 10, 2019 13:47

houndci-bot reviewed Aug 10, 2019

View reviewed changes

mrsaicharan1 force-pushed the paytm-workflow branch from ecac5e5 to 327055a Compare August 10, 2019 13:52

houndci-bot reviewed Aug 10, 2019

View reviewed changes

app/api/orders.py Outdated Show resolved Hide resolved

app/api/orders.py Outdated Show resolved Hide resolved

mrsaicharan1 force-pushed the paytm-workflow branch from 327055a to d0cce67 Compare August 10, 2019 13:57