Skip to content

Conversation

@d-bytebase
Copy link
Contributor

Summary

  • Removed icon frontmatter properties from onboarding documentation pages
  • Affects plan.mdx and poc.mdx files
  • Simplifies page metadata by removing unnecessary icon properties

Test plan

  • Verify documentation renders correctly without icon properties
  • Check that navigation still works properly for these pages

🤖 Generated with Claude Code

Remove icon frontmatter from plan.mdx and poc.mdx to simplify page metadata

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>
Copilot AI review requested due to automatic review settings August 21, 2025 06:36
@vercel
Copy link

vercel bot commented Aug 21, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
bytebase-com Building Building Preview Comment Aug 21, 2025 6:36am

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 removes icon frontmatter properties from onboarding documentation pages to simplify page metadata and eliminate unnecessary icon configuration.

  • Removed icon: 'check' from poc.mdx frontmatter
  • Removed icon: 'notebook-pen' from plan.mdx frontmatter

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
mintlify/onboarding/poc.mdx Removed icon property from POC Checklist page frontmatter
mintlify/onboarding/plan.mdx Removed icon property from Onboarding Plan page frontmatter

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@d-bytebase d-bytebase requested a review from h3n4l August 21, 2025 06:36
@github-actions
Copy link

📝 Broken Links Check Report

Note: Entries under /api-reference/ are likely false positives and can be ignored.

The following is the output from the broken links check:

found 16 broken links in 9 files

integrations/api/audit-log.mdx
 ⎿  /api-reference/auditlogservice/post-v1auditlogs:search

integrations/api/authentication.mdx
 ⎿  /api-reference/authservice/post-v1authlogin

integrations/api/data-classification.mdx
 ⎿  /api-reference/settingservice/patch-v1settings
 ⎿  /api-reference/databaseservice/patch-v1instances-databases

integrations/api/issue.mdx
 ⎿  /api-reference/issueservice/post-v1projects-issues

integrations/api/permission.mdx
 ⎿  /api-reference/workspaceservice/get-v1workspaces-:getIamPolicy
 ⎿  /api-reference/projectservice/get-v1projects-:getIamPolicy
 ⎿  /api-reference/roleservice/get-v1roles
 ⎿  /api-reference/userservice/get-v1users
 ⎿  /api-reference/groupservice/get-v1groups

integrations/api/plan.mdx
 ⎿  /api-reference/planservice/post-v1projects-plans
 ⎿  /api-reference/sheetservice/post-v1projects-sheets

integrations/api/release.mdx
 ⎿  /api-reference/releaseservice/post-v1projects-releases

integrations/api/rollout.mdx
 ⎿  /api-reference/rolloutservice/post-v1projects-rollouts

integrations/api/sql-review.mdx
 ⎿  /api-reference/releaseservice/post-v1projects-releases:check
 ⎿  /api-reference/sqlservice/post-v1sqlcheck

Copy link
Member

@h3n4l h3n4l left a comment

Choose a reason for hiding this comment

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

LGTM

@h3n4l h3n4l merged commit a5cc267 into main Aug 21, 2025
4 of 5 checks passed
@h3n4l h3n4l deleted the docs/remove-onboarding-icons branch August 21, 2025 06:37
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.

3 participants