Skip to content

Conversation

@devin-ai-integration
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot commented Dec 6, 2025

Description

Refs: Slack request from Catherine

Adds a new right-text field to the LogoConfiguration schema that allows users to configure text displayed to the right of their logo image. This is similar to what Cohere uses (showing "Docs" next to their logo), but now configurable for all users.

Changes Made

  • Added right-text optional string field to LogoConfiguration in the docs-yml API definition
  • Updated ParsedDocsConfiguration.Logo interface to include rightText
  • Updated convertLogoReference function to pass through the new field
  • Wired logoRightText into the FDR write request in DocsDefinitionResolver.ts
  • Added @ts-expect-error comment for logoRightText until FDR SDK is regenerated
  • Added changelog entry for CLI version 3.6.0
  • Regenerated SDK types and JSON schemas

Companion PR: fern-api/fern-platform#5714 (adds logoRightText to FDR API definition and converters)

Updates Since Last Revision

  • Merged origin/main to resolve merge conflicts
  • Updated changelog version from 3.5.0 to 3.6.0 (3.5.0 was already taken by another feature)
  • Added @ts-expect-error comment in DocsDefinitionResolver.ts to suppress TypeScript error until FDR SDK is regenerated with the new logoRightText field

Human Review Checklist

  • The SDK regeneration also added hidden to PlaygroundSettings - verify this addition is intentional or if it was picked up from another change
  • Verify the logoRightText field will be accepted by FDR (the fern-platform PR adds this to the FDR schema)
  • Verify the schema field name right-text (kebab-case in YAML) mapping to rightText (camelCase in TS) is correct
  • The @ts-expect-error comment should be removed once the FDR SDK is regenerated

Testing

  • Lint checks pass (pnpm run check)
  • Local compile passes (pnpm turbo run compile --filter=@fern-api/docs-resolver)
  • End-to-end testing requires fern-platform PR to be merged first

Link to Devin run: https://app.devin.ai/sessions/42df904f84f64b34a0696ddc33533983
Requested by: Catherine Deskur ([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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants