Skip to content

fix(api-authorization-docs): removed hardcoded API authorization tokens, added more steps to make the token generation easier #418

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 10, 2025

Conversation

ravindra-dyte
Copy link
Contributor

Description

In a lot of places, API authorization was hardcoded to a random values and it lead to confusion. Added quickstart reference across files to make it easier for newcomers.

Resolved issues

Closes #1

Before submitting the PR, please take the following into consideration

  • It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. If you don't have an issue, please create one.
  • Prefix your PR title with feat: , fix: , chore: , docs:, or refactor:.
  • The description should clearly illustrate what problems it solves.
  • Ensure that the commit messages follow our guidelines.
  • Resolve merge conflicts (if any).
  • Make sure that the current branch is upto date with the main branch.

…ns, added more steps to make the token generation easier
Copy link

Deploying dyte-docs with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6124043
Status: ✅  Deploy successful!
Preview URL: https://8ef286b9.dyte-docs.pages.dev
Branch Preview URL: https://fix-api-authorisation-docs.dyte-docs.pages.dev

View logs

@ravindra-dyte ravindra-dyte requested review from vaibhavshn, ToxicityMax and palashgo and removed request for ToxicityMax February 10, 2025 16:05
@vaibhavshn vaibhavshn merged commit 77ae194 into main Feb 10, 2025
2 checks passed
@vaibhavshn vaibhavshn deleted the fix/api-authorisation-docs branch February 10, 2025 16:32
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