Skip to content

docs: remove 'development/authentication' from navigation panel. closes #1525#1532

Merged
talboren merged 2 commits intokeephq:mainfrom
frwl404:anton/mintlify-dev-docs
Aug 5, 2024
Merged

docs: remove 'development/authentication' from navigation panel. closes #1525#1532
talboren merged 2 commits intokeephq:mainfrom
frwl404:anton/mintlify-dev-docs

Conversation

@frwl404
Copy link
Copy Markdown
Contributor

@frwl404 frwl404 commented Aug 4, 2024

Closes #1525

📑 Description

Now we have item with non-existing routing in navigation panel (authentication). It is because it is present in 'development' section of mint.json, but there is no corresponding .mdx documentation file exists. 'authentication' is related to 'deployment' section and it is already there. Having 'authentication' in 'development' section seems to be a copy-paste error.

✅ Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

…hq#1525

Now we have item with non-existing routing in navigation panel (authentication).
It is because it is present in 'development' section of mint.json, but there is no corresponding .mdx documentation file exists.
'authentication' is related to 'deployment' section and it is already there.
Having 'authentication' in 'development' section seems to be a copy-paste error.
@vercel
Copy link
Copy Markdown

vercel bot commented Aug 4, 2024

Someone is attempting to deploy a commit to the KeepHQ Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Aug 4, 2024

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ talboren
❌ Anton


Anton seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@frwl404 frwl404 changed the title Docs: remove 'development/authentication' from navigation. close #1525 docs: remove 'development/authentication' from navigation panel. closes #1525 Aug 4, 2024
Copy link
Copy Markdown
Member

@talboren talboren left a comment

Choose a reason for hiding this comment

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

LGTM! thank you!

@vercel
Copy link
Copy Markdown

vercel bot commented Aug 5, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
keep ⬜️ Ignored (Inspect) Visit Preview Aug 5, 2024 0:01am

@talboren talboren enabled auto-merge (squash) August 5, 2024 11:58
@talboren talboren merged commit ea51967 into keephq:main Aug 5, 2024
@frwl404 frwl404 deleted the anton/mintlify-dev-docs branch August 5, 2024 12:01
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.

Documentation (Development): wrong set of items in navigation pane.

3 participants