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

connectivity_plus ^3.0.3 conflicts with easy_localization_loader ^1.0.1+1 #46

Closed
liplum opened this issue Feb 16, 2023 · 1 comment · Fixed by #50
Closed

connectivity_plus ^3.0.3 conflicts with easy_localization_loader ^1.0.1+1 #46

liplum opened this issue Feb 16, 2023 · 1 comment · Fixed by #50

Comments

@liplum
Copy link

liplum commented Feb 16, 2023

What's Wrong

As of press time, the latest connectivity_plus is v3.0.3, but easy_localization_loader still depends on the old one ^2.3.7.

In easy_localization_loader/pubspec.yaml.

connectivity_plus: ^2.3.7

And connectivity_plus is used in smart_network_asset_loader.

Log

Running "flutter pub get" in my_app...
Resolving dependencies...
Because my_app depends on easy_localization_loader ^1.0.1+1 which depends on connectivity_plus ^2.3.7, connectivity_plus ^2.3.7 is required.
So, because my_app depends on connectivity_plus ^3.0.3, version solving failed.

My Solutions

  1. To separate smart_network_asset_loader from the easy_localization_loader. As to Single-responsibility principle, an asset loader should not mess with the connectivity of user devices.
    It sounds cumbersome, but there has been issued before: Separate Packages #41 xml dependency incompatibility #36.
  2. To upgrade connectivity_plus depenedency. It's straightforward.
  3. Since every asset loader is in a single file, it's easy to copy which loader you want direcly without adding to pubspec.yaml.
@kpk-i710
Copy link

same problem

alexeyinkin added a commit to alexeyinkin/easy_localization_loader that referenced this issue Jun 26, 2023
alexeyinkin added a commit to alexeyinkin/easy_localization_loader that referenced this issue Jun 27, 2023
bw-flagship added a commit that referenced this issue Jul 4, 2023
Delete local AssetLoader class, update dependencies (#46)
bw-flagship added a commit that referenced this issue Oct 30, 2023
* Update README.md

* chore(): upgraded deps

* chore(): load concrete location

* fix(): correct decoding of cyrillic chars

* chore(): updated readme. added HttpAssetLoader doc

* fix(doc): update dir structure

* Delete local AssetLoader class, update dependencies (#46)

* Undo the breaking change of file names (#46)

* contributing.md

* added pipelines

* fix linting problems

* changelog

* devversion

* execute now

* revert temp changes

* update connectivity_plus, easy_localization, http and path_provider packages to latest version

* chore(): upgraded deps

* chore(): load concrete location

* fix(): correct decoding of cyrillic chars

* chore(): updated readme. added HttpAssetLoader doc

* fix(doc): update dir structure

---------

Co-authored-by: Frankdroid7 <[email protected]>
Co-authored-by: Alexey Inkin <[email protected]>
Co-authored-by: Benjamin Weber <[email protected]>
Co-authored-by: BW Personal <[email protected]>
Co-authored-by: Abdulaziz Rasulbek <[email protected]>
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 a pull request may close this issue.

2 participants