Skip to content

feat(scully): idle detection for non angular apps (experimental)#515

Merged
SanderElias merged 2 commits intomasterfrom
sander/injectZoneInBareProject
Apr 28, 2020
Merged

feat(scully): idle detection for non angular apps (experimental)#515
SanderElias merged 2 commits intomasterfrom
sander/injectZoneInBareProject

Conversation

@SanderElias
Copy link
Copy Markdown
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@SanderElias SanderElias requested a review from jorgeucano April 28, 2020 10:25
Comment thread docs/getting-started.md Outdated
and make sure that the site's navigation still works and most parts of it should still work without JS enabled.

** Extra Extra credits **: If you want to debug your blog page sing ngServe, make sure you do a full run off `npm run scully` and then start `npm run scully:serve`. Then scully will use the generated HTML to fill in the content in your `ng serve` session.
**Extra Extra credits**: If you want to debug your blog page sing ngServe, make sure you do a full run off `npm run scully` and then start `npm run scully:serve`. Then scully will use the generated HTML to fill in the content in your `ng serve` session.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove one "extra"

@jorgeucano jorgeucano self-requested a review April 28, 2020 11:23
@SanderElias SanderElias merged commit fc297c8 into master Apr 28, 2020
@SanderElias SanderElias deleted the sander/injectZoneInBareProject branch April 28, 2020 11:24
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