Skip to content
This repository has been archived by the owner on Nov 29, 2023. It is now read-only.

feat: outbound message in handle message #20

Merged
merged 3 commits into from
Jul 5, 2023

Conversation

dbluhm
Copy link
Contributor

@dbluhm dbluhm commented Jul 5, 2023

This updates to align with changes made upstream in: openwallet-foundation/acapy#2170

And some other minor tweaks.

@swcurran
Copy link
Contributor

swcurran commented Jul 5, 2023

Thanks, Daniel!

This updates to align with changes made upstream in:

openwallet-foundation/acapy#2170

Signed-off-by: Daniel Bluhm <[email protected]>
@dbluhm dbluhm force-pushed the feature/outbound-message-changes branch from 716db6c to 3e10830 Compare July 5, 2023 16:31
@dbluhm
Copy link
Contributor Author

dbluhm commented Jul 5, 2023

This PR doesn't yet account for needing to point to a newer version of ACA-Py, though; I'll defer to the maintainers' preferences for when and how to make that cut-over.

@swcurran
Copy link
Contributor

swcurran commented Jul 5, 2023

Daniel — could you add a comment to the README — something like this:

Deployments of the main branch of this repository must be used with ACA-Py artifacts created **after** [ACA-Py PR #2170](https://github.com/hyperledger/aries-cloudagent-python/pull/2170)). If you are using an earlier ACA-Py release (e.g., version 0.8.2 and earlier), you **MUST** use the v0.0.1 tag of this repository.

dbluhm added 2 commits July 5, 2023 17:32
And remove unused imports

Signed-off-by: Daniel Bluhm <[email protected]>
@shaangill025 shaangill025 self-requested a review July 5, 2023 18:05
@swcurran swcurran merged commit c9f0ae6 into bcgov:main Jul 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants