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

Adding errors for state rent change #268

Closed
wants to merge 2 commits into from
Closed

Conversation

ilblackdragon
Copy link
Member

At this point, keep old errors and just add new ones.

At this point, keep old errors and just add new ones.
@gitpod-io
Copy link

gitpod-io bot commented Mar 18, 2020

Copy link
Contributor

@lexfrl lexfrl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to change
https://github.com/nearprotocol/nearlib/blob/master/gen_error_types.js#L7-L9
to the revision with the new rpc_errors_schema.json and then run to generate new types for errors:
yarn gen_error_types.js

And then yarn build to build lib/**.js files.

@ilblackdragon
Copy link
Member Author

@fckt
yarn gen_error_types.js doesn't work:

$ ls -lh gen_error_types.js                                                                                                  
rw-r--r--  1 cypress  staff   2.4K Mar 18 09:06 gen_error_types.js
$ yarn gen_error_types.js
yarn run v1.21.0
error Command "gen_error_types.js" not found.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

@ilblackdragon
Copy link
Member Author

Added README.md how to update the error types.

@ilblackdragon
Copy link
Member Author

Closing this, instead using #270

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants