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

[amplify_analytics_pinpoint_dart] Upgrade drift dependency. #5323

Open
asaarnak opened this issue Aug 21, 2024 · 6 comments
Open

[amplify_analytics_pinpoint_dart] Upgrade drift dependency. #5323

asaarnak opened this issue Aug 21, 2024 · 6 comments
Labels
analytics Issues related to the Amplify Analytics Category dependencies Pull requests that update a dependency file question A question about the Amplify Flutter libraries

Comments

@asaarnak
Copy link
Contributor

asaarnak commented Aug 21, 2024

Increase drift from ^2.18.0 to ^2.20.0

Because no versions of amplify_auth_cognito match >2.4.0 <3.0.0 and amplify_auth_cognito 2.4.0 depends on amplify_analytics_pinpoint_dart ^0.4.4, amplify_auth_cognito ^2.4.0 requires amplify_analytics_pinpoint_dart ^0.4.4.
Because amplify_analytics_pinpoint_dart 0.4.4 depends on drift >=2.18.0 <2.19.0 and no versions of amplify_analytics_pinpoint_dart match >0.4.4 <0.5.0, amplify_analytics_pinpoint_dart ^0.4.4 requires drift >=2.18.0 <2.19.0.
Thus, amplify_auth_cognito ^2.4.0 requires drift >=2.18.0 <2.19.0.
So, because my_app depends on both amplify_auth_cognito ^2.4.0 and drift ^2.20.0, version solving failed.
@NikaHsn
Copy link
Member

NikaHsn commented Aug 21, 2024

@asaarnak thank you for submitting this request. We will take a look and get back to you with any updates.

@NikaHsn NikaHsn added analytics Issues related to the Amplify Analytics Category question A question about the Amplify Flutter libraries labels Aug 21, 2024
@dkliss
Copy link

dkliss commented Aug 22, 2024

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.

Because no versions of amplify_auth_cognito match >2.4.0 <3.0.0 and amplify_auth_cognito 2.4.0 depends on amplify_analytics_pinpoint_dart ^0.4.4, amplify_auth_cognito ^2.4.0 requires amplify_analytics_pinpoint_dart ^0.4.4.
Because amplify_analytics_pinpoint_dart 0.4.4 depends on drift >=2.18.0 <2.19.0 and no versions of amplify_analytics_pinpoint_dart match >0.4.4 <0.5.0, amplify_analytics_pinpoint_dart ^0.4.4 requires drift >=2.18.0 <2.19.0.
Thus, amplify_auth_cognito ^2.4.0 requires drift >=2.18.0 <2.19.0.
Because drift 2.18.0 depends on web ^0.5.0 and no versions of drift match >2.18.0 <2.19.0, drift >=2.18.0 <2.19.0 requires web ^0.5.0.
Thus, amplify_auth_cognito ^2.4.0 requires web ^0.5.0.
So, because my_app depends on both amplify_auth_cognito ^2.4.0 and web ^1.0.0, version solving failed.

@NikaHsn
Copy link
Member

NikaHsn commented Aug 22, 2024

@dkliss thanks for prviding this info. we will look into this and will provide update as we have them.

@ma-vai
Copy link

ma-vai commented Sep 5, 2024

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.

Because no versions of amplify_auth_cognito match >2.4.1 <3.0.0 and amplify_auth_cognito 2.4.1 depends on amplify_analytics_pinpoint_dart ^0.4.5, amplify_auth_cognito ^2.4.1 requires amplify_analytics_pinpoint_dart ^0.4.5. And because no versions of amplify_analytics_pinpoint_dart match >0.4.5 <0.5.0 and amplify_analytics_pinpoint_dart 0.4.5 depends on drift >=2.18.0 <2.19.0, amplify_auth_cognito ^2.4.1 requires drift >=2.18.0 <2.19.0. Because drift 2.18.0 depends on web ^0.5.0 and no versions of drift match >2.18.0 <2.19.0, drift >=2.18.0 <2.19.0 requires web ^0.5.0. Thus, amplify_auth_cognito ^2.4.1 requires web ^0.5.0. And because flutter_webrtc >=0.11.7 depends on web ^1.0.0, amplify_auth_cognito ^2.4.1 is incompatible with flutter_webrtc >=0.11.7. So, because verseai_flutter_client depends on both amplify_auth_cognito ^2.4.1 and flutter_webrtc ^0.11.7, version solving failed.

@github-actions github-actions bot added the pending-maintainer-response Pending response from a maintainer of this repository label Sep 5, 2024
@NikaHsn
Copy link
Member

NikaHsn commented Sep 5, 2024

would you be able to override the drift version in your pubspec_overrides.yaml to use the latest version?
we need to upgrade other dependecies e.g. pigeon, to major versions, in order to upgrade the drift version. this will take some time, we'll keep you updated here.

@github-actions github-actions bot removed the pending-maintainer-response Pending response from a maintainer of this repository label Sep 5, 2024
@dkliss
Copy link

dkliss commented Sep 16, 2024

drift

Thanks for this suggestion. Below is what worked for me, in case it helps, still waiting for a fix though:

dependencies:
  amplify_flutter: ^2.4.1
  amplify_auth_cognito: ^2.4.1
  drift: ^2.18.0
sqlite3: 
dependency_overrides:
  sqlite3: ^2.4.3
  drift: ^2.20.2

@github-actions github-actions bot added the pending-maintainer-response Pending response from a maintainer of this repository label Sep 16, 2024
@Equartey Equartey added dependencies Pull requests that update a dependency file and removed pending-maintainer-response Pending response from a maintainer of this repository labels Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
analytics Issues related to the Amplify Analytics Category dependencies Pull requests that update a dependency file question A question about the Amplify Flutter libraries
Projects
None yet
Development

No branches or pull requests

5 participants