search

magmodules / magento2-sooqr

Sooqr Connect for Magento® 2
https://www.magmodules.eu/sooqr-connect.html
Other
2 stars 3 forks source link

Version number in configuration not being updated #49

Closed Aquive closed 2 years ago

Aquive commented 2 years ago

Describe the bug I just updated the Sooqr module to the latest version (v1.5.0) on my local machine. But in the configuration it still lists the old version number (v1.1.9).

Used versions

To Reproduce Install new version composer require magmodules/magento2-sooqr:1.5.0

Expected behavior New version being reflected in the config.

Actual behavior Older version being shown in the config.

Additional context This is also the case when running a deploy to our test server. Tested in dev and production modes.

Marvin-Magmodules commented 2 years ago

Hi Aquive, thank you for opening this issue! I'm not able to reproduce this issue, after updated the module to the latest version the admin shows the correct version nummer. This version number is pulled from: https://github.com/magmodules/magento2-sooqr/blob/master/etc/config.xml#L14

Can you double check if this has the correct version number? I'm 99% sure you already did, but can you also try and do a full cache flush (magento/redis) to see if this changes anything?

Aquive commented 2 years ago

@Marvin-Magmodules

Thank you for your reply.

The thing is, I think the extension is actually updated since I see new options became available. 

Add to cart Controller
Use Ajax based Add to Cart

Those are new options, right?

Did you test your upgrade from v1.1.9 to v1.5.0?

Marvin-Magmodules commented 2 years ago

@Aquive, also tested this using a clean installation from v1.1.9 and update to v1.5.0 but still unable to reproduce this issue. Do you have a dev env available with this issue where we can debug this? If possible, please contact us directly through our contact form so we can exchange some logins.

Aquive commented 2 years ago

It seems it is coming from app/etc/config.php (after a config:dump). I had checked it since I assumed it was the source. But I guess I overlooked it. Solved. Thanks for the support.