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

Change NotifyPropertyChanged to OnPropertyChanged #223

Open
noufionline opened this issue Nov 2, 2018 · 1 comment
Open

Change NotifyPropertyChanged to OnPropertyChanged #223

noufionline opened this issue Nov 2, 2018 · 1 comment
Labels

Comments

@noufionline
Copy link

Some third party tools like postsharp heavily relies on conventions. Many occations when we try to inherit from EntityBase which has an implementation of INotifyPropertyChanged most of the third party tooling follows the same convention.

It would be a welcome change.

Thanks

@tonysneed
Copy link
Collaborator

That's interesting. Yeah, the client-side change tracking relies on INotifyPropertyChanged. Not familiar enough with PostSharp to know how this poses a conflict.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants