Skip to content
This repository has been archived by the owner on Aug 17, 2020. It is now read-only.

Added new PoGoBaseViewModel #1616

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Added new PoGoBaseViewModel #1616

wants to merge 4 commits into from

Conversation

Markusch85
Copy link

Closes issues

no issues to close

Changes

  • Added: PoGoViewModelBase

Change details

The new PoGoViewModelBase is an Abstract ViewModel which ca be implemted by other ViewModels. Currently it handles Standard Lifecycycles for:

  • OnNavigatedToAsync
  • OnNavigatedFromAsync
  • OnNavigatingFromAsync

and Bindable Game Vars for:

  • PlayerStats
  • PlayerProfile

Other informations

If anyone have to use the standard implementation of the above called Vars and LifeCycle methods, the one can use the PoGoViewModelBase.
Another reason, why I've added the new ViewModel is, that we reduce duplicate Code.

@robertmclaws
Copy link
Collaborator

This is cool. I don't know if it's going to get accepted for 1.1, but we are doing something similar for the 2.0 architecture. It's going to use Dependency Injection to inject different game services into the different ViewModels, which you'll be able to bind to directly. Should make things orders of magnitude more simple.

@Markusch85
Copy link
Author

Nice to hear, that you are working on dependency injection for Version 2.0 can I help you implementing that?

@trainerfloripa
Copy link

What will be implement in the next update?

Obs: My English sucks I know, sorry!🙂

@kaantantr
Copy link

@Jehu147 @trainerfloripa Please no questions regarding when the next release will come. It is ready when it is ready.

@Somebody2804
Copy link

@trainerfloripa Don't worry about your English you don't have to speak it here it's just that lots of people do.

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

Successfully merging this pull request may close these issues.

5 participants