Skip to content

Conversation

@MantisClone
Copy link
Member

@MantisClone MantisClone commented Nov 7, 2025

Problem

Mintlify's banner dismissal is permanent (stored in browser data). Once users dismiss the beta docs banner, they never see it again, meaning they:

  • May not realize they're viewing beta documentation on subsequent visits
  • Lose consistent access to the "View stable docs" link

Proposed Solution

Changed dismissible: false so the beta banner persists across all sessions, ensuring users always have context about documentation status and access to stable docs alternative.

Considerations

  • Trade-off: Slight increase in visual clutter vs. critical context preservation
  • Future: Remove banner entirely once beta period ends
  • Feature request to Mintlify: Add session-based dismissal option (dismissible: "session")

Summary by CodeRabbit

  • Style
    • The beta banner is now permanently visible and cannot be dismissed.

Copilot AI review requested due to automatic review settings November 7, 2025 19:02
@coderabbitai
Copy link

coderabbitai bot commented Nov 7, 2025

Walkthrough

The PR modifies docs.json to change the banner.dismissible property from true to false, making the beta banner non-dismissible while maintaining all other banner content and navigation structure.

Changes

Cohort / File(s) Change Summary
Banner Configuration
docs.json
Changed banner.dismissible from true to false, making the beta banner persistently visible

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: making the beta docs banner non-dismissible by changing the dismissible property from true to false.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch non-dismissable-banner

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2dbadca and 74d7971.

📒 Files selected for processing (1)
  • docs.json (1 hunks)
🔇 Additional comments (1)
docs.json (1)

33-36: Configuration change aligns with PR objectives.

The change to set dismissible: false ensures the beta banner persists across sessions, keeping users informed of the beta status and providing consistent access to the stable docs link. The banner content remains unchanged and appropriately communicates the current documentation status.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modifies the banner configuration in the documentation settings to prevent users from dismissing the beta documentation notice.

Key Changes:

  • Changed the dismissible property of the banner from true to false

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@MantisClone MantisClone self-assigned this Nov 7, 2025
@MantisClone MantisClone merged commit 2efb991 into main Nov 7, 2025
10 checks passed
@MantisClone MantisClone deleted the non-dismissable-banner branch November 7, 2025 19:15
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