Skip to content

Add ARP support - #759

Merged
GyulyVGC merged 12 commits into
GyulyVGC:mainfrom
shu-kitamura:add-arp
Mar 22, 2025
Merged

GyulyVGC merged 12 commits into
GyulyVGC:mainfrom
shu-kitamura:add-arp

Conversation

@shu-kitamura

Copy link
Copy Markdown
Contributor

Hello!
I'm shu-kitamura.

I took on the challenge #680.

ARP is listed in the protocol list in the menu.

image

ARP is also shown in the Inspect table.

image

When I click on the ARP row in the Inspect page table, the type of ARP operation is displayed.

image

@GyulyVGC

Copy link
Copy Markdown
Owner

Thank you so much!

I'm pretty busy these days, but I'll review the PR as soon as I get some time. 🙌

@GyulyVGC GyulyVGC added this to the v1.4.0 milestone Mar 16, 2025
@GyulyVGC GyulyVGC added the enhancement New feature, request, or improvement label Mar 16, 2025
@GyulyVGC GyulyVGC linked an issue Mar 16, 2025 that may be closed by this pull request
Comment thread src/networking/manage_packets.rs Resolved

@GyulyVGC GyulyVGC left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

🚢

@GyulyVGC

Copy link
Copy Markdown
Owner

@all-contributors please add @shu-kitamura for code.

@allcontributors

Copy link
Copy Markdown
Contributor

@GyulyVGC

I've put up a pull request to add @shu-kitamura! 🎉

@GyulyVGC
GyulyVGC merged commit 3da9a47 into GyulyVGC:main Mar 22, 2025
@shu-kitamura
shu-kitamura deleted the add-arp branch March 22, 2025 23:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature, request, or improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for ARP

2 participants