Skip to content

docs: add JSONViewerTool to useful links - #449

Closed
coderaviverma wants to merge 1 commit into
RetroAchievements:mainfrom
coderaviverma:codex/add-json-viewer-tool
Closed

coderaviverma wants to merge 1 commit into
RetroAchievements:mainfrom
coderaviverma:codex/add-json-viewer-tool

Conversation

@coderaviverma

Copy link
Copy Markdown

Summary

  • add JSONViewerTool as another browser-based option for viewing and formatting raw achievement JSON
  • preserve the existing jsonviewer.stack.hu recommendation

Validation

  • git diff --check passes
  • both the current docs page and the proposed HTTPS destination return HTTP 200
  • no overlapping open pull request was found
  • the full VitePress build could not run locally because npm ci stopped with ENOSPC before dependencies were installed

Disclosure

I am the maker of JSONViewerTool and am affiliated with the proposed resource. Please include it only if it is useful for RetroAchievements contributors; editorial acceptance is entirely up to the maintainers.

@coderaviverma

Copy link
Copy Markdown
Author

After re-reviewing the page and the campaign’s earlier audit, I agree this adds a redundant alternative beside a working viewer without enough additional value. I’m withdrawing it rather than asking maintainers to review a promotional-only change. Sorry for the noise.

@greptile-apps

greptile-apps Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Low impact] This documentation change appears safe to merge.

Summary

Adds JSONViewerTool as another JSON viewer while keeping the existing recommendation. No actionable issues found.

  • The Useful Links page offers another raw achievement viewer.

Reviews (1) · Last reviewed commit: "docs: add JSONViewerTool to useful links" · Reviewed by Greptile

@wescopeland

Copy link
Copy Markdown
Member

I guess spamming PRs on open source repos is one way to get product exposure.

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