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.

Sign up for GitHub

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

Jump to bottom

Corrected two dutch translation which made no sense #4261

Merged
merged 1 commit into from Nov 4, 2022
Merged

Corrected two dutch translation which made no sense #4261

merged 1 commit into from Nov 4, 2022

Conversation

joost00719
Copy link
Contributor

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 3, 2022 •
edited

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@joost00719
Copy link
Contributor Author

"duidelijk" technically means "clear", but it means something more close to "understood".
I change it to "Reset", which also means "reset" in english. There is no good translation which goes for "clear", but "reset" fits the context better.

@joost00719
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@David-Moreira
Copy link
Contributor

David-Moreira commented Nov 3, 2022 •
edited

@joost00719 can you check that your github registered email is the same you've used for your commit?

github-actions bot added a commit that referenced this pull request Nov 3, 2022
@github-actions
@joost00719 has signed the CLA from Pull Request #4261
dbd93d7
@joost00719
Copy link
Contributor Author

@David-Moreira I have resolved the check issues

@joost00719 joost00719 changed the base branch from master to rel-1.1 November 3, 2022 22:13
stsrki
stsrki approved these changes Nov 4, 2022
View reviewed changes
Copy link
Collaborator

@stsrki stsrki left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you!

@stsrki stsrki mentioned this pull request Nov 4, 2022
Closed
@stsrki stsrki merged commit c9f8f23 into Megabit:rel-1.1 Nov 4, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Nov 4, 2022
@joost00719 joost00719 deleted the dutch-translation-correction branch November 4, 2022 13:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@stsrki stsrki stsrki approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@joost00719 @David-Moreira @stsrki