search

labscommunity / arweave-wallet-kit

A React library for interaction with different Arweave wallets
https://docs.arweavekit.com/wallets/wallet-kit
MIT License
19 stars 11 forks source link

question(docs): update `permissions` App config to reflect current codebase? #9

Open crookse-cl opened 11 months ago

crookse-cl commented 11 months ago

Summary

While adding in the ArweaveWalletKit component without passing in the permissions config, I noticed these default permissions were set: https://github.com/labscommunity/arweave-wallet-kit/blob/4249701afddae9457f8323e18ea3817c8d462f8d/src/components/Provider.tsx#L99

Question

Should the docs show the defaults that are set in the code or should the code default to an empty array?

7i7o commented 11 months ago

IMO we should update the docs to show the defaults set in the code