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

Added new PoGoBaseViewModel#1616

Open
Markusch85 wants to merge 4 commits intoST-Apps:masterfrom
Markusch85:PoGoViewModelBase
Open

Added new PoGoBaseViewModel#1616
Markusch85 wants to merge 4 commits intoST-Apps:masterfrom
Markusch85:PoGoViewModelBase

Conversation

@Markusch85
Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown
Author

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

@trainerfloripa
Copy link
Copy Markdown

What will be implement in the next update?

Obs: My English sucks I know, sorry!🙂

@kaantantr
Copy link
Copy Markdown

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

@Somebody2804
Copy link
Copy Markdown

@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