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

Update NFT Error messages to be more descriptive #232

Closed
joshuahannan opened this issue Sep 9, 2024 · 0 comments · Fixed by #233
Closed

Update NFT Error messages to be more descriptive #232

joshuahannan opened this issue Sep 9, 2024 · 0 comments · Fixed by #233
Assignees

Comments

@joshuahannan
Copy link
Member

Works towards onflow/docs#795

Issue To Be Solved

Error messages for panics in transactions and contracts are not very helpful to end-users.
They should be updated to be more descriptive

Suggest A Solution

Go through all the standard contracts, transactions, and scripts to update them with good error messages. Error messages could contain:

  • Contract name and function name if coming from a contract
  • Description of literal error
  • Description of what high-level reason might be causing the error
  • Suggestion for fixing it if relevant
@joshuahannan joshuahannan self-assigned this Sep 9, 2024
@joshuahannan joshuahannan moved this to 🏗 In Progress in 🌊 Flow 4D Sep 9, 2024
@joshuahannan joshuahannan changed the title Update Error messages to be more descriptive Update NFT Error messages to be more descriptive Sep 10, 2024
@github-project-automation github-project-automation bot moved this from 👀 In Review to ✅ Done in 🌊 Flow 4D Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant