FR: don't return node if it's not type heading and has no url

studioespresso / craft-navigate

Navigate makes adding and maintaining menu's and navigations a breeze. Set what types of links a menu may contain, make multi-level navigations, built-in active-state check, custom classes per item.

https://plugins.craftcms.com/navigate

Other

26 stars 7 forks source link

FR: don't return node if it's not type heading and has no url #39

Closed jan-dh closed 3 years ago

jan-dh commented 4 years ago

At this point, we're having a customer removing entries from the CMS without adjusting the navigation. Templating breaks since, the navigation still shows the entry, but without uri. Is it possible to not return entries if the type is entry and it has no valid url?

janhenckens commented 4 years ago

Hey Jan! Valid point.

Here's how I'd like to solve it:

when an entry is removed, we also remove the navigation item
when an entry is disabled, we disabled the navigation item

I have a couple of other fixes in the pipeline for a release, I'll try to add this so it's out soon!

jan-dh commented 4 years ago

Sound good!

janhenckens commented 3 years ago

Out in 2.8.