Feature/buttons

[HerrKanin]

Added "back to dashboard" buttons on createincident, incidents and viewincident

Summary by CodeRabbit

• New Features
  • Added "Back to dashboard" navigation links across incident creation, incidents list, and incident detail pages to streamline navigation back to the main dashboard.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Three HTML templates received "Back to dashboard" navigation links with matching pill-shaped button styling. The incidents template also updated the incident detail navigation URL path from /incident/ to /incidents/. No form functionality or page logic was modified.

Changes

Cohort / File(s)	Summary
Template Navigation UI src/main/resources/templates/createincident.html, src/main/resources/templates/incidents.html, src/main/resources/templates/viewincident.html	Added "Back to dashboard" link (/dashboard) with consistent .back-dashboard-btn CSS styling (pill-shaped button with gray default, green hover state). incidents.html also updated incident detail URL from /incident/${id} to /incidents/${id}.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• Feature/view incident #71: Modifies the same templates (incidents.html, viewincident.html) to add "Back to dashboard" link and updates incident navigation to /incidents/{id}.
• Created HTML-pages for incidents (handler&admin) and createincident(r… #56: Originally added these three template pages; this PR updates them with consistent navigation styling and routing refinements.
• Refactor incident creation flow and dashboard links #70: Changes the same Thymeleaf templates and navigation routing between /dashboard, /create-incident, and incident detail URLs.

Poem

🐰 A dashboard button hops with glee,
Back-to-home for all to see!
Green on hover, pill so neat,
Navigation made complete! 🏠✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'Feature/buttons' is vague and non-descriptive, using generic terminology that doesn't convey meaningful information about the specific changes made.	Use a more descriptive title such as 'Add back-to-dashboard navigation buttons to incident views' to clearly summarize the main change.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feature/buttons

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}