MetaMask/snaps-registry

[New Snap] SCI

Closed this issue · 1 comments

Checklist

All items in the list below needs to be satisfied.

  • Is the Snap repository publicly accessible and linked in this ticket? https://github.com/sci-domains/snaps
  • Is the Snap distributed on npm and linked in this ticket? https://www.npmjs.com/package/@secure-ci/snap
  • Has an audit been performed and the audit report attached or linked in this issue? [Audit relaxed]
  • Is a complete list of discovered vulnerabilities from the audit documented in this issue?
  • For vulnerabilities that have been deemed necessary to be addressed, are the links to the fixes attached to this issue?
  • For vulnerabilities that have been deemed not necessary to be addressed, is a reason for each of them documented in this issue?
  • The corresponding pull request in this repo has been merged.
  • The Audit has been skipped for this Snap.

Code review:

  • Panel uses text() repeatedly instead of row() [Not a Blocker]

Version to be allowListed: 0.0.12