-
Notifications
You must be signed in to change notification settings - Fork 4k
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
docs: explain how to submit TemplateRef instances in global config #4321
Conversation
Deploy preview for ng-zorro-master ready! Built with commit dc53a65 |
Codecov Report
@@ Coverage Diff @@
## master #4321 +/- ##
==========================================
- Coverage 92.16% 92.15% -0.01%
==========================================
Files 520 520
Lines 11016 11016
Branches 1997 1997
==========================================
- Hits 10153 10152 -1
Misses 431 431
- Partials 432 433 +1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks great! Thank you. I will translate it into Chinese later.
docs/global-config.en-US.md
Outdated
@@ -35,7 +35,85 @@ export class AppModule {} | |||
|
|||
These global configuration would be injected into a service named `NzConfigService` and gets stored. | |||
|
|||
## Dynamically Chaing Configs | |||
### Templates |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This title could be more explicit. Something like "Provide Templates in Global Configutation".
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wendzhue I've choosen "Provide Template Instances", as we already are in global configuration scope and it's sub-sub-sub header. Is it ok?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks!
closes #4318
PR Checklist
Please check if your PR fulfills the following requirements:
Tests for the changes have been added (for bug fixes / features)PR Type
What kind of change does this PR introduce?
What is the current behavior?
See issue.
Issue Number: #4318
Plus fixed a typo: Dynamically
ChaingChaining ConfigurationsDoes this PR introduce a breaking change?