You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hey @Adlai-Holler ! I already wrote you on Slack, but you are not using it probably, so I'll post it here as public question as I think I'll not be the only one.
Regarding BETA viewModels on cells ( #356 )
It was merged to version 2.3.4 - so it’s currently in production.
We (as a lot of another users I believe) use view models normally even without Texture support. But if I update to 2.3.4 Texture - I can’t build the app.
It’s because we have property viewModel on node as a (strong, nonatomic) and it needs @dynamic or rename and rewrite to get it working with new Texture. I don’t want to use dynamic to get it working as your property is defined differently (atomic, nullable) + it’s still beta, and I don’t wanna rewrite all cells to have differently named property. I think I’m not the only one facing this problem and decision.
Would it be possible to instead of viewModel as default property by texture use for example nodeViewModel name? Wouldn’t it be better as well if you introduce viewModel for controllers etc - then all of them named based on what kind of viewModel it is?
Sorry that this is half-issue and half-question. But I would like to know where viewModels are going on Texture.
The text was updated successfully, but these errors were encountered:
Hey @Adlai-Holler ! I already wrote you on Slack, but you are not using it probably, so I'll post it here as public question as I think I'll not be the only one.
Regarding BETA viewModels on cells ( #356 )
It was merged to version 2.3.4 - so it’s currently in production.
We (as a lot of another users I believe) use view models normally even without Texture support. But if I update to 2.3.4 Texture - I can’t build the app.
It’s because we have property
viewModel
on node as a(strong, nonatomic)
and it needs@dynamic
or rename and rewrite to get it working with new Texture. I don’t want to usedynamic
to get it working as your property is defined differently(atomic, nullable)
+ it’s still beta, and I don’t wanna rewrite all cells to have differently named property. I think I’m not the only one facing this problem and decision.Would it be possible to instead of
viewModel
as default property by texture use for examplenodeViewModel
name? Wouldn’t it be better as well if you introduce viewModel for controllers etc - then all of them named based on what kind of viewModel it is?Sorry that this is half-issue and half-question. But I would like to know where viewModels are going on Texture.
The text was updated successfully, but these errors were encountered: