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

feat(service): add option to set default language in root config #1105

Merged
merged 1 commit into from
Feb 12, 2020

Conversation

arunredhu
Copy link
Contributor

  • Default language option added for specifying the language at the time of defining the configuration in forRoot or forChild method.

  • Now user don't need to explicitly calls the setRootLanguage method for defining the default language at the initial time.

  • Code formatted.

  • Example added for defaultLanguage.

@ocombe
Copy link
Member

ocombe commented Feb 5, 2020

hello and thanks for the PR, it's a good idea, but can you rebase it on master and remove the code formatting changes (it makes it hard to see the actual changes, and creates lots of conflicts)

@arunredhu
Copy link
Contributor Author

@ocombe I have done the changes as you recommended. Please review and let me know in case you have any concerns.

@arunredhu arunredhu requested a review from ocombe February 8, 2020 17:09
…Now user can specifiy defaultLanguage while specifiying the other configuration at the time of registering the module with the application.

Code formatted with prettifier
@ocombe ocombe changed the title Default language option in root config feat(service): add option to set default language in root config Feb 12, 2020
@ocombe ocombe merged commit a13b700 into ngx-translate:master Feb 12, 2020
@ghost
Copy link

ghost commented Apr 2, 2020

https://github.com/ngx-translate/core/pull/1105/files#diff-04c6e90faac2675aa89e2176d2eec7d8R242

Why set the default lang again if it was set in the root config?
Maybe add a comment to mention that not both is necessary?

btw. in markdown, if you use only 1. it will count up automatically ;)

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

Successfully merging this pull request may close these issues.

2 participants