Skip to content

Update NEAR Protocol#1489

Merged
Montoya merged 1 commit into
mainfrom
update-near-protocol
May 28, 2026
Merged

Update NEAR Protocol#1489
Montoya merged 1 commit into
mainfrom
update-near-protocol

Conversation

@ajeetd
Copy link
Copy Markdown
Collaborator

@ajeetd ajeetd commented May 26, 2026

Updates the website URLs for the npm:@near-snap/plugin (NEAR Protocol) registry entry.

  • Current state: metadata.website points to https://my.herewallet.app/ and metadata.author.website points to https://www.herewallet.app/.
  • Change: Both URLs now point to https://hot-labs.org/, the correct upstream site for the snap publisher.

Context: https://consensys.slack.com/archives/CBHSSFEN5/p1779807079111949

Registry-data-only change — no schema, code, or version/checksum changes. yarn lint:misc --check and yarn lint:changelog pass locally.


Note

Low Risk
Registry metadata URL corrections only; no functional, security, or package integrity changes.

Overview
Updates registry metadata only for npm:@near-snap/plugin (NEAR Protocol): both metadata.website and metadata.author.website now point to https://hot-labs.org/ instead of the previous HERE Wallet / herewallet.app URLs.

No snap version, checksum, schema, or application code changes.

Reviewed by Cursor Bugbot for commit ba30c72. Bugbot is set up for automated code reviews on this repo. Configure here.

@ajeetd ajeetd requested review from a team and Montoya as code owners May 26, 2026 15:28
@Montoya
Copy link
Copy Markdown
Contributor

Montoya commented May 26, 2026

This is fine for the developer website but is there a webpage that actually functions with the Snap?

mcmire
mcmire previously approved these changes May 26, 2026
Copy link
Copy Markdown

@mcmire mcmire left a comment

Choose a reason for hiding this comment

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

LGTM.

@mcmire mcmire dismissed their stale review May 26, 2026 15:58

Taking this back following Christian's comment.

@ajeetd
Copy link
Copy Markdown
Collaborator Author

ajeetd commented May 26, 2026

This is fine for the developer website but is there a webpage that actually functions with the Snap?

@Montoya unsure. Can find out. Are you suggesting the second link should link to that?

@Montoya
Copy link
Copy Markdown
Contributor

Montoya commented May 26, 2026

@ajeetd yes that's my assumption

@ajeetd
Copy link
Copy Markdown
Collaborator Author

ajeetd commented May 26, 2026

@Montoya - Jeremy says it is the same link (https://consensys.slack.com/archives/CBHSSFEN5/p1779807079111949) but I could reach out to hot-labs to ask to confirm they are expecting it to work this way. I don't have a direct contact but can ask through support (probably worth me writing to them regardless). The current link is already broken.

@Montoya
Copy link
Copy Markdown
Contributor

Montoya commented May 28, 2026

lgtm then @ajeetd @mcmire

Copy link
Copy Markdown

@mcmire mcmire left a comment

Choose a reason for hiding this comment

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

Re-approving.

@Montoya Montoya added this pull request to the merge queue May 28, 2026
Merged via the queue into main with commit f363135 May 28, 2026
23 checks passed
@Montoya Montoya deleted the update-near-protocol branch May 28, 2026 17:20
@ajeetd
Copy link
Copy Markdown
Collaborator Author

ajeetd commented May 29, 2026

Thanks for merging. FYI - I did contact hot-labs and have not heard back yet. If they want anything changed I will open an new PR.

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.

3 participants