Skip to content

Update authentication.md#80

Closed
rahuldimri wants to merge 2 commits intohoppscotch:mainfrom
rahuldimri:patch-6
Closed

Update authentication.md#80
rahuldimri wants to merge 2 commits intohoppscotch:mainfrom
rahuldimri:patch-6

Conversation

@rahuldimri
Copy link
Copy Markdown
Contributor

No description provided.

@netlify
Copy link
Copy Markdown

netlify Bot commented Sep 22, 2022

👷 Deploy request for hoppscotch-docs pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 8283fba

@rahuldimri
Copy link
Copy Markdown
Contributor Author

Can someone please have a look into this PR request.

@liyasthomas
Copy link
Copy Markdown
Member

@rahuldimri, what’s the benefit of this PR?

@rahuldimri
Copy link
Copy Markdown
Contributor Author

Just correcting the format. Its not going to add any benefit, but it will help us to make the document in correct format going forward as well. All the document headlines need to be started with capital letter.

Let me know if you have any other thought on this.

@liyasthomas
Copy link
Copy Markdown
Member

The code block changed in this PR updates the metadata of the page. Usually metadata keys are in lowercase.

Hence closing this PR.

@liyasthomas liyasthomas closed this Oct 6, 2022
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