-
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
fix: Base validation fetchStatusList2021 #299
Conversation
DaevMithran
commented
Jul 4, 2023
•
edited
edited
- fetchStatusList2021 now does a basic validation resource content
- minor bug fix in returning error in catch block, the error should be a template literal instead of assigning it directly
No, error of type unknown shouldn't be type casted to string, especially throwable interfaces with trivial interpolation behaviour. Plus, what's the reasoning behind "should"? I see value in fetch validation, so approved. |
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.
@Eengineer1 without typecasting to a string the error is not returned at all. I'm just getting error as an empty object |
## [3.4.2-develop.1](3.4.1...3.4.2-develop.1) (2023-07-04) ### Bug Fixes * Base validation fetchStatusList2021 ([#299](#299)) ([208dde3](208dde3))
🎉 This PR is included in version 3.4.2-develop.1 🎉 The release is available on: Your semantic-release bot 📦🚀 |
## [3.5.0](3.4.1...3.5.0) (2023-07-10) ### Features * Include fetchRemoteContexts in verify apis ([#305](#305)) ([ae6632f](ae6632f)) * Merge Down ([#304](#304)) ([3037687](3037687)) * Support custom verification policies && domain ([#287](#287)) ([403ec16](403ec16)) * Support statusOptions in cheqdCheckCredentialStatus ([#303](#303)) ([14c4f8c](14c4f8c)) ### Bug Fixes * Base validation fetchStatusList2021 ([#299](#299)) ([208dde3](208dde3)) * Make verification result aligned with upstream [DEV-2827] ([#302](#302)) ([eddd11f](eddd11f))
🎉 This PR is included in version 3.5.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |