Skip to content

[23647] Add documentation for IP mobility #1129

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

[23647] Add documentation for IP mobility #1129

wants to merge 9 commits into from
+67 −11

Conversation

MiguelCompany
Copy link
Member

@MiguelCompany MiguelCompany commented Sep 17, 2025 •
edited by cferreiragonz
Loading

Description

Depends on:

Contributor Checklist

  • Commit messages follow the project guidelines.
  • N/A: Code snippets related to the added documentation have been provided.
  • Documentation tests pass locally.
  • The Pro version badge has been added if the documented feature is exclusive to Fast DDS Pro.
  • N/A: Applicable backports have been included in the description.

Reviewer Checklist

  • The PR has a milestone assigned.
  • The title and description correctly express the PR's purpose.
  • Check contributor checklist is correct.
  • CI passes without warnings or errors.

@MiguelCompany MiguelCompany added this to the v3.4.0 milestone Sep 17, 2025
@MiguelCompany MiguelCompany requested review from richiprosima and removed request for richiprosima September 18, 2025 08:04
@MiguelCompany MiguelCompany requested review from richiprosima and removed request for richiprosima September 18, 2025 08:43
@MiguelCompany MiguelCompany requested review from richiprosima and removed request for richiprosima September 18, 2025 10:05
cferreiragonz
cferreiragonz requested changes Sep 23, 2025
View reviewed changes
Copy link
Contributor

@cferreiragonz cferreiragonz left a comment

Choose a reason for hiding this comment

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

Just a small NIT

@MiguelCompany MiguelCompany marked this pull request as ready for review September 23, 2025 13:35
cferreiragonz
cferreiragonz approved these changes Sep 23, 2025
View reviewed changes
Copy link
Contributor

@cferreiragonz cferreiragonz left a comment

Choose a reason for hiding this comment

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

LGTM with green CI

@cferreiragonz cferreiragonz requested review from richiprosima and removed request for richiprosima September 23, 2025 13:40
cferreiragonz
cferreiragonz previously approved these changes Sep 24, 2025
View reviewed changes
@cferreiragonz cferreiragonz added implementation-pending Marks doc PRs which corresponding implementation is not yet available on Fast DDS and removed ci-pending labels Sep 24, 2025
@MiguelCompany
Refs #23647. Add empty section and reference it in old feature.
e48b3ea 
Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
@MiguelCompany
Refs #23647. Add feature documentation.
ea491b6 
Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
@MiguelCompany
Refs #23647. Update TOC tree.
216ac8c 
Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
@MiguelCompany
Refs #23647. Copy restrictions and notes from Dynamic network interfa…
0533565 
…ces.

Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
@MiguelCompany
Apply suggestions
891e1dc 
Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
@MiguelCompany
Refs #23647. Fix pro badges in DDS TSN.
17ca290 
Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
@MiguelCompany
Refs #23647. Update use cases section.
03eed99 
Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
rsanchez15
rsanchez15 requested changes Sep 24, 2025
View reviewed changes
@MiguelCompany
Refs #23647. Apply suggestions.
439c13f 
Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
@MiguelCompany
Refs #23647. Fix line length.
915daa1 
Signed-off-by: Miguel Company <miguelcompany@eprosima.com>
@MiguelCompany MiguelCompany requested review from rsanchez15 and removed request for rsanchez15 September 24, 2025 12:10
@MiguelCompany MiguelCompany mentioned this pull request Sep 24, 2025
Merged
12 tasks
@MiguelCompany MiguelCompany removed the implementation-pending Marks doc PRs which corresponding implementation is not yet available on Fast DDS label Sep 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rsanchez15 rsanchez15 rsanchez15 approved these changes

@cferreiragonz cferreiragonz cferreiragonz left review comments

Assignees
No one assigned
Labels
fast dds pro ready-to-merge
Projects
None yet
Milestone
v3.4.0
Development

Successfully merging this pull request may close these issues.
3 participants
@MiguelCompany @rsanchez15 @cferreiragonz