sthewissen / MVP

Unofficial app to help MVPs manage their community activities
GNU General Public License v3.0
112 stars 31 forks source link

[Improvement] AccessTokenExpired and RequestErrorOccurred #16

Closed sthewissen closed 4 years ago

sthewissen commented 4 years ago

Both of these events exist under MvpApiService and should probably be handled accordingly. However, currently, the MvpApiService is being injected into the viewmodels through the constructor. Wiring up additional events there feels... smelly.

Would love to figure out a better approach to this.

sthewissen commented 4 years ago

The App.xaml.cs is taking care of this now, using a singleton instance of the MVP API service and hooking up the events there.