-
Notifications
You must be signed in to change notification settings - Fork 251
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
[amplify_analytics_pinpoint_dart] Upgrade drift
dependency.
#5323
Comments
@asaarnak thank you for submitting this request. We will take a look and get back to you with any updates. |
Hi, I also have same problem, i think because with latest flutter web now needs min web 1.0.0 and drift over 2.20.
|
@dkliss thanks for prviding this info. we will look into this and will provide update as we have them. |
Hello, adding that we are also experiencing the same issue. Here is one example of several packages we cannot update due to the same constraint. Bumping the version would be greatly appreciated.
|
would you be able to override the drift version in your |
Thanks for this suggestion. Below is what worked for me, in case it helps, still waiting for a fix though:
|
Is there a reason for the upper version bound on the drift dependency? Me messing up semantic versioning? If so, apologies! If not, are you using internal APIs or is there something I can do to help remove the upper bound and enable you to support all |
Hello @simolus3, it's common practice for us to include an upper bound on community dependencies to safeguard the user experience from any potential issues caused by new versions. Some package maintainers don't adhere to SemVer, so breaking changes can sneak into minor/patch versions.
We appreciate the excitement to get this resolved, truly. With this Drift update in particular we have at lease one transitive dependency update that contains breaking changes. This requires extra effort and care to ensure nothing is broken during the update. Like mentioned above, we're working to get this resolved and will update y'all here when we can. |
I'm also blocked by this because I'm using firebase_core and crashlytics. Currently there's a bug keeping apps from getting built on older versions of those packages and many other popular packages using Xcode 16 and the "workaround" is not recommended at all. The issue is resolved in the new version but more and more packages require |
@jamilsaadeh97 sorry that you are facing this issue. you may override the dependecies for now as mentioned above. |
@dkliss have you encountered any issues when using dependency override, everything working fine? |
Yes, have everything is working fine. Have not experienced any issues. |
We are waiting for amplify to update drift version. |
I am unable to build the app after dependency override. I am getting the following error:
|
Hi there, I’m just checking in to see if there’s been any recent progress on this issue. Are there any updates or next steps you could share? |
Hi @tetsujp84, I've been working on this issue as time permits, but the changes have been pretty involved. I've updated our Flutter version, dart sdk, and several packages alongside Drift. The main task remaining is fixing all of the native breaking changes. Thank you for being patient while we clean up this technical debt and we will post here once we finalize the fix! |
Increase
drift
from^2.18.0
to^2.20.0
The text was updated successfully, but these errors were encountered: