Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Merchant Cannot Do Integration-401] Add section how switch to production mode #87

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

Xaxxis
Copy link

@Xaxxis Xaxxis commented Jul 15, 2022

  • Add how to check the active payment method on page switch to production mode.
  • Put a link on how to switch production mode in every integration step (Snap & CoreAPI)

@rizdaprasetya
Copy link
Collaborator

rizdaprasetya commented Jul 18, 2022

haven't review, but quick comment for now @Xaxxis :

  • image files seems too big, have you follow the repo's Readme to compress/optimize image files?

Copy link
Collaborator

@rizdaprasetya rizdaprasetya left a comment

Choose a reason for hiding this comment

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

Overall ideas/plan of the improvement is good, but the execution details like grammar & wording could be improved.

1. Change API domain URL from `api.sandbox.midtrans.com` to `api.midtrans.com`.
2. Use *Client Key* and *Server Key* for *Production* environment. For more details, refer to [Retrieving API Access Keys](/en/midtrans-account/overview.md#retrieving-api-access-keys).
### 5. Switching to Production Mode
In this section, you will learn how switching from Sandbox environment to Production environment, in order to start accepting real world payment for your business. This is also known as "Going Live".
Copy link
Collaborator

Choose a reason for hiding this comment

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

how switching from Sandbox environment

bad grammar. Have you use any tool to check grammar? use tool, at least jsut paste to a google docs, it will auto detect grammar mistakes.

OR just use the original wording:

to switch to Midtrans Production environment and to accept real payments from real customers.

## Switching to Production Environment
Follow the steps given below to switch to Midtrans *Production* environment and to accept real payments from real customers.
1. Change API domain URL from `api.sandbox.midtrans.com` to `api.midtrans.com`.
2. Use *Client Key* and *Server Key* for *Production* environment. For more details, refer to [Retrieving API Access Keys](/en/midtrans-account/overview.md#retrieving-api-access-keys).
### 5. Switching to Production Mode
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why change heading structure from ## to ### 5. ? IMO it shouldn't be part of integration step's sub heading, as it would seems that the integration steps is now longer.

Copy link
Author

Choose a reason for hiding this comment

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

Thanks for the feedback Da. I get the details from Yana, about the merchant that facing 401. They claim they weren't found the information in the integration steps and that they should change the environment to production before going live. That's why I put the section going live, into integration steps. But I will let you decide what the best to choose ya.

Thank you

Copy link
Collaborator

Choose a reason for hiding this comment

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

hmm I think make sense, please proceed with your approach for now 👌 we can re-evaluate sometimes later if we need to.

<div class="my-card">

#### [Going Live](/en/technical-reference/going-live.md)
Best practices to build a production-ready
Copy link
Collaborator

Choose a reason for hiding this comment

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

to build a production-ready

hmm the wording doesn't quite make sense. try something else

Copy link
Author

Choose a reason for hiding this comment

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

How about

Steps to take before going live

Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's use the page description as reference:

... switching from Sandbox environment to Production environment, in order to start accepting real world payment

Simplified to:

Learn how to switch from Sandbox to Production environment, to start accepting real payment


## 1. Login to Your Midtrans Production Environment Dashboard
## 1. Checking Your Production Account
To make sure your payment channels on Production account is already active, follow the steps given below.
Copy link
Collaborator

Choose a reason for hiding this comment

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

payment channels

Is an internal wording. to talk with external audience we use Payment Methods. The term has been used all over the docs too, so you should just need to be consistent with the term.

Copy link
Collaborator

Choose a reason for hiding this comment

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

better wording:

To make sure that your Midtrans account already has activated payment method(s) on Production Environment mode, follow the steps given below.

Comment on lines 6 to 7
?>Make sure you have already completed the test payment using our [Payment Simulator](/en/technical-reference/sandbox-test.md) in the Sandbox environment to ensure that your implementation is working fine.

Copy link
Collaborator

Choose a reason for hiding this comment

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

better:

Make sure you have already performed some successful payment testing in the Sandbox environment, to ensure that your implementation is working properly.

Comment on lines 14 to 15
- Go to menu **PAYMENT METHODS** from the dashboard page
- *Active Payment Methods* page is displayed.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is all merchants confirmed to have access to this page? AFAIK based on pilar issues, some manually onboarded merchant etc was unable to access this page.

  • maybe need confirmation to CMP?
  • or maybe add notes to skip this step if they are unable to see the page?
  • or maybe ask them to check snap payment method list?

Comment on lines 28 to 29
1. Login to your MAP account.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I personally prefer to avoid using MAP terms as not all merchants maybe familiar with the terms, and would use your Midtrans Account as it is more straight forward to understand.

6. Enter **Server key** with the *Server Key* retrieved from [Step 3 above](#_3-retrieving-server-key-and-client-key).
7. Enter **Client key** with the *Client Key* retrieved from [Step 3 above](#_3-retrieving-server-key-and-client-key).
6. Enter **Server key** with the *Server Key* retrieved from [Step 2 above](#_2-retrieving-production-server-key-and-client-key).
7. Enter **Client key** with the *Client Key* retrieved from [Step 2 above](#_2-retrieving-production-server-key-and-client-key).
8. Click **Save**.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Also please replace contact email on bottom of this page with Contact Us web page URL

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants