Skip to content

Fix misleading wording: searching is affected, not user access. - #3550

Merged
LukasReschke merged 1 commit into
masterfrom
fix-misleading-wording
Feb 20, 2017
Merged

Fix misleading wording: searching is affected, not user access.#3550
LukasReschke merged 1 commit into
masterfrom
fix-misleading-wording

Conversation

@blizzz

Copy link
Copy Markdown
Member

What the title says. Label fix, not a code one.

Since the wording is confusing and misleading, it is worth to backport this change. OKs, @karlitschek?

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@blizzzblizzz added 3. to review Waiting for reviews backport-request bug feature: ldap papercut Annoying recurring UX issue with possibly simple fix. labels Feb 20, 2017
@mention-bot

Copy link
Copy Markdown

@blizzz, thanks for your PR! By analyzing the history of the files in this pull request, we identified @davicente to be a potential reviewer.

@blizzzblizzz added this to the Nextcloud 12.0 milestone Feb 20, 2017
@karlitschek

Copy link
Copy Markdown
Member

backport makes sense

@codecov-io

Copy link
Copy Markdown

Codecov Report

Merging #3550 into master will not change coverage.
The diff coverage is n/a.

@@ Coverage Diff @@## master #3550 +/- ##
=========================================
Coverage 54.14% 54.14% Complexity 21039 21039 =========================================
Files 1306 1306 Lines 80298 80298 Branches 1250 1250 =========================================
Hits 43479 43479 Misses 36819 36819
Impacted FilesCoverage ΔComplexity Δ
...pps/user_ldap/templates/part.wizard-userfilter.php0% <ø> (ø)0 <ø> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cab27ea...866b218. Read the comment docs.

@LukasReschke
LukasReschke merged commit fba2a9f into masterFeb 20, 2017
@LukasReschke
LukasReschke deleted the fix-misleading-wording branch February 20, 2017 20:16
@LukasReschke

Copy link
Copy Markdown
Member

@blizzz Can you open backports? THX! :)

@blizzz

Copy link
Copy Markdown
MemberAuthor

stable11: #3560
stable10: #3561
stable9: #3563

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to reviewWaiting for reviewsbugfeature: ldappapercutAnnoying recurring UX issue with possibly simple fix.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@blizzz@mention-bot@karlitschek@codecov-io@LukasReschke@MorrisJobke