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.

Sign up for GitHub

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

Jump to bottom

A brand new version that's up-to-date with Flutter #2

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

A brand new version that's up-to-date with Flutter #2

wants to merge 2 commits into from

Conversation

moazelsawaf
Copy link

  • Migrated the example app to Flutter Embedding Version 2.
  • Migrated the package to Null Safety.
  • Added flutter_lints package to both the package and the example and applied all the lints.
  • Upgraded the provider version to the latest one ( until the date of the commit ).
  • Disposing the PageController, thanks to @davidsdearaujo, I have noticed it in one of his commit in his fork.
  • Removed all the warnings from the package and the example.

@moazelsawaf
A brand new version that's up-to-date with Flutter
1c009b7 
- Migrated the example app to Flutter Embedding Version 2.
- Migrated the package to Null Safety.
- Added flutter_lints package to both the package and the example and applied all the lints.
- Upgraded the provider version to the latest one ( until the date of the commit ).
- Removed all the warnings from the package and the example.
@moazelsawaf
Disposing the PageController, thanks to @davidsdearaujo, I have notic…
043ff7f 
…ed it in one of his commit in his fork.
@moazelsawaf
Copy link
Author

@retroportalstudio
Any updates about this one? 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@moazelsawaf