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

German language update #4016

Closed
wants to merge 5 commits into from
Closed

German language update #4016

wants to merge 5 commits into from

Conversation

CHR15714N
Copy link

No description provided.

@PieterGit
Copy link
Contributor

@CHR15714N Please use dev branch version for language updates. Please change branch or retarget with a new PR.

@PieterGit PieterGit added this to the 0.11.0 milestone Oct 25, 2018
@PieterGit PieterGit changed the title update German language update Oct 25, 2018
@CHR15714N
Copy link
Author

CHR15714N commented Oct 25, 2018

@PieterGit Sorry, I just wanted to modify my version. Did I modify the general one?
I only use Github for nightscout and I am no Developer so sorry for my ignorance...

@PieterGit
Copy link
Contributor

@CHR15714N You didn't modify anything, you proposed a change, but others have made translations in the same file. Please check https://openaps.readthedocs.io/en/latest/docs/Resources/my-first-pr.html and use the dev branch. Please retry with another PR and close this PR when your done. If you need help hop on https://gitter.im/nightscout/public

@MilosKozak MilosKozak changed the base branch from master to dev December 4, 2018 10:06
@MilosKozak MilosKozak changed the base branch from dev to master December 4, 2018 10:07
@MilosKozak
Copy link
Contributor

simply change base is not possible now. closing. please reopen for dev branch if needed

@MilosKozak MilosKozak closed this Dec 4, 2018
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.

3 participants