Fix: Nav Item Api & Webhook and Functions stay selected

[tushar110302]

Fixed Nav Item Api & Webhook and Functions stay selected like data model settings.
Now when clicked stays selected and deos not loose its selection

Fixes #7573

Screen.Recording.2024-10-13.at.9.43.25.AM.mp4

[greptile-apps]

PR Summary

This pull request modifies the SettingsNavigationDrawerItems component to fix the selection behavior of 'API & Webhooks' and 'Functions' navigation items in the settings menu.

• Added matchSubPages prop to 'API & Webhooks' and 'Functions' items in packages/twenty-front/src/modules/settings/components/SettingsNavigationDrawerItems.tsx
• Ensures these items remain selected when navigating to sub-pages, similar to data model settings
• Addresses issue Settings Nav item doesn't stay selected  #7573 where 'API & Webhooks' menu item lost selection when accessing webhook pages
• Improves user experience by maintaining consistent navigation state across settings sub-sections

_{1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}

[ehconitin]

LGTM! @tushar110302 Thank you for contributing!

[tushar110302]

LGTM! @tushar110302 Thank you for contributing!

Happy to contribute and become a part of it 🙂

[FelixMalfait]

Thanks @tushar110302!
@ehconitin as discussed this works well but I think we should consider inverting the API so that people who add sub-pages to a page that didn't have any don't have to think about changing this setting. Currently the only page where we want to disable this is accounts where we display calendar/email directly

[oss-gg]

Awarding tushar110302: 150 points 🕹️ Well done! Check out your new contribution on oss.gg/tushar110302