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

fix internal links in security & setup warnings, resolves #1048 #1050

Merged
merged 2 commits into from
Aug 25, 2016

Conversation

blizzz
Copy link
Member

@blizzz blizzz commented Aug 25, 2016

@jospoortvliet @LukasReschke @nickvergessen

Fix #1048

small regressions within Admin page, should be backport to Nc 10.

@blizzz blizzz added this to the Nextcloud 11.0 milestone Aug 25, 2016
@mention-bot
Copy link

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

@nickvergessen
Copy link
Member

👍 works

although I think we should just link to the docs directly instead of redirecting to a list of links where the user then has to find the one link that matters for them

@LukasReschke
Copy link
Member

👍

@LukasReschke LukasReschke merged commit ccce3dc into master Aug 25, 2016
@LukasReschke LukasReschke deleted the fix-1048 branch August 25, 2016 20:44
blizzz added a commit that referenced this pull request Aug 25, 2016
fix internal links in security & setup warnings, resolves #1048

adjust js tests
LukasReschke added a commit that referenced this pull request Aug 26, 2016
Backport of fix internal links in security & setup warnings #1050
@MorrisJobke
Copy link
Member

@blizzz Please remove the "backport-request" label once the backport PR is opened. Makes it easier to spot not yet backported PRs ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants