Skip to content
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

Set event handler after receiving acknowledgement from Grandeur. #25

Open
instanceofMA opened this issue Feb 18, 2021 · 0 comments
Open
Assignees
Labels
bug Something isn't working P1 🔥 Need to fix this in next iteration

Comments

@instanceofMA
Copy link
Contributor

Right now, we call subscriptions.on() right as we send subscription request message to Grandeur. The request could get rejected. Therefore, we should call it after we receive acknowledgement from Grandeur. For that, we can add a task which would schedule a function calling subscriptions.on(). This can be done using a lambda function.

But our Callback class doesn't accept a lambda functions as of now. So we need to redesign the Callback to accept lambda functions.

@instanceofMA instanceofMA added the bug Something isn't working label Feb 18, 2021
@instanceofMA instanceofMA changed the title Set event handler after receiving acknowledgement from Grandeur. Set event handler after receiving acknowledgement from Grandeur. 1🔥 Feb 22, 2021
@instanceofMA instanceofMA changed the title Set event handler after receiving acknowledgement from Grandeur. 1🔥 1🔥 Set event handler after receiving acknowledgement from Grandeur. Feb 22, 2021
@instanceofMA instanceofMA changed the title 1🔥 Set event handler after receiving acknowledgement from Grandeur. Set event handler after receiving acknowledgement from Grandeur. Feb 22, 2021
@moizhusnain moizhusnain added P1 🔥 Need to fix this in next iteration and removed P1 🔥 Need to fix this in next iteration labels Feb 22, 2021
@instanceofMA instanceofMA added the P1 🔥 Need to fix this in next iteration label Feb 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working P1 🔥 Need to fix this in next iteration
Projects
None yet
Development

No branches or pull requests

2 participants