Problem/Motivation

Commerce POS contains 2 payment screens:

  • a customer-facing display
  • a cashier interface,

both will be updated to display accepted credit cards.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#4 download (4).jpeg53.59 KBskyredwang
#4 download (3).jpeg56.66 KBskyredwang
#4 download (2).jpeg110.68 KBskyredwang

Issue fork commerce_pos-3405313

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

alexpott created an issue. See original summary.

alexpott opened merge request !5

skyredwang made their first commit to this issue’s fork.

skyredwang’s picture

skyredwang
Location Shanghai
commented
Status: Active » Needs review
StatusFileSize
new download (2).jpeg110.68 KB
new download (3).jpeg56.66 KB
new download (4).jpeg53.59 KB

Please see the screenshots.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Needs review » Reviewed & tested by the community

I've tested this locally and added automated test coverage. It's looking great.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Needs work

We need to enable all the logos by default and we probably should make the logos a bit smaller so they all fit on one line.

alexpott credited cyoung01.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Needs work » Reviewed & tested by the community

This has been tested by @cyoung01 for Acro Commerce

  • alexpott committed 3490f07f on 3.0.x 
    Issue #3405313 by alexpott, skyredwang, cyoung01: Add credit card logos...
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Category: Task » Feature request
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

alexpott opened merge request !8

  • alexpott committed 7d8d36a8 on 3.0.x 
    Issue #3405313 by alexpott, skyredwang, cyoung01: Add credit card logos...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.