Skip to content

N°9340 - fix: Align FiberChannelInterface and LogicalInterface with PhysicalInterface naming#784

Open
Hipska wants to merge 1 commit intoCombodo:developfrom
Super-Visions:feat/interface/naming
Open

N°9340 - fix: Align FiberChannelInterface and LogicalInterface with PhysicalInterface naming#784
Hipska wants to merge 1 commit intoCombodo:developfrom
Super-Visions:feat/interface/naming

Conversation

@Hipska
Copy link
Contributor

@Hipska Hipska commented Dec 8, 2025

Base information

Question Answer
Related to a SourceForge thread / Another PR / Combodo ticket? N/A
Type of change? Translations

Objective

Align the naming of FiberChannelInterface and LogicalInterface interfaces to be consistent with PhysicalInterface naming. This will make ordering more logical when listing network interfaces.

Proposed solution

Add missing dictionary entries.

No unit test as it are just dictionary items.

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have tested all changes I made on an iTop instance
  • I have added a unit test, otherwise I have explained why I couldn't
  • Is the PR clear and detailed enough so anyone can understand digging in the code?

Copy link
Contributor

@xtophe38 xtophe38 left a comment

Choose a reason for hiding this comment

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

That makes sense.

@jf-cbd
Copy link
Member

jf-cbd commented Feb 27, 2026

Accepted in functional review.
To make it easier to review, would it be possible to add before/after screenshots or description please ?

@jf-cbd jf-cbd moved this from First review needed to Pending technical review in Combodo PRs dashboard Feb 27, 2026
@jf-cbd jf-cbd changed the title fix: Align FiberChannelInterface and LogicalInterface with PhysicalInterface naming N°9340 - fix: Align FiberChannelInterface and LogicalInterface with PhysicalInterface naming Feb 27, 2026
@Hipska
Copy link
Contributor Author

Hipska commented Feb 27, 2026

Well, the naming will be Device Interface instead of Interface Device.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Pending technical review

Development

Successfully merging this pull request may close these issues.

3 participants