-
-
Notifications
You must be signed in to change notification settings - Fork 287
docs: add requireSync note, ssr root route, app generation, and custom URL prefix #1379
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
Conversation
…custom URL prefix
✅ Deploy Preview for analog-ng-app ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for analog-app ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for analog-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for analog-blog ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
apps/docs-app/docs/features/data-fetching/server-side-data-fetching.md
Outdated
Show resolved
Hide resolved
This instructs Angular on how recognize and generate URLs. | ||
|
||
```ts | ||
import { ApplicationConfig } from '@angular/core'; |
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.
Why was this removed?
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.
I got rid of app.config.env.ts with mergeApplicationConfig in favor of using import.meta.env.BASE_URL
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.
Yes, but ApplicationConfig
is still being used in the code snippet
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.
Got it! Done.
Co-authored-by: Brandon Roberts <[email protected]>
Co-authored-by: Brandon Roberts <[email protected]>
Thanks! @allcontributors add @leblancmeneses for doc |
@leblancmeneses already contributed before to doc |
docs(content): requireSync note, ssr root route, app generation, and custom URL prefix