-
Notifications
You must be signed in to change notification settings - Fork 2
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
Imported error classes from sdk #38
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks fine, but you need to update your linting settings in visual studio code.
@@ -4673,4 +4673,4 @@ zip-stream@^4.1.0: | |||
dependencies: | |||
archiver-utils "^3.0.4" | |||
compress-commons "^4.1.2" | |||
readable-stream "^3.6.0" | |||
readable-stream "^3.6.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why is this missing?
Maybe in your settings you don't include line break at the bottom.
|
||
export declare class SignatureMalformedError extends MessageError {} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why remove this?
I also noticed that the linting isnt working can we try to fix this? |
fixes KILTProtocol/ticket#3077
The error classes from the extension now extend the error class from the sdk. They should be now an instance of the sdk error class. Also the error classes from the sdk are imported and are not implemented in the extension again.
I first imported the class directly from '@kiltprotocol/utils', where the SDKError.ts file is located, but this resulted in the issue of having different instances of the classes.
Checklist: