-
Notifications
You must be signed in to change notification settings - Fork 77
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
Deprecate ScopedModel? #86
Comments
I'm currently due to release my first Flutter app soon and that relies completely on ScopedModel. If this is deprecated does this mean potentially with future Flutter upgrades you will no longer support any required changes and my app could at some point stop working? Looks like I will have to look into Provider for my next project. I'm hoping a potential migration from ScopedModel to Provider wouldn't be too difficult! |
Yep, that's exactly right. It will be deprecated and should continue to work for Flutter at the moment, but if there is a large breaking change to Flutter, this project would not be updated to fix those changes. Migrating to Provider should be quite straight-forward. Please see this example in another discussion of the changes that need to be made: #61 (comment) |
Hi Brian, |
Same here. I have all of my projects with ScopedModel. EDIT: But I have to add, that I totally love ScopedModel because of its simplicity. It's totally easy to use and powerful even for complex applications. I never needed to use BLoC pattern or anything different just because ScopedModel is totally sufficient! So, I hope that migration to Provider will be pretty straight-forward as already mentioned above. EDIT2: @brianegan It would be perfect if ScopedModel remains maintained at least until the end of 2020. That would be totally safe timeline for migration considering our workload. Thanks!! |
@brianegan I am heavily dependent on scoped_model in quite a major commercial app that is about to go live so I will need a smooth transition when you deprecate the package. Will there be comprehensive documentation provided on how best to migrate to Provider? With Flutter becoming more mainstream, there are people building stuff for the enterprise and using packages and tools provided (sorry) by clever people like yourself. However, we need to feel confident that we are not painting ourselves into a corner when we make use of packages fundamental to the apps we write. Would appreciate your views on this. |
@GrahamDi The pub.dev constantly tries to get support for yours packages, for example with Favorite Flutter or verified editors. |
@GrahamDi I recently ported a decent sized commercial app from ScopedModel to Provider.
It took me approximately 2 days. |
Very nice :D |
Thanks so much for the feedback, everyone! Really appreciate the perspective from folks using this library :) I'll plan on supporting ScopedModel for the rest of the year, and I'll look to see if I can write a codemod that will perform the upgrade from Scoped Model to ChangeNotifier / Provider automatically for you. This should make moving from Scoped Model to provider much simpler! |
@brianegan Thanks a lot. A codemod would be much appreciated :-) |
@brianegan instead of deprecating |
Anyway, this library is ultra simple, and the saying goes: "Simplicity is the ultimate sophistication". So while So I'd vote for keeping it alive. |
Here's my shot at a "Rebirth" of ScopeModel https://github.com/icnahom/scoped_model |
Interesting...thanks for this...still using scoped model in my app so will
try your version out...very happy that it still has life :-)
Krgrds, Graham
…On Thu, Jun 30, 2022, 17:31 icnahom ***@***.***> wrote:
Here's my shot for a "Rebirth of ScopeModel"
https://github.com/icnahom/scoped_model
—
Reply to this email directly, view it on GitHub
<#86 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AHVES4EIQSI6MC3URES4CCDVRW4URANCNFSM4JBXYC2A>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hey all, thanks for the feedback.
I’d ask to hold off on forking this library for now unless you really want
to take it in a new direction, in which case I’d recommend giving the
library a new name.
it would be better to onboard you as a new maintainer if you’re interested.
Pub.dev is already very fractured and this library has provided support for
all versions of Flutter for a very long time (including the latest
versions) with a perfect pana score.
While we may not add new features… that in itself is a major bonus for some
folks.
…On Thu, Jun 30, 2022 at 5:40 PM GrahamDi ***@***.***> wrote:
Interesting...thanks for this...still using scoped model in my app so will
try your version out...very happy that it still has life :-)
Krgrds, Graham
On Thu, Jun 30, 2022, 17:31 icnahom ***@***.***> wrote:
> Here's my shot for a "Rebirth of ScopeModel"
> https://github.com/icnahom/scoped_model
>
> —
> Reply to this email directly, view it on GitHub
> <
#86 (comment)
>,
> or unsubscribe
> <
https://github.com/notifications/unsubscribe-auth/AHVES4EIQSI6MC3URES4CCDVRW4URANCNFSM4JBXYC2A
>
> .
> You are receiving this because you were mentioned.Message ID:
> ***@***.***>
>
—
Reply to this email directly, view it on GitHub
<#86 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAA65DCOM7EYI7SEIRHTXS3VRW5XNANCNFSM4JBXYC2A>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hey Brian, it's nice hearing from you! Do you think thay we should avoid making changes like renaming ScopedModelDescendant to the now very familiar way of naming Builders - ScopedBuilder? |
The new feature is |
My concern in becoming a maintainer of brianegan/scoped_model is that you'll be too busy to push up releases to pub.dev |
From my perspective, I think it makes sense to bring In the new features
from the Armadillo library since this was a port out of the Google repo.
The only code I really added was tests to ensure everything worked as
expected.
I also think API stability is more important than renames, since this
library has been essentially unchanged for 5 years and continues to do the
job. If you want renames, I think provider might make more sense? Again,
this is just my opinion.
With regards to pushing more often, I’m happy to move this library over to
the flutter community where we can invite more folks to maintain the repo
and push to pub.dev.
Apologies for the short responses, on the last days of vacation, but happy
to hear what you all think as well.
…On Fri, Jul 1, 2022 at 7:58 PM Luke Pighetti ***@***.***> wrote:
Hey all, thanks for the feedback. I’d ask to hold off on forking this
library for now unless you really want to take it in a new direction, in
which case I’d recommend giving the library a new name. it would be better
to onboard you as a new maintainer if you’re interested. Pub.dev is already
very fractured and this library has provided support for all versions of
Flutter for a very long time (including the latest versions) with a perfect
pana score. While we may not add new features… that in itself is a major
bonus for some folks.
… <#m_7089176748146218621_>
My concern in becoming a maintainer of brianegan/scoped_model is that
you'll be too busy to push up releases to pub.dev
—
Reply to this email directly, view it on GitHub
<#86 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAA65DBGXP3OUMA5U3JMIQ3VR4WVFANCNFSM4JBXYC2A>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hey all! This might be a big change, but I think it might be the right time.
Scoped Model is a great library and a lot of folks have found success with it. However, it has largely been eclipsed by the
Provider
package. In many ways, you can think ofProvider
as Scoped Model version 2: It includes the same functionality, but extends that functionality in really nice ways.Model
class is no longer needed. You can simply useChangeNotifier
from the Flutter library directly!ChangeNotifierProvider
andConsumer
do the same job asScopedModel
andScopedModelDescendant
.Selector
Widget as well, which is really handyMultiProvider
makes providing multiple Models very easy without a bunch of nesting.At this point, I think it might make sense to deprecate this library so it no longer appears in the search results of pub.dev and encourage folks to migrate to Provider. The library will still be available to projects that reference Scoped Model, so you shouldn't have any disruptions for projects that currently use it or need to import it from Pub.dev.
Would love any thoughts from folks in the community before making such a drastic change!
The text was updated successfully, but these errors were encountered: