-
Notifications
You must be signed in to change notification settings - Fork 0
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
14 implement mediation coordination workflow #18
base: main
Are you sure you want to change the base?
Conversation
@Christiantyemele @IngridPuppet @Hermann-Core @Tekum-Emmanuella @Blindspot22 @chendiblessing Pleas review this PR |
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.
Apart form that, I think you good to go
|
||
// Extract the body from the mediation response | ||
const message: IMessage = mediation_response.as_value(); | ||
// //Extract the body from the mediation response |
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.
Please what is the use of all this comments?
From line 35 - 48
If the implementations are no more needed, then they should be deprecated
@@ -71,11 +83,23 @@ export async function pack_encrypt( | |||
forward: true, | |||
}, | |||
); | |||
return result[0]; | |||
|
|||
// let message = await wrap_in_forward.Message.wrap_in_forward( |
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.
Same here
@Christiantyemele @IngridPuppet @Hermann-Core @Tekum-Emmanuella @Blindspot22 @chendiblessing Pleas review this PR