-
Notifications
You must be signed in to change notification settings - Fork 204
Hide database migration post from table of content list #2861
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
Conversation
|
/approve |
|
This is a blog post, not documentation. I feel like you basically shouldn't delete it once you publish it even if it gets outdated as someone is still likely to refer to this page. |
|
/approve cancel |
|
I don't have a strong opinion so if you guys are fine, proceed as is. |
I feel you, but the change I made here does not delete the post content. It's possible to access the post by URL directly, just hide it from the table of content list, since migration post may cause users to misunderstand that our PipeCD highly depends on a database, that may not good for the product image. |
|
That makese total sense. In that case fine to me 👍 |
What this PR does / why we need it:
Hide the database migration post from blog table of content, since that version is staled and we're not going to support that migration script in the future.
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?: