-
Notifications
You must be signed in to change notification settings - Fork 77
Description
Issues for the kit version:
- There is no way to change the cluster
- When trying to deposit in the vault the transaction fails with a transaction plan error which will confuse users. There is no actual error message shown:
"Error: The provided transaction plan failed to execute. See thetransactionPlanResultattribute for more details. Note that thecauseproperty is deprecated, and a future version will not set it."
-
The real error of the transaction plan is "Caused by: SolanaError: Attempt to load a program that does not exist" The example is supposed to use a program that is deployed to devnet on first start i assume. But there is no hint to deploy it to local net first or how to switch the cluster
-
There is no error handling for wallet connect when there is an error in the connection:
- Description is a bit confusing for new devs maybe. I think it would be better to explain the project a bit. That its an escrow program and how to use it. The link to the anchor docs returns 404.
-
The old template had a nice view where you could see your current balance.
-
Regarding the wallet selection. I think the Ui is not really usuable for a new project. So its not a great fit for a starter template. It should be some nice drop down imo
-
After deploying to local net sending transactions works. The transaction signature is truncated, so one can not copy it to look at is in the explorer.
- When causing an error like for example not enough sol the actual error is not shown:
- The description of the template probably needs a proper review.
"npm install # Builds program and generates client automatically"
npm install does not build the program and generate the clients for example.
In general i think the old template was much nicer. Showing wallet balance, transactions with direct link to explorer, links to learn more, airdrop, cluster selection, send and recieve, error handling etc. Would be good to get both the old and the new template on par.
