Skip to content

Comments

DOC-5016 Replace CDC content with docs-common#130

Merged
aimurphy merged 10 commits intomainfrom
doc-5016
Apr 16, 2025
Merged

DOC-5016 Replace CDC content with docs-common#130
aimurphy merged 10 commits intomainfrom
doc-5016

Conversation

@aimurphy
Copy link
Contributor

@aimurphy aimurphy commented Apr 8, 2025 •
edited
Loading

  • Moved CDC content to docs-common.
  • Replace contents of astream-cdc.adoc with the docs-common include
  • Slight change to nav - Astra CLI felt misplaced under Development, moved it under Operations?
  • Small copyedits related to CDC content updates

Preview build of CDC page: https://d5rxiv0do0q3v.cloudfront.net/doc-5016/astra-streaming/developing/astream-cdc.html

@aimurphy aimurphy self-assigned this Apr 8, 2025
@plpesvc-ds

This comment was marked as resolved.

Sign in to view
@plpesvc-ds

This comment was marked as outdated.

Sign in to view
@plpesvc-ds

This comment was marked as outdated.

Sign in to view
@plpesvc-ds

This comment was marked as outdated.

Sign in to view
@plpesvc-ds

This comment was marked as outdated.

Sign in to view
@plpesvc-ds
Copy link

plpesvc-ds commented Apr 8, 2025 •
edited
Loading

Build successful! ✅
Pull requests with matching branch name doc-5016 included in the build:

Deploying draft.
Deploy successful! View draft

@plpesvc-ds
Copy link

plpesvc-ds commented Apr 8, 2025 •
edited
Loading

Build successful! ✅
Pull requests with matching branch name doc-5016 included in the build:

Deploying draft.
Deploy successful! View draft

@plpesvc-ds

This comment was marked as resolved.

Sign in to view
@plpesvc-ds

This comment was marked as resolved.

Sign in to view
@plpesvc-ds
Copy link

plpesvc-ds commented Apr 12, 2025 •
edited
Loading

Build successful! ✅
Pull requests with matching branch name doc-5016 included in the build:

Deploying draft.
Deploy successful! View draft

3 similar comments
@plpesvc-ds
Copy link

plpesvc-ds commented Apr 12, 2025 •
edited
Loading

Build successful! ✅
Pull requests with matching branch name doc-5016 included in the build:

Deploying draft.
Deploy successful! View draft

@plpesvc-ds
Copy link

plpesvc-ds commented Apr 12, 2025 •
edited
Loading

Build successful! ✅
Pull requests with matching branch name doc-5016 included in the build:

Deploying draft.
Deploy successful! View draft

@plpesvc-ds
Copy link

plpesvc-ds commented Apr 12, 2025 •
edited
Loading

Build successful! ✅
Pull requests with matching branch name doc-5016 included in the build:

Deploying draft.
Deploy successful! View draft

mendonk
mendonk approved these changes Apr 16, 2025
View reviewed changes
Copy link
Contributor

@mendonk mendonk left a comment

Choose a reason for hiding this comment

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

LGTM!

@plpesvc-ds
Copy link

plpesvc-ds commented Apr 16, 2025 •
edited
Loading

Build successful! ✅
Pull requests with matching branch name doc-5016 included in the build:

Deploying draft.
Deploy successful! View draft

@aimurphy aimurphy merged commit 0269168 into main Apr 16, 2025
1 check passed
@aimurphy aimurphy deleted the doc-5016 branch April 16, 2025 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mendonk mendonk mendonk approved these changes

Assignees

@aimurphy aimurphy

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@aimurphy @plpesvc-ds @mendonk