Skip to content

chore(demo-smart): provide Grafana links in the expected grafanaLink format - #1897

Merged
clara-layus-cc merged 1 commit into
masterfrom
fix/demo-smart-grafana-context
Nov 12, 2026
Merged

clara-layus-cc merged 1 commit into
masterfrom
fix/demo-smart-grafana-context

Conversation

@clara-layus-cc

Copy link
Copy Markdown
Contributor

Fixes #1543

What does this PR do?

  • Replace the consoleGrafanaLink and grafanaBaseLink string params with the grafanaLink object ({ base, console }) expected by smart components such as cc-tile-metrics in the demo-smart "App" contexts,
  • Fix malformed Grafana URLs (https:grafana… instead of https://grafana…) in one addon context.

How to review?

  • Check the commits,
  • Check the preview
  • 1 reviewer should be enough.

@clara-layus-cc clara-layus-cc self-assigned this Nov 10, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔎 A preview has been automatically published : https://clever-components-preview.cellar-c2.services.clever-cloud.com/fix/demo-smart-grafana-context/index.html.

This preview will be deleted once this PR is closed.

@clara-layus-cc
clara-layus-cc force-pushed the fix/demo-smart-grafana-context branch from 36072a7 to 517926a Compare November 12, 2026 14:24
@clara-layus-cc
clara-layus-cc merged commit db64525 into master Nov 12, 2026
7 checks passed
@clara-layus-cc
clara-layus-cc deleted the fix/demo-smart-grafana-context branch November 12, 2026 14:33
@github-actions

Copy link
Copy Markdown
Contributor

🔎 The preview has been automatically deleted.

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.

demo-smart: context related to grafana doesn't match the expected format

2 participants