Skip to content

feat!: add DiscoveryItem::user_data method and adjust locally-discovered-nodes example #3215

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

Merged
merged 3 commits into from
Mar 14, 2025

Conversation

ramfox
Copy link
Contributor

@ramfox ramfox commented Mar 3, 2025

Description

Adjusts the locally-discovered-nodes example to use the new user_data and discovery_stream methods.

Renamed LocalSwarmDiscovery to MdnsDiscovery

Breaking Changes

  • changed
    • renamed iroh::discovery::local_swarm_discovery to iroh::discovery::mdns
    • renamed iroh::discovery::local_swarm_discovery::LocalSwarmDiscovery to iroh::discovery::mdns::MdnsDiscovery

Change checklist

  • Self-review.
  • Documentation updates following the style guide, if relevant.

Copy link

github-actions bot commented Mar 3, 2025

Documentation for this PR has been generated and is available at: https://n0-computer.github.io/iroh/pr/3215/docs/iroh/

Last updated: 2025-03-14T00:46:03Z

Copy link

github-actions bot commented Mar 3, 2025

Netsim report & logs for this PR have been generated and is available at: LOGS
This report will remain available for 3 days.

Last updated for commit: afb22e7

@n0bot n0bot bot added this to iroh Mar 3, 2025
@github-project-automation github-project-automation bot moved this to 🏗 In progress in iroh Mar 3, 2025
@ramfox ramfox force-pushed the ramfox/local-discovery-example branch from cb2091c to 3fa7fee Compare March 5, 2025 21:21
@Frando
Copy link
Member

Frando commented Mar 10, 2025

Looks good! Needs a Breaking changes note for the rename to Mdns.

@ramfox ramfox requested a review from Frando March 10, 2025 13:49
@ramfox ramfox changed the title feat: add DiscoveryItem::user_data method and adjust locally-discovered-nodes example feat!: add DiscoveryItem::user_data method and adjust locally-discovered-nodes example Mar 12, 2025
@ramfox ramfox force-pushed the ramfox/local-discovery-example branch from bf233d8 to 11280b5 Compare March 14, 2025 00:44
@ramfox ramfox added this pull request to the merge queue Mar 14, 2025
Merged via the queue into main with commit f6b5f5c Mar 14, 2025
25 of 26 checks passed
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in iroh Mar 14, 2025
@ramfox ramfox deleted the ramfox/local-discovery-example branch March 14, 2025 02:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

2 participants