Skip to content

Issue #63#64

Open
TesteurManiak wants to merge 8 commits intoNorbert515:masterfrom
TesteurManiak:issue_#63
Open

Issue #63#64
TesteurManiak wants to merge 8 commits intoNorbert515:masterfrom
TesteurManiak:issue_#63

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