Skip to content

fix(examples): subscribe to topic in chat-tokio#3296

Merged
mergify[bot] merged 2 commits intolibp2p:masterfrom
elenaf9:fix-chat-tokio
Jan 4, 2023
Merged

fix(examples): subscribe to topic in chat-tokio#3296
mergify[bot] merged 2 commits intolibp2p:masterfrom
elenaf9:fix-chat-tokio

Conversation

@elenaf9
Copy link
Copy Markdown
Member

@elenaf9 elenaf9 commented Jan 2, 2023

Description

With #3097, subscribing to the floodsub topic in examples/chat-tokio.rs was removed. I assume this was unintentional (cc @umgefahren), so this PR adds it back.

Links to any relevant issues

Most likely the reason why the example did not work in #3217.

Change checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • A changelog entry has been made in the appropriate crates

Copy link
Copy Markdown
Member

@jxs jxs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM Elena, great catch!

Copy link
Copy Markdown
Member

@mxinden mxinden left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@mxinden mxinden added the send-it label Jan 4, 2023
@mxinden
Copy link
Copy Markdown
Member

mxinden commented Jan 4, 2023

@Mergifyio refresh

@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Jan 4, 2023

refresh

✅ Pull request refreshed

@mergify mergify bot merged commit bda1420 into libp2p:master Jan 4, 2023
@elenaf9 elenaf9 deleted the fix-chat-tokio branch January 4, 2023 20:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants