Skip to content

feat(traverseapparoutes): make sure there is allwyas an root Route /#266

Merged
jorgeucano merged 1 commit intomasterfrom
sander/enforceRootRoute
Feb 3, 2020
Merged

feat(traverseapparoutes): make sure there is allwyas an root Route /#266
jorgeucano merged 1 commit intomasterfrom
sander/enforceRootRoute

Conversation

@SanderElias
Copy link
Copy Markdown
Contributor

@SanderElias SanderElias commented Feb 3, 2020

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?

if there is no / route in the router, the index.html is the original version, and not rendered by Scully

Issue Number: N/A

What is the new behavior?

The / route will always be rendered unless explicitly ignored by route configuration.

Does this PR introduce a breaking change?

  • Yes
  • No

@jorgeucano jorgeucano merged commit 8954b8d into master Feb 3, 2020
@jorgeucano jorgeucano deleted the sander/enforceRootRoute branch February 3, 2020 14:01
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