Skip to content

Conversation

@CamSoper
Copy link
Contributor

@CamSoper CamSoper commented Oct 31, 2025

Implement a redirect script to guide users to the new update conflict page when accessing the troubleshooting guide.

Fixes pulumi/pulumi#20846

<soapbox>
We really need a URL shortener. This type of fix doesn't scale and adds technical debt to a repo that already has more than enough.

Thank you for attending my TED Talk. 🎤⬇️
</soapbox>

Edit: pulumi/pulumi#20847

@claude
Copy link
Contributor

claude bot commented Oct 31, 2025

Documentation Review

I've reviewed the changes in this PR. Overall, this is a simple and effective solution to preserve backward compatibility for the old #conflict anchor link.

Issues Found

1. Missing newline at end of file (line 61 in content/docs/support/troubleshooting/_index.md)

The file does not end with a newline after the closing </script> tag. According to AGENTS.md, "Markdown: Must always end with a newline."

Suggestion:

</script>

(Add a blank line after </script>)

Observations

✅ The redirect target /docs/support/troubleshooting/common-issues/update-conflicts/ is correct and the file exists
✅ JavaScript implementation is simple and appropriate for this use case
✅ Comment in the code references the GitHub issue
✅ The redirect preserves SEO and user experience for existing links

Summary

The PR successfully addresses issue #20846 by redirecting users from the old #conflict anchor to the new dedicated page. Once the newline issue is fixed, this will be ready to merge.


Mention @claude if you'd like me to review any additional changes or provide further feedback.

@pulumi-bot
Copy link
Collaborator

@CamSoper CamSoper merged commit 8f62d9a into master Nov 4, 2025
15 checks passed
@CamSoper CamSoper deleted the CamSoper/pulumi/20846 branch November 4, 2025 01:56
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.

Link to conflict docs are broken

4 participants