fix: prevent nested <a> element in NavigationItem

[devanshu0x]

What does this PR do?

Problem

The NavigationItem component renders nested <a> elements due to an additional Link wrapper.

Solution

Instead of using Link, render the badge inside a non interactive <div> with an onClick handler that calls router.push("/bookings/unconfirmed") to preserve the intended navigation without introducing nested interactive elements.

• Fixes NavigationItem renders nested <a> elements when wrapped in Tooltip #27984
• Fixes CAL-7217

Visual Demo

Before

Before.mp4

After

After-2.mp4

Note: After the fix, all the errors and warnings related to nested <a> element have been resolved

Mandatory Tasks (DO NOT REMOVE)

• I have self-reviewed the code (A decent size PR without self-review might be rejected).
• I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. (N/A)
• I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

Steps to Reproduce

• Run the project locally (yarn dx)
• Navigate to a http://localhost:3000/event-types or a page which renders the navigation sidebar
• Ensure sidebar navigation is visible
• Error pops up from Next and in browser console

Expected Behavior (After Fix)

• Repeat the steps above after applying this PR.
• No console error related to nested anchor tag should appear.

---

[cubic-dev-ai]

No issues found across 1 file

[sahitya-chandra]

Can you add a before and after video of the changes?

and always make sure to add these videos in your future prs, otherwise we will not be able to approve your prs.

[devanshu0x]

Can you add a before and after video of the changes?

and always make sure to add these videos in your future prs, otherwise we will not be able to approve your prs.

@sahitya-chandra I have added before and after video of the fix, I will make sure to include relevant visual demos in future prs as well

[devin-ai-integration]

Devin Review found 1 new potential issue.

View 7 additional findings in Devin Review.