Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade browser wallet (js) deps #2977

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mDuo13
Copy link
Collaborator

@mDuo13 mDuo13 commented Jan 31, 2025

Upgrades xrpl.js and Vite dependencies in the "Build a Browser Wallet in JavaScript" code sample. This was a bit more involved than just bumping version dependencies.

  • Changed some usage to match API v2 formats used in xrpl.js 4.x
  • Improved error handling of the sample wallet (now it alerts you if you forgot to edit the .env file)
  • Removed the special configs & dependencies that used to be necessary for using xrpl.js in Vite. (It works out of the box now.)
  • Split some longer lines into two to make the code display nicer in the dev site's code blocks.

This PR supersedes #2912 and #2961.

Copy link

New, updated, and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/[email protected] 🔁 npm/[email protected] unsafe 0 46.6 kB indutny
npm/[email protected] Transitive: eval +20 499 kB ljharb
npm/[email protected] None 0 175 kB peterolson
npm/[email protected] 🔁 npm/[email protected] None 0 10.7 kB ljharb
npm/[email protected] 🔁 npm/[email protected] None +1 23.5 kB ljharb
npm/[email protected] None +2 108 kB feross
npm/[email protected] None 0 4.98 kB isaacs
npm/[email protected] 🔁 npm/[email protected] None 0 56 kB ljharb
npm/[email protected] None 0 284 kB mikemcl
npm/[email protected] 🔁 npm/[email protected] None +1 41.4 kB indutny
npm/[email protected] 🔁 npm/[email protected] None 0 75.8 kB motdotla
npm/[email protected] 🔁 npm/[email protected] None 0 120 kB indutny
npm/[email protected] Transitive: environment, filesystem +4 1.91 MB interledger
npm/[email protected] None 0 794 B ehsalazar
npm/[email protected] None 0 54.5 kB ljharb
npm/[email protected] 🔁 npm/[email protected] None 0 57.1 kB ai
npm/[email protected] 🔁 npm/[email protected] None 0 24.7 kB ljharb
npm/[email protected] 🔁 npm/[email protected] None 0 6.37 kB alexeyraspopov
npm/[email protected] 🔁 npm/[email protected] None 0 202 kB ai
npm/[email protected] Transitive: environment, filesystem +2 2.21 MB fanatid
npm/[email protected] 🔁 npm/[email protected] None 0 140 kB 7rulnik
npm/[email protected] 🔁 npm/[email protected] None 0 3.35 MB antfu, patak, soda, ...2 more

🚮 Removed packages: npm/@jridgewell/[email protected], npm/@rollup/[email protected], npm/@rollup/[email protected], npm/@types/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected], npm/[email protected]

View full report↗︎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant