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

provide fpx logo in checkout screen #318

Merged
merged 3 commits into from
Dec 2, 2021
Merged

provide fpx logo in checkout screen #318

merged 3 commits into from
Dec 2, 2021

Conversation

nimitomise
Copy link
Contributor

@nimitomise nimitomise commented Dec 1, 2021

  • Add FPX logo to checkout screen due to FPX CI
  • Add FPX term and condition to checkout screen

1. Objective

Regarding to FPX CI so FPX request Omise to display their payment method under the checkout screen with FPX logo and also display a link to their term and conditions page.

Related information:

https://omise.atlassian.net/browse/APM2-206

2. Description of change

Screen Shot 2564-12-01 at 17 11 09

3. Quality assurance

FPX logo along with term and condition link displayed on checkout screen for Malaysia merchant whose enable FPX payment method.

🔧 Environments:

Platform version: Magento CE 2.2.3.
Omise plugin version: Omise-Magento 2.2.0
PHP version: 7.0.16.

✏️ Details:

4. Impact of the change

None

5. Priority of change

High

@@ -71,6 +65,12 @@
</fieldset>
</form>
<div class="actions-toolbar">
<div class="checkout-agreements-block">
<span>clicking on tht "Place Order" button, you agree to FPX's</span>
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
<span>clicking on tht "Place Order" button, you agree to FPX's</span>
<span>By clicking on the "Place Order" button, you agree to FPX's</span>

@nimitomise nimitomise merged commit 089bd7c into master Dec 2, 2021
@nimitomise nimitomise mentioned this pull request Dec 3, 2021
@aashishgurung aashishgurung deleted the APM2-206-fpx-logo branch September 27, 2022 05:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants