Reactiveui: Make Fody ToPropertyEx work against IReactiveObject

Created on 17 Nov 2019  路  6Comments  路  Source: reactiveui/ReactiveUI

Is your feature request related to a problem? Please describe.

I would like to use ToPropertyEx against an instance of IReactiveObject, but it unfortunately requires a ReactiveObject

Describe the solution you'd like

Change the generic constraint on ToPropertyEx to class, IReactiveObject to match ToProperty method.

Describe alternatives you've considered

Implementing the method manually, but that is less convenient that having this out of the box.

Describe suggestions on how to achieve the feature

Change the generic constraint on ToPropertyEx to class, IReactiveObject to match ToProperty method. Unfortunately this causes failure of API approval test. The change makes the constraint more permissive, but it is still a breaking change.

feature request reactiveui-fody

Most helpful comment

Yeah. I have this happening in my testing version.

All 6 comments

Hey @MartinZikmund :wave:,

Thank you for opening an issue. We will get back to you as soon as we can. Also, check out our Open Collective and consider contributing financially.

https://opencollective.com/reactiveui

PS.: We offer priority support for all financial contributors. Don't forget to add priority label once you start contributing :smile:

An advanced, composable, functional reactive model-view-viewmodel framework for all .NET platforms!

If we agree on a way to resolve this, I would love to contribute the change

@glennawatson shall we consider this request at the same time as changing the ToPropertyEx name?

Yeah. I have this happening in my testing version.

Any update on this capability?

Next major release is the plan for this one. There is a major rewrite for the Fody coming up that will support these features.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

glennawatson picture glennawatson  路  3Comments

favesing picture favesing  路  7Comments

jorisvergeer picture jorisvergeer  路  6Comments

ghuntley picture ghuntley  路  4Comments

seanalford picture seanalford  路  3Comments