Midtrans / midtrans-nodejs-client

Official Midtrans Payment API Client for Node JS | https://midtrans.com
MIT License
181 stars 57 forks source link

Ambiguous transaction status & layout problem #2

Closed heryvandoro closed 5 years ago

heryvandoro commented 5 years ago

Sorry if this issue not belongs this repo but I don't see any special repository for snap products.

"expired" status is ambiguous on a transaction detail/status and when receiving a callback. On the transaction status, I get a json response with "expire" word. But at the time of the callback, the status is "expired" (there is letter d). Is this really intentional or is there an error? It is recommended that this be generalized to prevent future errors.

Then, I want to report a layout problem with snap for the customer section that should be corrected because there was a not used bootstrap column (col-md-6). It should be removed just so that the customer email can be sideways.

image

Thanks!

rizdaprasetya commented 5 years ago

Hi @heryvandoro

Thanks for the feedback. If you have any other questions, for faster response please contact us via email: support@midtrans.com. Github issues might not be best place to ask questions about Midtrans product.

1) About expired keyword, can you let me know where do you find expired? Because both our notification/callback is sending expire, also our /status API is responding with expire status.

2) Layout on Snap customer details page are supposed to be consist of 2 column, left column is for customer name & email, the right column is for address (if exists). screen shot 2018-12-28 at 2 38 59 pm

In your case there is no address, so right column is there but empty, which result in email seems like not properly displayed. But yes the feedback is valid, I will pass it to our Product team to take a look and see if it can be improved.

Thanks.

heryvandoro commented 5 years ago

Ok noted, I will inform you about expired status to support mail.

One more request, can you develop SNAP sdk for react-native? I think this platform will be very widely used. Currently, we trick it with opening snap redirect_url into a webview and inject postMessage script at the bottom of code like what snap.js do in <iframe>.

Thanks in advance.

rizdaprasetya commented 5 years ago

Mobile based integration SDK is handled by our Mobile SDK team, they are still observing the market demand on react-native, last time they observe the demand is not big enough so they are still prioritizing the native mobile SDK development. Will let them know about this requests.

For now the official SDK that is available for Mobile is native mobile SDK, if you want to try out.

Or using your method of using webview to open redirect_url.

Thanks