Skip to content

VReplication: Improve Overview Page#1211

Merged
rohit-nayak-ps merged 4 commits intoprodfrom
maxeng-rm-legacy-sharding
Nov 8, 2022
Merged

VReplication: Improve Overview Page#1211
rohit-nayak-ps merged 4 commits intoprodfrom
maxeng-rm-legacy-sharding

Conversation

@maxenglander
Copy link
Copy Markdown
Collaborator

@maxenglander maxenglander commented Nov 1, 2022

Remove reference to legacy sharding workflows.

Fixes: vitessio/vitess#11598.

Signed-off-by: Max Englander <max@planetscale.com>
@netlify
Copy link
Copy Markdown

netlify bot commented Nov 1, 2022

Deploy Preview for vitess ready!

Name Link
🔨 Latest commit 91bb000
🔍 Latest deploy log https://app.netlify.com/sites/vitess/deploys/63694412979400000826859b
😎 Deploy Preview https://deploy-preview-1211--vitess.netlify.app/docs/16.0/reference/vreplication/vreplication
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@maxenglander
Copy link
Copy Markdown
Collaborator Author

@maxenglander maxenglander marked this pull request as ready for review November 1, 2022 15:25
@maxenglander maxenglander requested a review from mattlord November 1, 2022 15:25
@mattlord
Copy link
Copy Markdown
Member

mattlord commented Nov 1, 2022

Do you mind tag teaming on this PR to address all of the refactoring of this page?

@mattlord mattlord changed the title fix(vreplication): remove reference to legacy workflows VReplication: Improve Overview Page Nov 4, 2022
@mattlord mattlord force-pushed the maxeng-rm-legacy-sharding branch from 2b04abc to 617d24c Compare November 4, 2022 20:52
Signed-off-by: Matt Lord <mattalord@gmail.com>
@mattlord mattlord force-pushed the maxeng-rm-legacy-sharding branch from 617d24c to b8501dc Compare November 7, 2022 01:01
Copy link
Copy Markdown
Member

@rohit-nayak-ps rohit-nayak-ps left a comment

Choose a reason for hiding this comment

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

changes looks good! left some comments.

Signed-off-by: Matt Lord <mattalord@gmail.com>
@mattlord
Copy link
Copy Markdown
Member

mattlord commented Nov 7, 2022

Thanks, @rohit-nayak-ps ! I made adjustments based on your helpful feedback here: d7815a6

Signed-off-by: Matt Lord <mattalord@gmail.com>
Copy link
Copy Markdown
Member

@rohit-nayak-ps rohit-nayak-ps left a comment

Choose a reason for hiding this comment

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

lgtm

@rohit-nayak-ps rohit-nayak-ps merged commit edbc0d2 into prod Nov 8, 2022
@rohit-nayak-ps rohit-nayak-ps deleted the maxeng-rm-legacy-sharding branch November 8, 2022 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docs: Overhaul VReplication Overview Page

3 participants