Skip to content

Add repair step and revert "Open updater" button - #1499

Merged
LukasReschke merged 5 commits into
masterfrom
updater
Sep 26, 2016
Merged

Add repair step and revert "Open updater" button#1499
LukasReschke merged 5 commits into
masterfrom
updater

Conversation

@MorrisJobke

Copy link
Copy Markdown
Member

cc @LukasReschke@nickvergessen@karlitschek

I will open the PRs for stable9 and stable10 soon

This reverts commit 202ae42.
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@mention-bot

Copy link
Copy Markdown

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

@karlitschek

Copy link
Copy Markdown
Member

nice 👍 As discussed!

@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

needed to rebase to fix the sign-off message in the commits ;)

'updateLink' => 'https://docs.nextcloud.com/myUrl',
'downloadLink' => 'https://downloads.nextcloud.org/server',
'updateVersion' => 'ownCloud 123',
'updateLink' => 'https://owncloud.org/myUrl',

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Id not revert all in this, when possible

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

True story. I will fix it.

@nickvergessen

Copy link
Copy Markdown
Member

Looks good otherwise

@MorrisJobkeMorrisJobke self-assigned this Sep 26, 2016
This reverts commit 27a3c68.
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
Signed-off-by: Morris Jobke <hey@morrisjobke.de>
@MorrisJobke

Copy link
Copy Markdown
MemberAuthor

I added the wrongly removed tests here.

@LukasReschke

Copy link
Copy Markdown
Member

LGTM

@LukasReschke
LukasReschke merged commit a43c680 into masterSep 26, 2016
@LukasReschke
LukasReschke deleted the updater branch September 26, 2016 20:28
@MorrisJobkeMorrisJobke removed their assignment Oct 6, 2016
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@MorrisJobke@mention-bot@karlitschek@nickvergessen@LukasReschke