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

Serve Linked Assets at the right time in serve #130

Open
wants to merge 1 commit into
base: 0.2
Choose a base branch
from

Conversation

allejo
Copy link
Member

@allejo allejo commented Mar 1, 2023

Summary

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Fixed issues fixes #129

Description

Linked assets end up with a route that matches their parent collection item and therefore, the web server attempts to treat them as a dynamic pageview since that's where the collection item belongs.

For example, documentation/server-administration/player-slots.jpg will match the documentation/server-administration/{docpath} route from the dynamic pageview.

This PR changes the logic so that linked assets are handled first before the dynamic pageview controller has time to kick in.

Check List

  • Added appropriate PhpDoc for modifications
  • Added unit test to ensure fix works as intended

@allejo allejo added Bug: Fix A tag for PRs to signify it's fixing a bug Core: Dev Server Relating to the internal development server used for the 'serve' command labels Mar 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: Fix A tag for PRs to signify it's fixing a bug Core: Dev Server Relating to the internal development server used for the 'serve' command
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant