Skip to content

fix: style unification (back to overview button) - #229

Merged
HardMax71 merged 1 commit into
mainfrom
fix/provision-issues
Feb 24, 2026
Merged

fix: style unification (back to overview button)#229
HardMax71 merged 1 commit into
mainfrom
fix/provision-issues

Conversation

@HardMax71

@HardMax71 HardMax71 commented Feb 24, 2026

Copy link
Copy Markdown
Owner

Summary by cubic

Unifies the “Back to Overview” navigation across Grafana dashboards and renames the overview UID to integr8scode-overview for consistent, predictable links. This standardizes title, tooltip, and same-tab behavior to prevent broken or mismatched links.

  • Refactors

    • Added the “Back to Overview” link where missing and normalized existing links.
    • Standardized: title “Back to Overview”, tooltip “Return to main dashboard”, keepTime=true, targetBlank=false, icon “external link”.
    • Pointed all links to /d/integr8scode-overview; renamed System Overview UID from integr8scode to integr8scode-overview and set id=null for provisioning.
    • Quoted alerting folder name 'Integr8sCode' in alerting.yml.
  • Migration

    • Grafana will create a new System Overview due to the UID change; remove the old integr8scode dashboard if it remains.
    • Update any bookmarks or internal links to /d/integr8scode-overview.

Written for commit 6950dc8. Summary will update on new commits.

Summary by CodeRabbit

  • Chores
    • Added "Back to Overview" navigation links across dashboards for improved navigation between related dashboards
    • Updated dashboard references to point to unified overview page
    • Minor configuration formatting refinement

Copilot AI review requested due to automatic review settings February 24, 2026 11:58
@HardMax71
HardMax71 merged commit a0565d9 into main Feb 24, 2026
9 checks passed
@HardMax71
HardMax71 deleted the fix/provision-issues branch February 24, 2026 11:58
@coderabbitai

coderabbitai Bot commented Feb 24, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 66345ec and 6950dc8.

📒 Files selected for processing (12)
  • backend/grafana/provisioning/alerting/alerting.yml
  • backend/grafana/provisioning/dashboards/coordinator-execution.json
  • backend/grafana/provisioning/dashboards/dlq-monitoring.json
  • backend/grafana/provisioning/dashboards/event-replay.json
  • backend/grafana/provisioning/dashboards/event-stream-monitoring.json
  • backend/grafana/provisioning/dashboards/http-middleware.json
  • backend/grafana/provisioning/dashboards/integr8scode.json
  • backend/grafana/provisioning/dashboards/kafka-events-monitoring.json
  • backend/grafana/provisioning/dashboards/kubernetes-pods.json
  • backend/grafana/provisioning/dashboards/notifications.json
  • backend/grafana/provisioning/dashboards/rate-limiting-dashboard.json
  • backend/grafana/provisioning/dashboards/security-auth.json

📝 Walkthrough

Walkthrough

This PR standardizes Grafana dashboard provisioning by renaming the overview dashboard UID from "integr8scode" to "integr8scode-overview" and adding consistent "Back to Overview" navigation links across multiple dashboards. Minor formatting adjustment to alerting configuration.

Changes

Cohort / File(s) Summary
YAML Formatting
backend/grafana/provisioning/alerting/alerting.yml
Quote formatting applied to folder value string (unquoted to quoted), no functional impact.
Dashboard ID/UID Rename
backend/grafana/provisioning/dashboards/integr8scode.json
Dashboard id changed from 1 to null; uid renamed from "integr8scode" to "integr8scode-overview".
Link URL Updates (Existing)
backend/grafana/provisioning/dashboards/dlq-monitoring.json, rate-limiting-dashboard.json
Updated link URLs from /d/integr8scode to /d/integr8scode-overview.
New Navigation Links
backend/grafana/provisioning/dashboards/coordinator-execution.json, event-replay.json, http-middleware.json, kubernetes-pods.json, notifications.json, security-auth.json
Added top-level "links" array containing a single "Back to Overview" link pointing to /d/integr8scode-overview with standard display properties.
First Link Property Updates
backend/grafana/provisioning/dashboards/event-stream-monitoring.json, kafka-events-monitoring.json
Updated first link object: targetBlank (truefalse), title ("System Overview""Back to Overview"), tooltip text, and url (/d/integr8scode/d/integr8scode-overview).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • #224: Modifies Grafana dashboard provisioning with UID/identifier and navigation link URL changes across multiple dashboards
  • #227: Updates the same integr8scode.json dashboard file with related provisioning changes
  • #202: Modifies alerting.yml provisioning configuration alongside dashboard changes

Poem

🐰 A Dashboard Odyssey

Through JSON paths, a rabbit hops with glee,
Each dashboard now links home with a "Back to Overview" spree,
The overview renamed, more grand and bright,
Nine dashboards connected—navigation made just right! 🎯✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/provision-issues

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

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

@sonarqubecloud

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 12 files

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request standardizes navigation across Grafana dashboards by unifying the "Back to Overview" button implementation. The main overview dashboard's UID is changed from integr8scode to integr8scode-overview, and all dashboard links are updated to reference the new UID.

Changes:

  • Updated main overview dashboard UID from integr8scode to integr8scode-overview
  • Added "Back to Overview" links to 7 dashboards that previously lacked them
  • Updated "Back to Overview" links in 4 existing dashboards to use consistent styling and the new UID
  • Standardized link properties across all dashboards (title, tooltip, targetBlank, icon)

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated no comments.

Show a summary per file
File Description
integr8scode.json Changed main overview dashboard UID to "integr8scode-overview" and reset ID to null
security-auth.json Added new "Back to Overview" link section
notifications.json Added new "Back to Overview" link section
kubernetes-pods.json Added new "Back to Overview" link section
http-middleware.json Added new "Back to Overview" link section
event-replay.json Added new "Back to Overview" link section
coordinator-execution.json Added new "Back to Overview" link section
rate-limiting-dashboard.json Updated existing link URL to new UID
kafka-events-monitoring.json Updated existing link with consistent title, tooltip, targetBlank, and new URL
event-stream-monitoring.json Updated existing link with consistent title, tooltip, targetBlank, and new URL
dlq-monitoring.json Updated existing link URL to new UID
alerting.yml Added quotes around folder name (cosmetic formatting)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants