Skip to content

[8.x] Extended link to include Product and Category Runway models - #213

Merged
royduin merged 4 commits into
masterfrom
feature/runway-model-links
Aug 11, 2026
Merged

[8.x] Extended link to include Product and Category Runway models#213
royduin merged 4 commits into
masterfrom
feature/runway-model-links

Conversation

@kevinmeijer97

@kevinmeijer97 kevinmeijer97 commented Jul 29, 2026

Copy link
Copy Markdown
Member

Added an extend to the link field to register the Product and Category Runway models so they can be used in the link field.

Normally Runway models would automatically be registered, but since the Product and Category models aren't your default Runway models this was not possible so we had to register and resolve the urls like this instead.

Screenshot 2026-07-29 at 14 35 20

BobWez98
BobWez98 previously approved these changes Jul 29, 2026
indykoning
indykoning previously approved these changes Jul 29, 2026
@kevinmeijer97 kevinmeijer97 changed the title Extended link to include Product and Category Runway models [8.x] Extended link to include Product and Category Runway models Aug 4, 2026
@royduin

royduin commented Aug 5, 2026

Copy link
Copy Markdown
Member

Related to:


  • Do we still need the determineEntryUrl()?
  • Do we need to raise the versions of Statamic and Runway in the composer.json as this is feature was recently added?
  • With this we can just use $item['url'] everywhere? We don't need things like this:
<a href="{{ isset($item['product']->value()['url']) ? '/'.$item['product']->value()['url'] : $item['url'] }}">

@kevinmeijer97
kevinmeijer97 dismissed stale reviews from indykoning and BobWez98 via f9d21df August 11, 2026 08:52
@kevinmeijer97

Copy link
Copy Markdown
Member Author
  • Yes, for now we still need determineEntryUrl() as it is used for nav trees. Changing this to work with the models is possible but would create some breaking changes. If we really want that i'd suggest to create a separate PR for this, some type of migration script would be nice to have here as well. The same goes for Can we just use $item['url'] everywhere?
  • I bumped the versions to "statamic/cms": "^6.26" & "statamic-rad-pack/runway": "^9.6"

@royduin
royduin merged commit 48d4f20 into master Aug 11, 2026
@royduin
royduin deleted the feature/runway-model-links branch August 11, 2026 11:06
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.

4 participants