search

VOKO-Utrecht / voko

Administration and automation for VOKO
4 stars 8 forks source link

Mollie upgrade #178

Closed mgvd closed 2 years ago

mgvd commented 2 years ago

Major upgrade van mollie-api-python naar 2.12.0 Intern is de structuur van de library aangepast, maar ook methodes zoals setApiKey of isPaid zijn hernoemd naar set_api_key en is_paid

Structuur van betaalmethodes return value is veranderd, iDeal issuers zitten er in, dus maar 1 call nodig.

Aanpassing in create payment, nl 'amount' is nu een dict met currency + bedrag

Testen zijn bijgewerkt, Lokaal volledige roundtrip getest met mollie test key

rikva commented 2 years ago

Hmm, there seems to be a problem:

File "./vokou/urls.py", line 7, in <module>
    import finance.urls
  File "./finance/urls.py", line 3, in <module>
    from finance import views, admin_views
  File "./finance/views.py", line 1, in <module>
    from mollie.api.client import Client as MollieClient
  File "/home/voko/.virtualenvs/voko/lib/python3.5/site-packages/mollie/api/client.py", line 64
    raise RequestSetupError(f"Invalid API key: '{api_key}'. An API key must start with 'test_' or 'live_'.")

Which is weird, I confirmed that the production API key is still configured and it does start with live_.

rikva commented 2 years ago

Ah, I have a suspicion that the production Python version is outdated so it does not support f-strings. Sigh!

rikva commented 2 years ago

I need to upgrade the Python environment, this will take some time. Production is running the previous version for now.

rikva commented 2 years ago

All right, we are running Python 3.6 now and all seems to be working.