DOCS: generate smaller tables for feature matrix (2nd try)#3589
Merged
tlimoncelli merged 2 commits intoStackExchange:mainfrom May 29, 2025
Merged
DOCS: generate smaller tables for feature matrix (2nd try)#3589tlimoncelli merged 2 commits intoStackExchange:mainfrom
tlimoncelli merged 2 commits intoStackExchange:mainfrom
Conversation
chicks-net
changed the title
Collaborator
Author
|
GCORE shows up as concurrency-verified in the preview so #3588 is still reflected in the docs. |
tlimoncelli
approved these changes
May 29, 2025
cafferata
reviewed
May 30, 2025
| DomainModifierDname = "[`DNAME`](../language-reference/domain-modifiers/DNAME.md)" | ||
| DomainModifierDnskey = "[`DNSKEY`](../language-reference/domain-modifiers/DNSKEY.md)" | ||
| DualHost = "dual host" | ||
| DualHost = "[dual host](../dual-host.md)" |
Collaborator
There was a problem hiding this comment.
Hi @chicks-net, thanks for the contribution!
Just a quick heads-up:
- The links/paths in the Markdown files currently point to GitHub URLs. This is likely because the new documents (concurrency-verified.md and dual-host.md) haven’t yet been added to the SUMMARY.md file.
- Once they’re included in SUMMARY.md, they’ll be part of the DNSControl documentation site structure, and the links can (and should) reference the documentation site instead.
Let me know if you need any help updating this!
Collaborator
Author
There was a problem hiding this comment.
I was wondering about that. #3579 (comment)
I will try your suggestion while working on #3584 next week. Unless you want this fix as its own PR.
Collaborator
There was a problem hiding this comment.
Thanks, sounds good!
No rush — feel free to include the fix as part of #3584 if that’s more convenient. But if it turns out to be a quick change, a separate PR would be welcome too. Totally up to you.
Let me know if you want a quick review once it’s ready.
This was referenced Jun 7, 2025
Closed
AsifNawaz-cnic
pushed a commit
to centralnicgroup-opensource/rtldev-middleware-dnscontrol
that referenced
this pull request
Jan 19, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
context
#3305 notes that the provider feature table does not display well in the GitBook-based documentation site.
This has also been brought up in a couple of the monthly live calls.
I tried to do this in #3579 but rebase led to weird merge conflicts so I started with a fresh branch.
progress
This PR splits the features into different groups for different tables. It generates the tables with H3 headings. The changes to the generator and the documentation are included in this PR.
Some additions to
/.gitignoreare also included.future
Updating individual provider pages (#3584) is coming next.
Fixes: #3305