Skip to content

fix(KNO-11924): Fix hardcoded account ID in New Relic dashboard starter kit JSON - #1336

Merged
rachael-t merged 5 commits into
mainfrom
rt-new-relic-json
Mar 5, 2026
Merged

fix(KNO-11924): Fix hardcoded account ID in New Relic dashboard starter kit JSON#1336
rachael-t merged 5 commits into
mainfrom
rt-new-relic-json

Conversation

@rachael-t

@rachael-t rachael-t commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

Description

The dashboard starter kit JSON contained a hardcoded account ID, so any customer importing the JSON would see an error message "We couldn't fetch this data. This widget was added from an account you don't have access to" for every widget, since the queries were pointing at an account they had no access to.

I updated the JSON to replace the hardcoded account ID with 0 as a placeholder and then updated the dashboard starter kit instructions to direct customers to replace the 0 placeholder with their own New Relic account ID before importing.

https://docs-git-rt-new-relic-json-knocklabs.vercel.app/integrations/extensions/new-relic#dashboard-starter-kit

Tasks

KNO-11924

Screenshots

Error messages customers receive if they just copy and paste the JSON without updating the accountId:
Screenshot 2026-03-05 at 11 14 49 AM

@linear

linear Bot commented Mar 5, 2026

Copy link
Copy Markdown

@vercel

vercel Bot commented Mar 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Mar 5, 2026 7:08pm

Request Review

@rachael-t
rachael-t marked this pull request as ready for review March 5, 2026 18:19
@rachael-t
rachael-t requested a review from a team March 5, 2026 18:19
Comment thread content/integrations/extensions/new-relic.mdx Outdated

@cellomatt cellomatt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Left a couple nits

Comment thread content/integrations/extensions/new-relic.mdx Outdated
@rachael-t
rachael-t requested a review from cellomatt March 5, 2026 18:58
@rachael-t
rachael-t merged commit 4cfadfd into main Mar 5, 2026
5 checks passed
@rachael-t
rachael-t deleted the rt-new-relic-json branch March 5, 2026 19:11
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