Skip to content

refactor(metadata): move getProxiedUri to shared/lib - #1396

Open
askov wants to merge 1 commit into
solana-foundation:masterfrom
hoodieshq:refactor/move-get-proxied-uri-to-shared
Open

askov wants to merge 1 commit into
solana-foundation:masterfrom
hoodieshq:refactor/move-get-proxied-uri-to-shared

Conversation

@askov

@askov askov commented Oct 13, 2026

Copy link
Copy Markdown
Collaborator

Description

Moves getProxiedUri from the metadata feature to app/shared/lib/metadata-proxy.ts.

Changes

  • Moves getProxiedUri to app/shared/lib/metadata-proxy.ts, so entities/nft and features/receipt import it from shared.
  • Removes getProxiedUri from the @features/metadata barrel.
  • Removes get-metadata-json.ts and receipt-page.tsx from the boundaries/dependencies cleanup list in eslint.config.mjs.

Type of change

  • Other (please describe): refactor

Screenshots

No UI change.

Testing

Related Issues

Closes HOO-2080

Checklist

  • My code follows the project's style guidelines
  • I have updated documentation as needed
  • I have run build:info script to update build information

Additional Notes

The nft entity and the receipt feature imported getProxiedUri from the metadata feature.
The move removes both files from the boundaries/dependencies cleanup list.
@vercel

vercel Bot commented Oct 13, 2026

Copy link
Copy Markdown

@askov is attempting to deploy a commit to the Solana Foundation Team on Vercel.

A member of the Team first needs to authorize it.

@greptile-apps

greptile-apps Bot commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

[Medium impact] The PR appears safe to merge.

Summary

Moves getProxiedUri into app/shared/lib/metadata-proxy.ts without changing its behavior.

  • The shared metadata URL helper now lives outside the metadata feature.

Reviews (1) · Last reviewed commit: "refactor(metadata): move getProxiedUri t..." · Reviewed by Greptile

@vercel

vercel Bot commented Oct 13, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
explorer Ready Ready Preview Oct 13, 2026 11:20am UTC

Request Review

This branch was successfully deployed

1 active deployment
Preview — b893150d Deployed Oct 13, 2026 by vercel[bot]
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.

1 participant