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

Compilation error #186 Solved ✅ #188

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

VaibhavYT
Copy link

Fixes #186

Describe the changes you have made in this PR -
I have updated and resolved all the dependencies in project which are causing compilation error and changed compile Sdk version to latest Sdk version for running the application.

Screenshots:

beacon#186

missing .env file which is because .env is placed in .gitignore.

@ItsAdityaKSingh
Copy link
Collaborator

Some of the tests are failing @VaibhavYT, can you fix them?

@Rushour0
Copy link

I believe this issue has been resolved in my latest PR #190
@VaibhavYT @ItsAdityaKSingh if you can check

@@ -46,6 +46,7 @@ dependencies:
skeleton_text: ^3.0.0
sliding_up_panel: ^2.0.0+1
uni_links: ^0.5.1
web_socket_channel: ^2.2.0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please remove this. ws 2.2.0 is a transitive dependency from v5.1.3 of graphql_flutter.
no need to make it a main dependency
https://github.com/zino-hofmann/graphql-flutter/blob/main/packages/graphql/CHANGELOG.md

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 this pull request may close these issues.

Compilation error
4 participants