Skip to content

docs: Improve tutorial#947

Merged
britaniar merged 23 commits intoAzure:mainfrom
jamyct:improve-tutorial
Nov 8, 2024
Merged

docs: Improve tutorial#947
britaniar merged 23 commits intoAzure:mainfrom
jamyct:improve-tutorial

Conversation

@jamyct
Copy link
Contributor

@jamyct jamyct commented Nov 8, 2024

Description of your changes

Improve tutorial for Migration with Override DR

Fixes #

I have:

  • Run make reviewable to ensure this PR is ready for review.

How has this code been tested

Special notes for your reviewer

@jamyct jamyct marked this pull request as draft November 8, 2024 01:22
@britaniar britaniar changed the title Improve tutorial docs: Improve migration with override tutorial Nov 8, 2024
@jamyct jamyct changed the title docs: Improve migration with override tutorial docs: Improve tutorial Nov 8, 2024
Copy link
Contributor

@britaniar britaniar left a comment

Choose a reason for hiding this comment

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

Add links within the tutorial to the correlated files, otherwise the user will not know they exist for this purpose.

@jamyct jamyct marked this pull request as ready for review November 8, 2024 02:12
Copy link
Contributor

@britaniar britaniar left a comment

Choose a reason for hiding this comment

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

Please similarly link the cluster resource override yaml as well.

jamyct and others added 6 commits November 8, 2024 13:36
Co-authored-by: Britania Rodriguez Reyes <145056127+britaniar@users.noreply.github.com>
Co-authored-by: Britania Rodriguez Reyes <145056127+britaniar@users.noreply.github.com>
Co-authored-by: Britania Rodriguez Reyes <145056127+britaniar@users.noreply.github.com>
@britaniar britaniar merged commit 3cb2de9 into Azure:main Nov 8, 2024
@jamyct jamyct deleted the improve-tutorial branch November 8, 2024 03:40
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.

2 participants