-
Notifications
You must be signed in to change notification settings - Fork 50
Doc-1996: Schema Registry Contexts #1658
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
Open
Feediver1
wants to merge
39
commits into
main
Choose a base branch
from
Feediver1-patch-7
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
3c19d16
Create schema-reg-contexts.adoc
Feediver1 51f0e5a
docs(DOC-1658): add Schema Registry Contexts to site nav
Feediver1 49a620c
First draft grimness
Feediver1 0657a06
Update modules/manage/pages/schema-reg/schema-reg-contexts.adoc
Feediver1 7566a77
Apply suggestions from code review
Feediver1 24eb29d
Apply suggestions from code review
Feediver1 b8de5b2
Apply suggestions from code review
Feediver1 b7e87ea
Apply suggestions from code review
Feediver1 1ac6b7e
Apply suggestions from code review
Feediver1 d0c1357
docs: add single-source tags and cross-link cloud-docs playbook
micheleRP c8d6312
Apply suggestions from code review
Feediver1 c81afb0
Apply suggestion from @Feediver1
Feediver1 c998717
Apply suggestion from @Feediver1
Feediver1 8613e3c
Apply suggestion from @Feediver1
Feediver1 d797a76
Apply suggestion from @Feediver1
Feediver1 69243b9
Add contexts to schema-reg-api.adoc and cross-references
kbatuigas b04e90e
Apply suggestion from @Feediver1
Feediver1 a548896
Apply suggestions from code review
Feediver1 88c76ad
Merge branch 'main' into Feediver1-patch-7
Feediver1 e445bba
Apply suggestion from @Feediver1
Feediver1 d493995
Apply suggestion from @Feediver1
Feediver1 49e88e3
Apply suggestion from @Feediver1
Feediver1 fbeee42
docs(DOC-1658): add Schema Registry contexts to What's New
Feediver1 2f9d3df
Apply suggestions from code review
Feediver1 304d7ee
fix: restore cloud-docs branch to main in local-antora-playbook.yml
Feediver1 915b5fe
chore: trigger Netlify preview rebuild
Feediver1 b2a9211
docs(DOC-1658): reduce learning objectives to 3
Feediver1 9802088
Apply suggestion from @Feediver1
Feediver1 c84400a
Apply suggestion from @Feediver1
Feediver1 6cb5311
Merge branch 'main' into Feediver1-patch-7
Feediver1 ffa12f5
Apply suggestions from code review
Feediver1 14f8c6e
Update modules/manage/pages/schema-reg/schema-reg-contexts.adoc
Feediver1 4e10c42
Apply suggestions from code review
Feediver1 3d7c996
Apply suggestion from @Feediver1
Feediver1 37869ad
Apply suggestions from code review
Feediver1 fdd8cf8
Add Schema Registry contexts section to Console schema-reg page
Feediver1 aa094ce
Add anchor and link to Cross-context schema references heading
Feediver1 ef3312c
Apply suggestions from code review
Feediver1 22a29ca
Apply suggestion from @Feediver1
Feediver1 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is still WIP, as per https://github.com/redpanda-data/docs/pull/1658/changes#r3123466186
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So remove the entire point about the base URL prefix?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@trevpanda @mattschumpert what do you think?
We need to either remove it or call out that it's not yet supported. But from slack I vaguely recall a policy that we don't document future features, so I am leaning removing this for now, yes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, we should remove until we support