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

append login to name #134

Merged
merged 4 commits into from
Feb 16, 2020
Merged

append login to name #134

merged 4 commits into from
Feb 16, 2020

Conversation

jonathanlambrechts
Copy link
Contributor

Hello,
This is a simple commit adding an option to append the login to the password path. Useful if you naming scheme is website.com/login (which allows to have several account for the same website in a single repository).
The german translation of the option description is missing.

jonathanlambrechts and others added 2 commits January 12, 2020 11:30
Signed-off-by: Jonathan Lambrechts <jonathan.lambrechts@uclouvain.be>
@codecov
Copy link

codecov bot commented Jan 20, 2020

Codecov Report

Merging #134 into master will increase coverage by 1.53%.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #134      +/-   ##
========================================
+ Coverage   98.46%   100%   +1.53%     
========================================
  Files           9      9              
  Lines         586    586              
  Branches      101    101              
========================================
+ Hits          577    586       +9     
+ Misses          8      0       -8     
+ Partials        1      0       -1
Impacted Files Coverage Δ
web-extension/create.js 100% <100%> (+28.12%) ⬆️

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 dcf74f5...07ff1d8. Read the comment docs.

@martinhoefling
Copy link
Collaborator

@ccorail I have added the German translation of the option. Can you fix the tests please?

Signed-off-by: Jonathan Lambrechts <jonathan.lambrechts@uclouvain.be>
@jonathanlambrechts
Copy link
Contributor Author

all tests are successful now

@martinhoefling martinhoefling merged commit db9eb32 into gopasspw:master Feb 16, 2020
@martinhoefling
Copy link
Collaborator

@ccorail thx

@jonathanlambrechts jonathanlambrechts deleted the append-login-to-name branch February 17, 2020 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants