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

Issue #63 #64

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

TesteurManiak
Copy link

Fix #63

@guillempuche
Copy link

guillempuche commented Dec 20, 2021

Nice work Guillaume!

I'm trying to install your fork, but I have this error:
Because every version of dynamic_theme from git depends on shared_preferences >=2.0.8 which requires SDK version >=2.14.0 <3.0.0, dynamic_theme from git is forbidden.

My company project has Dart >=2.12.
Could you downgrade shared_preferences to 2.0.7?

I suggest you to make another version of this package that supports Dart >=2.14

@TesteurManiak
Copy link
Author

TesteurManiak commented Dec 20, 2021

Hi @guillempuche,
I've changed the dependency from shared_preferences: ^2.0.9 to shared_preferences: ">=2.0.0 <3.0.0" which should fix your issue. Note that you can also use dependency_overrides in your pubspec.yaml to force the use of a specific version of shared_preferences:

dependencies:
  # my dependencies
dependency_overrides:
  shared_preferences: 2.0.7

@guillempuche
Copy link

Thanks a lot for sharing your knowledge, Guillaume!

@guillempuche
Copy link

@Norbert515, could you update the package?

@jackbrown1993
Copy link

jackbrown1993 commented Feb 12, 2023

@Norbert515, could you merge this, please?

@freemansoft
Copy link

+1 @Norbert515

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants