search

rrousselGit / state_notifier

ValueNotifier, but outside Flutter and with some extra perks
MIT License
311 stars 28 forks source link

Show that state_notifier is no longer recommended for riverpod #77

Closed noriHanda closed 1 year ago

noriHanda commented 1 year ago

Why

I'm in a riverpod discord server and answering questions asked by riverpod newcomers. As I see those questions, I realized so many people still use StateNotifier when there is Notifier and so force.

Explicitly stating that this is to be deprecated for Riverpod would be helpful for those who start using the state management package since there still are a lot of literature that tells you to use StateNotifier.

Changes made

Just a simple add-on to README. Format or wording suggestions are very much welcomed.

rrousselGit commented 1 year ago

I appreciate the thought, but I don't think this is the correct place. state_notifier is a standalone package. It has nothing to do with Riverpod.

It's Riverpod docs instead that should mention it.

But I do consider archiving this package

rrousselGit commented 1 year ago

Closing due to the previous comment. I'd suggest making a PR on Riverpod's docs instead