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

Only display wizard popup in files app #52

Merged
merged 1 commit into from Sep 20, 2016

Conversation

@PVince81
Copy link
Member

PVince81 commented Sep 20, 2016

Prevents the popup to appear in other pages like in the two factor
validation one.

Fixes owncloud/core#26137 (comment)

Please review @DeepDiver1975 @butonic @cornelinux

Backport to stable9.1 required to fix the two factor situation

Prevents the popup to appear in other pages like in the two factor
validation one.
@PVince81 PVince81 added this to the 9.1.2 milestone Sep 20, 2016
@mention-bot

This comment has been minimized.

Copy link

mention-bot commented Sep 20, 2016

@PVince81, thanks for your PR! By analyzing the annotation information on this pull request, we identified @georgehrke and @LukasReschke to be potential reviewers

@PVince81 PVince81 modified the milestones: 9.2, 9.1.2 Sep 20, 2016
Copy link

cornelinux left a comment

This works for me. The loop in case of 2FA and a new user does not appear anymore. I have not tested anything else of the firstrunwizard.

@butonic

This comment has been minimized.

Copy link
Member

butonic commented Sep 20, 2016

5.4 and 5.5 no longer supported. rest is green

@butonic butonic merged commit f6249c6 into master Sep 20, 2016
1 check failed
1 check failed
continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
@butonic butonic deleted the onlyshowinfilesapp branch Sep 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.