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

creating password store directory, if it doesnot exists, de_DE translation fixes and removed obsolete translations #216

Merged
merged 3 commits into from Nov 13, 2016

Conversation

@YoshiMan
Copy link

@YoshiMan YoshiMan commented Nov 13, 2016

No description provided.

@annejan
Copy link
Member

@annejan annejan commented Nov 13, 2016

The more important bit is d93aa12 creating password store directory, if it doesnot exists.

Seems like a good addition to the "first run" scenario and makes adding profiles easier.

@YoshiMan
Copy link
Author

@YoshiMan YoshiMan commented Nov 13, 2016

I really struggled because i didnt had the $USERHOME/.password-store directory. So on the right side of the mainwindow was an info QProcess::FailedToStart and i had no idea, whats wrong. The working directory of the was process ($USERHOME/.password-store) didnt exist. That was the fault.

@annejan annejan changed the title improvement of passStore, de_DE translation fixes and removed obsolete translations creating password store directory, if it doesnot exists, de_DE translation fixes and removed obsolete translations Nov 13, 2016
@annejan annejan merged commit 2069cfe into IJHack:master Nov 13, 2016
4 checks passed
4 checks passed
Snap CI The Snap CI build passed on Nov 13, 2016!
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
linthub Linthub has cleared the pull request, no code style suggestions found.
Details
@YoshiMan
Copy link
Author

@YoshiMan YoshiMan commented Nov 13, 2016

thanks for merging
yes, the title is better ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.