search

surprisehighway / craft-avatax

Calculate and add sales tax to an order's base tax using Avalara's AvaTax service with Craft Commerce 5.
MIT License
7 stars 11 forks source link

Guzzle Error #44

Closed chadcrowell closed 2 years ago

chadcrowell commented 2 years ago

I am trying to install an update to Enupal Translated and getting this error on composer update:

 Problem 1
    - Root composer.json requires enupal/translate 2.3.0 -> satisfiable by enupal/translate[2.3.0].
    - Conclusion: don't install guzzlehttp/guzzle 7.4.0 (conflict analysis result)
    - Conclusion: don't install guzzlehttp/guzzle 7.4.2 (conflict analysis result)
    - enupal/translate 2.3.0 requires viniciusgava/google-translate-api 3.1.0 -> satisfiable by viniciusgava/google-translate-api[3.1.0].
    - Root composer.json requires surprisehighway/craft-avatax 2.1.8 -> satisfiable by surprisehighway/craft-avatax[2.1.8].
    - guzzlehttp/guzzle 7.4.x-dev is an alias of guzzlehttp/guzzle dev-master and thus requires it to be installed too.
    - viniciusgava/google-translate-api 3.1.0 requires guzzlehttp/guzzle ^7.0.1 -> satisfiable by guzzlehttp/guzzle[7.0.1, ..., 7.4.x-dev (alias of dev-master)].
    - You can only install one version of a package, so only one of these can be installed: guzzlehttp/guzzle[dev-master, v1.0.3, v1.0.4, v2.0.0, ..., v2.8.8, v3.0.0, ..., v3.8.1, 4.0.0-rc.1, ..., 4.x-dev, 5.0.0, ..., 5.3.x-dev, 6.0.0, ..., 6.5.x-dev, 7.0.0-beta.1, ..., 7.4.x-dev].
    - Conclusion: install guzzlehttp/guzzle 6.5.4 (conflict analysis result)

Which relates to this recent update: https://github.com/enupal/translate/issues/43

Can you update your Guzzle requirement so I can update Enupal Translate?

imagehat commented 2 years ago

@chadcrowell - It looks like it is the Avalara PHP SDK requiring Guzzle 6. I'll try a pull request with them but they haven't updated that package in a long time. If we have to I suppose we could use our fork as the dependency.

imagehat commented 2 years ago

@chadcrowell - I was wrong they did release an update that includes Guzzle 7 in the dependencies. I've updated the plugin to v2.2.0 which should fix the Guzzle conflict.