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

To update all dependencies and version of This project #240

Open
andoriyaprashant opened this issue Jun 6, 2023 · 11 comments · May be fixed by #245
Open

To update all dependencies and version of This project #240

andoriyaprashant opened this issue Jun 6, 2023 · 11 comments · May be fixed by #245
Assignees

Comments

@andoriyaprashant
Copy link

Describe the bug
This Project is outdated, it is crucial to keep the project dependencies and version up to date to leverage the latest features, bug fixes, and improvements offered by the Flutter ecosystem and its associated packages.

To resolve this issue, the dependencies listed in the project's "pubspec.yaml" file need to be updated to their latest versions.
And we also have to update build.gradle file for this

To Reproduce
Steps to reproduce the behavior:

  1. Go to pubspec.yaml file and build.gradle file for check the version
  2. Click on terminal and type "flutter pub get" to know the updates before this we have to change the version of environment: sdk in
    pubspec.yaml file
  3. Scroll down to see the updates

Screenshots

Dependencies-
Screenshot 2023-06-06 141335

Build.gradel -
Screenshot 2023-06-06 141344

Expected behavior
Version up to date

Are you working on this issue? (Yes/No)

  • Yes
@andoriyaprashant
Copy link
Author

Hello @Akshatji800
Please review my pull request #241
Thank you

@Akshatji800
Copy link
Member

Hello @Akshatji800 Please review my pull request #241 Thank you

Hello @andoriyaprashant I will review the PR soon.

@andoriyaprashant
Copy link
Author

Ok Thank you @Akshatji800 :)

@andoriyaprashant
Copy link
Author

Hello @Akshatji800
You had asked me to review my pull request. Please provide your response.
Thank you

@Akshatji800
Copy link
Member

Akshatji800 commented Jun 25, 2023

Hello @Akshatji800 You had asked me to review my pull request. Please provide your response. Thank you

Hello @andoriyaprashant, I just reviewed your PR and there are some things which needs to be addressed before the PR can be merged-

  • The Flutter CI is failing as the Flutter version given in the workflow is 3.7.11 and you probably migrated everything to the latest flutter version 3.10 so please change the version in workflow as well.
  • I tested things in my local and got some errors in some part of code due to updated new packages version, many of the functions have been changed and since old version functions were used so they will obviously give errors. For Example:
    image
    Here the BadgeColour line is showing an error as in the new badge package the BadgeColour is moved to BadgeStyle.
    You need to resolve tsuch errors as well if you want to get your PR merged.

@andoriyaprashant
Copy link
Author

I will test all the dependencies versions on the code to ensure compatibility with the new Flutter version, and then I will create a new PR. Thank you so much for providing feedback.
@Akshatji800

@andoriyaprashant andoriyaprashant linked a pull request Jun 27, 2023 that will close this issue
@andoriyaprashant
Copy link
Author

Hey @Akshatji800 I have made the changes you asked me to make, such as making the dependencies compatible with the Flutter version. Please check #245 and let me know.
Thank you.

@simplybest12
Copy link

may i fix this @Akshatji800 ??

@lalitkumawat1m
Copy link

Can I work on this issue ?

@Akshatji800
Copy link
Member

Akshatji800 commented Jul 12, 2023

@simplybest12 @lalitkumawat1m This issue is already being worked on by the issue creator @andoriyaprashant himself.

@asv018
Copy link

asv018 commented Dec 20, 2023

Can you assign this issue to me @Akshatji800

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