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

Updated Dutch language file with new strings #4588

Merged
merged 1 commit into from Oct 2, 2017
Merged

Updated Dutch language file with new strings #4588

merged 1 commit into from Oct 2, 2017

Conversation

silverxp
Copy link
Contributor

Description

Updated Dutch language with all the new strings that are available.

Specific Changes proposed

File lang/nl.json is updated.

Copy link
Member

@misteroneill misteroneill left a comment

Choose a reason for hiding this comment

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

I don't know Dutch, but the JSON is valid. Thanks a bunch for your help!

Copy link
Contributor

@thijstriemstra thijstriemstra left a comment

Choose a reason for hiding this comment

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

Ziet er goed uit (looks good to me).

@gkatsev gkatsev added the minor label Sep 19, 2017
@gkatsev
Copy link
Member

gkatsev commented Sep 19, 2017

Thanks for reviewing @thijstriemstra!

@gkatsev
Copy link
Member

gkatsev commented Sep 19, 2017

Looks like there are a few PRs that can be released as a patch, so, I'm going to wait on merging this and releasing till after those are released. Hopefully, soon.

@misteroneill misteroneill merged commit 5ca0992 into videojs:master Oct 2, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants