magento-lts icon indicating copy to clipboard operation
magento-lts copied to clipboard

Fixed: Fixer.io endpoint for currency updates

Open jankovandeventer opened this issue 9 months ago • 2 comments

I tested it, and it works, in our own project I used an overwrite to make sure we can keep collecting data.

Description (*)

Recently there were some changes, this is the new endpoint if anyone is also using it 💸 The platform had a few changes over the years, but I think recently an endpoint was deprecated, resulting in no more updating our currency information.

The format is the same, just the url is changed and apiKey was renamed to access_key in the params.

Related Pull Requests

I believe this solves: https://github.com/OpenMage/magento-lts/issues/3071

Fixed Issues (if relevant)

I believe this solves: https://github.com/OpenMage/magento-lts/issues/3071

Manual testing scenarios (*)

  1. set up a free account on fixer.io
  2. add 2 or more currencies
  3. go to System -> Manage Currencies -> Rates
  4. Press import
  5. Success

image

Questions or comments

First PR for Open Mage, keep it up!

jankovandeventer avatar Apr 25 '24 12:04 jankovandeventer

if I open https://api.apilayer.com/fixer/latest on the browser it seems to work, is it not working at all for you?

is there any official info about the change?

fballiano avatar Apr 25 '24 16:04 fballiano

https://apilayer.com/marketplace/fixer-api

image

hirale avatar Apr 26 '24 19:04 hirale

@hirale what do you mean? that the currently used address is working fine?

fballiano avatar Apr 29 '24 16:04 fballiano

Reads to me that apilayer is just a wrapper around the fixer api. So no change should be required.

https://apilayer.com/marketplace/fixer-api

pquerner avatar Apr 29 '24 16:04 pquerner

@jankovandeventer why did you need it? did the flexer api not work for you?

fballiano avatar Apr 29 '24 18:04 fballiano

@hirale what do you mean? that the currently used address is working fine?

the current one works fine.

hirale avatar Apr 29 '24 18:04 hirale

let's see if the author of the PR explains us

fballiano avatar Apr 29 '24 18:04 fballiano

For the time being I'll close this PR, @jankovandeventer feel free to reopen if you're ve the answers to the questions asked here :-)

fballiano avatar Apr 30 '24 09:04 fballiano

Hi there, sorry for not responding sooner. When I use the credentials with the apilayer, I get an invalid authentication warning.

image image

But I realize my mistake, the fixer.io credentials do not work on the apilayer so I had to move the subscription over to their new platform and now it works.

image

Sorry to bother :-)

jankovandeventer avatar May 03 '24 13:05 jankovandeventer