Skip to content

docs: add note explaining why x-fern-default is used instead of plain default#5332

Merged
Swimburger merged 1 commit intomainfrom
devin/1777851190-default-values-note
May 3, 2026
Merged

docs: add note explaining why x-fern-default is used instead of plain default#5332
Swimburger merged 1 commit intomainfrom
devin/1777851190-default-values-note

Conversation

@Swimburger
Copy link
Copy Markdown
Member

@Swimburger Swimburger commented May 3, 2026

Summary

Adds a <Note> callout to the default values extension page explaining why Fern uses x-fern-default instead of the standard OpenAPI default property.

The key distinction: OpenAPI default describes the value the server assumes when a parameter is omitted (a documentation hint), while x-fern-default instructs the SDK to actively send the value in every request, ensuring it's always present on the wire.

Review & Testing Checklist for Human

  • Verify the note renders correctly on the preview deployment
  • Confirm the semantic explanation is technically accurate

Notes

Low-risk, single-callout addition to an existing page.

Link to Devin session: https://app.devin.ai/sessions/8dbd9e9711624a8d8a8ef82cfcdd0134
Requested by: @Swimburger


Open in Devin Review

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration
Copy link
Copy Markdown
Contributor

🤖 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

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Open in Devin Review

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 3, 2026

@Swimburger Swimburger merged commit 7ac069c into main May 3, 2026
4 checks passed
@Swimburger Swimburger deleted the devin/1777851190-default-values-note branch May 3, 2026 23:36
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