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

Fixing SliverStickyHeaderState cast error due to null-safety migration #63

Merged
merged 2 commits into from
Mar 1, 2022

Conversation

amitkot
Copy link
Contributor

@amitkot amitkot commented Apr 22, 2021

Solves #62

@letsar please review, thanks!

@amitkot amitkot mentioned this pull request Apr 22, 2021
@StorminGorman
Copy link

Is development on this halted? I'm tempted to just add a git dep to your PR fork .....

@amitkot
Copy link
Contributor Author

amitkot commented May 18, 2021

That what I did for now.
@letsar would it be possible to merge this? I am currently using it in production on Android and iOS (https://bringitapp.co).

@ThexXTURBOXx
Copy link
Contributor

For the time being just use

dependency_overrides:
  flutter_sticky_header:
    git:
      url: https://github.com/amitkot/flutter_sticky_header.git
      ref: master

@oscarshaitan
Copy link

this should be merged months ago!!!!

@timcreatedit
Copy link

@letsar Pleeeease merge this pull request, it should be really quick and returns this great package to being usable

@oscarshaitan
Copy link

I get tired and create my own widgets with the help of Sliver_tools check that
https://pub.dev/packages/sliver_tools

YRFT added a commit to YRFT/flutter_sticky_header that referenced this pull request Nov 19, 2021
See letsar#63.

This repository is just a quick fix.
@letsar letsar merged commit 035f665 into letsar:master Mar 1, 2022
@letsar
Copy link
Owner

letsar commented Mar 1, 2022

Thank you a lot, I merged it and published a 0.6.1 version on pub

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.

6 participants