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

Fix deprecated string interpolation format #342

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

oxyc
Copy link
Contributor

@oxyc oxyc commented Jul 25, 2023

No description provided.

MichaelBengtsson and others added 5 commits June 20, 2023 15:36
* Test API credentials depending on the environment (krokedil#339)

Currently, we always test both production and test credentials regardless of whether the merchant has enabled test mode or not.

- Only test productions credentials if test mode is disabled.
- Only test test credential if test mode is enabled.
- Changed the error message when a test result in an HTTP 401 error.

* Remove the settings tab on the "Klarna Add-ons" page (krokedil#338)

* Fix "What is Klarna" not appearing (krokedil#334)

- "What is Klarna" should now appear as expected, and only when enabled in the settings.
- The icon should now be retrieved from the session as expected. Currently, due to a typo in the key name, the default icon was always used.
- Changed the settings text to make it clearer.

* Version 3.1.1
@MichaelBengtsson MichaelBengtsson changed the base branch from master to develop August 30, 2023 13:29
Copy link
Contributor

@MichaelBengtsson MichaelBengtsson left a comment

Choose a reason for hiding this comment

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

Looks good! Thank you @oxyc for the update

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.

None yet

3 participants