Skip to content
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

net/frr: Integrate layout_partials/base_bootgrid_table #4494

Merged
merged 9 commits into from
Jan 28, 2025

Conversation

Monviech
Copy link
Member

@Monviech Monviech commented Jan 22, 2025

Requires: opnsense/core#8187

Will be one commit per changed volt page.

  • ospf.volt
  • ospf6.volt
  • bgp.volt
  • bfd.volt
  • static.volt
  • Rewrite help texts

@Monviech Monviech force-pushed the frr-bootgrid-partial branch from 1e86ed3 to 4a5feb4 Compare January 23, 2025 10:50
@Monviech Monviech force-pushed the frr-bootgrid-partial branch from 75295c0 to ae57a16 Compare January 23, 2025 12:58
@Monviech Monviech marked this pull request as ready for review January 23, 2025 13:24
@Monviech Monviech added the cleanup Low impact changes label Jan 23, 2025
@Monviech
Copy link
Member Author

As discussed with @AdSchellevis I will also rework the help texts based on the ones I already wrote for the documentation:

https://github.com/opnsense/docs/blob/master/source/manual/dynamic_routing.rst

@Monviech
Copy link
Member Author

@AdSchellevis I'm done here for now.

Copy link
Member

@AdSchellevis AdSchellevis left a comment

Choose a reason for hiding this comment

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

Looks good, one remark, but that's more a know issue reported in opnsense/lang#68. Let's merge this

(because it will take more than one hop) and to listen on the multihop port (4784).
When using multi-hop mode echo-mode will not work (see RFC 5883 section 3).
</help>
<help>Multihop tells the BFD daemon that we should expect packets with TTL less than 254 (because it will take more than one hop) and to listen on the multihop port (4784). When using multi-hop mode echo-mode will not work (see RFC 5883 section 3).</help>
Copy link
Member

Choose a reason for hiding this comment

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

The question is if this is worth the effort as it will break translations (https://github.com/opnsense/lang/blob/fca8fe48bdba970f4560a8c5874e660d4ba53ddb/en_US.pot#L47944-L47951)

Padding unfortunately is still problematic (opnsense/lang#68), we might want to seek a solution for that, but that also means we need to change the existing translations to keep the identifiers matching.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cleanup Low impact changes
Development

Successfully merging this pull request may close these issues.

2 participants