-
Notifications
You must be signed in to change notification settings - Fork 38
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
Upgrade rest package to use latest aries deps #215
Comments
Hey @dblane-digicatapult! Thanks for opening this issue. It's been on our todo list for a long time to update the rest wrapper, but just haven't got around to it. I just finished wrapping up the updates for push-notifications, react-hooks, redux-store, so if we can also get the rest package to 0.4.0 that would be awesome. So, definitely would like you to proceed with the work. 👍 |
Hi @TimoGlastra. I'm currently working with @dblane-digicatapult on this and had a couple of questions. Specifically whether this should use It seems to me that the latter is clearly the path forward, but this would cause breaking changes to the API presented by the Basically are you happy for me to go straight to |
I'd recommend to update the REST API to use askar, indy-vdr and anoncreds. We're thinking of deprecating the Indy SDK and this would be a good step in that direction. Optionally you could make it configurable, but I'm not sure that would be worth the effort.
This is totally fine! The API of the framework is also constantly changing and as long as we are not at 1.0 yet, I don't want to get into technical debt of keeping the API the same. There's so much moving parts in this industry :)
💯% yes! |
Fabulous @TimoGlastra, working towards a PR currently. Hopefully have something for you to look at in the next few days. Thanks for the guidance |
@dblane-digicatapult @TimoGlastra will be update 0.4.0 also based on tsoa? |
We are interested in upgrading the rest package to use the latest available versions of
aries-framework/core
andaries-framework/node
. Our primary driver for this is because as far as we can tell earlier versions do not support the abstraction of hyperledger indy. We believe that it is important to display a variety of anoncreds registries and would benefit other users of this REST API.We have read your contributing guidelines and will conform to the standards set out within.
Can you please indicate if you would like us to proceed with this work or if you have other plans as to why we should not proceed?
The text was updated successfully, but these errors were encountered: