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.

Sign up for GitHub

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

Jump to bottom

Chance to elaborate currencies with not just precision 2 #12

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

Chance to elaborate currencies with not just precision 2 #12

wants to merge 4 commits into from

Commits on Nov 1, 2020

  1. I hope to help,this is my 1st time I try to help a github project and… 

    … I am not mainly a developer

Can be a way to fix the currencies that doesn't have the 2 decimals.
Please check at tests too in case i modified em wrongly
    @Rixamos
    Rixamos committed Nov 1, 2020
    Configuration menu
    Copy the full SHA
    6e9d692 View commit details
    Browse the repository at this point in the history

  2. Hopefully now should work correctly

    @Rixamos
    Rixamos committed Nov 1, 2020
    Configuration menu
    Copy the full SHA
    81ed1e2 View commit details
    Browse the repository at this point in the history

  3. Put an awful patch to the Charge but I suppose has to be redone properly 

    In add I suppose will have other implications I didn t discover yet
    @Rixamos
    Rixamos committed Nov 1, 2020
    Configuration menu
    Copy the full SHA
    6f42d96 View commit details
    Browse the repository at this point in the history

Commits on Nov 29, 2020

  1. fix: Fixing refund-support error. 

    - Parameter chargeId is mandatory.

- Refund model has some missing fields.
    @FcoLopera
    FcoLopera committed Nov 29, 2020
    Configuration menu
    Copy the full SHA
    68a71c2 View commit details
    Browse the repository at this point in the history