Skip to content

add getDataByLanguage into index.d.ts #1326

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

Merged
merged 2 commits into from
Sep 16, 2019
Merged

add getDataByLanguage into index.d.ts #1326

merged 2 commits into from
Sep 16, 2019

Conversation

msftjayongg
Copy link
Contributor

@msftjayongg msftjayongg commented Sep 14, 2019

update index.d.ts with the public getDataByLanguage API interface. Basically this allows typescript to call the API that already exists. No functionality change.

Closes #1273

@coveralls
Copy link

coveralls commented Sep 14, 2019

Coverage Status

Coverage remained the same at 88.568% when pulling 7813787 on msftjayongg:master into 85bbb39 on i18next:master.

@msftjayongg
Copy link
Contributor Author

@rosskevin I was told to tag you.

I am also unsure how to actually run the typescript tests - I guess they are there just for tslint purposes? (I'd feel better if I could run it :) )

@jamuhl jamuhl requested a review from rosskevin September 14, 2019 06:46
Copy link
Collaborator

@rosskevin rosskevin left a comment

Choose a reason for hiding this comment

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

Looks good, just one small change, thank you.

@msftjayongg
Copy link
Contributor Author

Updated - I left a line such that we can check that the .translation field exists.

@rosskevin rosskevin merged commit c28ca92 into i18next:master Sep 16, 2019
@rosskevin
Copy link
Collaborator

Thanks @msftjayongg

@jamuhl this can be a patch release.

@jamuhl
Copy link
Member

jamuhl commented Sep 17, 2019

published in i18next@17.0.16

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.

Is getDataByLanguage a public api?
4 participants