Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

Update redirects to use new default permanent behavior

Summary

This PR updates the redirect configuration to align with the platform change in fern-api/fern-platform#4141, which changed the default redirect behavior from temporary (307) to permanent (308).

Changes:

  1. Removed 187 instances of permanent: true from fern/docs.yml since it's now the default
  2. Updated redirect documentation to reflect the new default behavior
  3. Changed example code to demonstrate permanent: false for temporary redirects
  4. Clarified field descriptions per the user's requirements

Review & Testing Checklist for Human

  • CRITICAL: Verify that PR #4141 is deployed to production before merging this PR. If this PR is merged first, all redirects will behave incorrectly (temporary instead of permanent).
  • Test a few redirects from the docs site to confirm they return 308 status codes (permanent) by default
  • Review the updated documentation wording in fern/snippets/redirects.mdx to ensure it's clear for users
  • Verify YAML syntax is valid by running fern check or similar validation if available

Notes

Link to Devin run: https://app.devin.ai/sessions/405d220e4a32404b99f5fbe39d807bb0
Requested by: [email protected] (@dannysheridan)

Deployment dependency: This PR has a hard dependency on the fern-platform PR being deployed first. The platform code change must be live before these config changes go live, otherwise all existing redirects will break.

No behavioral change for existing redirects: Since all redirects in docs.yml already had permanent: true explicitly set, this change maintains the current behavior while removing redundant configuration.

- Remove all 'permanent: true' entries from docs.yml (now default)
- Update redirect documentation to reflect new default (permanent: true)
- Add example of using permanent: false for temporary redirects
- Update field descriptions per PR fern-api/fern-platform#4141

Co-Authored-By: [email protected] <[email protected]>
@devin-ai-integration
Copy link
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions
Copy link
Contributor

@dannysheridan
Copy link
Member

Waiting on fern-api/fern-platform#4141 to be released into prod.

@dannysheridan dannysheridan merged commit 78f748f into main Oct 17, 2025
1 check passed
@dannysheridan dannysheridan deleted the devin/1760638821-remove-permanent-true-redirects branch October 17, 2025 02:48
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