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

reword sessions to security in personal settings #5267

Merged
merged 1 commit into from
Jun 7, 2017

Conversation

icewind1991
Copy link
Member

@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Jun 6, 2017
@icewind1991 icewind1991 added this to the Nextcloud 13 milestone Jun 6, 2017
<p class="settings-hint hidden-when-empty"><?php p($l->t('Web, desktop and mobile clients currently logged in to your account.'));?></p>
<div id="security" class="section">
<h2><?php p($l->t('Security'));?></h2>
<p class="settings-hint hidden-when-empty"><?php p($l->t('Web, desktop mobile clients and app specific passwords that currently have access to your account.'));?></p>
Copy link
Member

Choose a reason for hiding this comment

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

missing , between desktop and mobile?

Copy link
Member Author

Choose a reason for hiding this comment

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

fixed

Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991 icewind1991 force-pushed the personal-sessions-renamwe branch from 61eeabe to 340fdc1 Compare June 6, 2017 13:25
@codecov
Copy link

codecov bot commented Jun 6, 2017

Codecov Report

❗ No coverage uploaded for pull request base (master@15314b6). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master    #5267   +/-   ##
=========================================
  Coverage          ?   54.17%           
  Complexity        ?    22302           
=========================================
  Files             ?     1380           
  Lines             ?    85383           
  Branches          ?     1321           
=========================================
  Hits              ?    46257           
  Misses            ?    39126           
  Partials          ?        0
Impacted Files Coverage Δ Complexity Δ
settings/templates/personal.php 0% <0%> (ø) 0 <0> (?)
settings/personal.php 0% <0%> (ø) 0 <0> (?)

@MariusBluem
Copy link
Member

Please backport.

@nickvergessen nickvergessen merged commit 3678be0 into master Jun 7, 2017
@nickvergessen nickvergessen deleted the personal-sessions-renamwe branch June 7, 2017 08:11
@MorrisJobke
Copy link
Member

Please remove the backporten label once the backporten PR is opened ;)

@jancborchardt
Copy link
Member

Cool! :) Simple wording fix which makes things more understandable.

blizzz added a commit that referenced this pull request Jun 16, 2017
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
blizzz added a commit that referenced this pull request Jun 23, 2017
Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants