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

Star wars sample update to v.6.2.0 #130

Merged
merged 3 commits into from
Sep 14, 2018
Merged

Star wars sample update to v.6.2.0 #130

merged 3 commits into from
Sep 14, 2018

Conversation

FabriBertani
Copy link
Contributor

Update MvvmCross and Plugins from 6.1.2 to 6.2.0.

Fix StarWarsSample.Forms.Android issue: app stuck on SplashScreen after update to v6.2.0. #3104

@dnfclas
Copy link

dnfclas commented Sep 14, 2018

CLA assistant check
All CLA requirements met.

@FabriBertani FabriBertani changed the title Star wars sample update to6.2.0 Star wars sample update to v.6.2.0 Sep 14, 2018
Copy link
Contributor

@nmilcoff nmilcoff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome, thanks! I left a single note

// workaround which should be removed when https://github.com/MvvmCross/MvvmCross/pull/2972 is released
var presenter = base.CreateFormsPagePresenter(viewPresenter);
Mvx.RegisterSingleton<IMvxFormsPagePresenter>(presenter);
var presenter = new MvxFormsPagePresenter(viewPresenter);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove these lines if the workaround isn't necessary anymore.

@nmilcoff nmilcoff merged commit a17a155 into MvvmCross:master Sep 14, 2018
PoLaKoSz pushed a commit to PoLaKoSz/MvvmCross-Samples that referenced this pull request May 14, 2019
* Update MvvmCross and Plugins from 6.1.2 to 6.2.0.

* Fix StarWarsSample.Forms.Android issue: app stuck on SplashScreen after update to v6.2.0.

* Removed unnecessary method from StarWarsSample.Forms.Android Setup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants