Skip to content

Conversation

@blizzz
Copy link
Member

@blizzz blizzz commented Mar 29, 2017

Downstream of owncloud/user_ldap#54

Simple to review: only adds tooltips :) @nextcloud/ldap

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

Update settings.php

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

Update settings.php

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>

Update settings.php

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@mention-bot
Copy link

@blizzz, thanks for your PR! By analyzing the history of the files in this pull request, we identified @LukasReschke, @jancborchardt and @MorrisJobke to be potential reviewers.

@codecov-io
Copy link

Codecov Report

Merging #4145 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4145      +/-   ##
============================================
- Coverage     54.18%   54.18%   -0.01%     
  Complexity    21316    21316              
============================================
  Files          1309     1309              
  Lines         81343    81343              
  Branches       1285     1285              
============================================
- Hits          44079    44078       -1     
- Misses        37264    37265       +1
Impacted Files Coverage Δ Complexity Δ
apps/user_ldap/templates/settings.php 0% <ø> (ø) 0 <0> (ø) ⬇️
lib/private/Files/Cache/Propagator.php 94.93% <0%> (-1.27%) 16% <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 626d03e...a4b9bd2. Read the comment docs.

Copy link
Member

@MorrisJobke MorrisJobke left a comment

Choose a reason for hiding this comment

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

Tested and works 👍

Sign up for free to 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.

7 participants