Closed AMBERMW13 closed 2 months ago
success
success
success
Output is too long and was truncated. You can read full Plan in Actions.
Pusher: @AMBERMW13, Action: pull_request
, Working Directory: /home/runner/work/liquibase-docs/liquibase-docs
, Workflow: Publish Redirects to Staging for Docs
@AMBERMW13 Thanks for handling this ticket. Some changes needed:
The new redirects have a path error in the source. On lines 414-418 the correct URLs would be /concepts/changelogs/attributes/xyz.html
and not /concepts/attributes/xyz.html
.
We also need to remember to update existing redirects, i.e. avoid double redirects. Because existing links go to the pages that we're now redirecting, we need to point people to the final destination instead of having them redirected to a different redirect:
/concepts/changelogs/attributes/use-psql-integration.html
to /tools-integrations/native-executors/use-psql-integration.html
/concepts/changelogs/attributes/use-sqlcmd-integration.html
to /tools-integrations/native-executors/use-sqlcmd-integration.html
/concepts/changelogs/attributes/use-sql-plus-oracle-proxy-user.html
to /tools-integrations/native-executors/use-sql-plus-oracle-proxy-server.html
/concepts/changelogs/attributes/use-sql-plus-integration.html
to /tools-integrations/native-executors/use-sql-plus-integration.html
Let me know when you've updated and I'll give it another scan. Thanks!
Hi @adrian-velonis1 thanks for these pointers! I forgot to look for previous redirects, so that's a good catch. I made all of your suggested changes, so it's ready for your review again. Let me know if you have any other changes. Thanks!
success
success
success
Output is too long and was truncated. You can read full Plan in Actions.
Pusher: @AMBERMW13, Action: pull_request
, Working Directory: /home/runner/work/liquibase-docs/liquibase-docs
, Workflow: Publish Redirects to Staging for Docs
@AMBERMW13 Looks good, merging
Moved Native Executor content from the concepts/attributes section to the new tools-integrations/native-executors section.