- Issue created by @Andrew.Macpherson
- First commit to issue fork.
- Status changed to Needs review
11 months ago 5:32am 2 January 2024 - last update
11 months ago Composer require failure - 🇮🇳India viren18febS
I have update the .json file & added patch, please review.
- First commit to issue fork.
- last update
11 months ago Composer require failure - 🇺🇸United States pbabin
If you have an existing installation of this package with Address 1 just updating to 2.0 prevents the ability to update from Address 1 to Address 2. Slightly modified patch.
- Status changed to RTBC
10 months ago 1:43pm 16 January 2024 - 🇩🇪Germany Anybody Porta Westfalica
RTBC for #5 - setting priority to major, as this blocks upgrades to the recommended address module version.
Dear maintainer, may I ask you humbly to please fix the address 2.0 compatibility issue soon. Uninstalling your module before upgrading Commerce with its requirement of Address 2.0 did not work out cleanly due to the outdated requirements for Address, and for me it has been impossible to remove the address_map_link module completely with the half baked Commerce and Address upgrade that I performed. I was confident that your fix would follow suit with the address 2.0 upgrade but have meanwhile spent a couple hours already trying to fix things. Rolling back everything also failed (my bad, I know), and now the database and Extensions files again report errors related to the missing address_map_link module. Deleting references in the DB only helped partially, and I am sure your upgrade would be the best solution, especially as I like the modules functionality and would like to rely on it again. Your help with this would be highly appreciated. Thank you!
- First commit to issue fork.
- last update
9 months ago Composer require failure - last update
9 months ago Composer require failure -
ChrisSnyder →
committed b2af6f4a on 8.x-1.x
Issue #3411810 by viren18febS, ChrisSnyder, joelsteidl, pbabin, Andrew....
-
ChrisSnyder →
committed b2af6f4a on 8.x-1.x
- Status changed to Fixed
9 months ago 3:06pm 20 February 2024 - 🇺🇸United States chrissnyder Maryland
Merged and new release tagged. Sorry for the delay on this.
Thank you very much, Chris, for your quick reaction with a perfect fix that installed withou a glitch, working as intended on D10.2.3 with PHP 8.3. Best regards, Andreas
- 🇪🇸Spain uridrupal
Until the patch is merged it won't work because if I'm correct composer checks dependencies at the time of install. So even with the patch you won't comply with the requirements. Another possible solution would be to fork the repo and create your own with the patch applied until the issue is solved. Something like that could work.
- 🇺🇸United States chrissnyder Maryland
@UriDrupal The patch/changes were merged, and a new release was deployed.
Update to the latest version of the module with the change:
composer require drupal/address_map_link:^1.4 Automatically closed - issue fixed for 2 weeks with no activity.