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

Upgrade dependencies #128

Merged
merged 8 commits into from
Oct 24, 2024
Merged

Upgrade dependencies #128

merged 8 commits into from
Oct 24, 2024

Conversation

maznnwell
Copy link
Contributor

I've upgraded the flutter_staggered_grid_view to 0.7.0 and replace the defunct SliverStaggeredGrid with StaggeredGrid.

I've delete one unused dependency and upgraded others to their latest versions. Some changes to the code where needed to handle the new APIs.

Now the app compiles with Flutter 3.24.3. But this caused a rendering bug: leanflutter/window_manager#464

I don't know exactly what caused it. I was able to fix it by adding a progress indicator while the lottie animation from the splash screen was being loaded.

I also fix a bug that was preventing my from compiling in release mode.

@KingGorrin
Copy link
Collaborator

That was easier than I thought. Very nicely done.

I was able to build on Windows using flutter 3.24.3. I did some quick testing and everything seems to work, maybe even slightly faster than before. I will do some more extensive testing on Linux and Windows later this week.

@KingGorrin
Copy link
Collaborator

Can confirm custom build with Flutter 3.24.3 commit hash 7adff15 is working on Linux Ubuntu 22.04.5 and Windows 10.

@alienc0der
Copy link
Collaborator

I've tested on macOS and both linux-arm64 and linux-amd64.

@maznnwell
Copy link
Contributor Author

I've added a new commit that fixes the issues with the caret and the text being decentralized inside the seed input fields.

@maznnwell maznnwell requested a review from alienc0der October 23, 2024 10:26
Copy link
Collaborator

@vilkris4 vilkris4 left a comment

Choose a reason for hiding this comment

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

Built with v3.24.3 on Win10 successfully and did some light testing. No issues found.

@alienc0der alienc0der merged commit 25993f6 into zenon-network:develop Oct 24, 2024
@KingGorrin KingGorrin mentioned this pull request Nov 1, 2024
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.

4 participants