Skip to content

Removed unecessary OAPH#1

Open
giusepe wants to merge 1 commit intocoolc0ders:masterfrom
giusepe:master
Open

Removed unecessary OAPH#1
giusepe wants to merge 1 commit intocoolc0ders:masterfrom
giusepe:master

Conversation

@giusepe
Copy link

@giusepe giusepe commented Mar 13, 2018

Changed the LoginCommand to use the return of the task in a Observable pipeline
Changed the AddCommand to use a simple sync procedure

Changed the LoginCommand to use the return of the task in a Observable pipeline
Changed the AddCommand to use a simple sync procedure
@DamienDoumer
Copy link
Member

Hi, thanks for the pull request,
I saw your code and you are right, OAPH could be replaced by observables, but My aim for making the demo was to demonstrate the use of as many Reactiveui functionalities as possible.
What I will do is make another blog post where I'll mention your pull request and a way of setting can execute on commands without OAPH.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants