Skip to content
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

Docs update cli 2.0 #1250

Open
wants to merge 28 commits into
base: main
Choose a base branch
from
Open

Docs update cli 2.0 #1250

wants to merge 28 commits into from

Conversation

choir241
Copy link
Contributor

@choir241 choir241 commented Aug 21, 2024

What does this PR do?

  • Replace appwrite projects get --project-id "<PROJECT_ID>" with appwrite users list
  • add appwrite login and paragraph to non-interactive
  • add note that commands won't work with an API key like project commands

Add two examples:

  • how to authorize with API key
  • how to authorize with email & password in CI/CD

Test Plan

Tested login with non-interactive

Related PRs and Issues

X

Have you read the Contributing Guidelines on issues?

Yes

@coolify-appwrite-org
Copy link

coolify-appwrite-org bot commented Aug 21, 2024

The preview deployment is ready. 🟢

Open Preview | Open Build Logs

Last updated at: 2024-09-03 15:33:28 CET


# Authentication {% #authentication %}

In non-interactive mode, you can authenticate your users using two methods: email and password. While using an [API key](/docs/advanced/platform/api-keys) does not require an email and password to authenticate users, you need to configure the API key [scopes](/docs/advanced/platform/api-keys#scopes) of the application and does not work for project API calls that update project settings.
Copy link
Contributor

Choose a reason for hiding this comment

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

In non-interactive mode, you can authenticate your users using two methods: email and password.

Let's remove the "your users" part because you're not really authenticating as a user. Also, what happened to the API key method?

While using an API key does not require an email and password to authenticate users

Don't mention users because the authentication is not as an end user.

and does not work for project API calls that update project settings.

you need to configure [...] and does not work [...] doesn't sound right.


## Email and password {% #email-and-password %}

Use `appwrite login` with your email, and password to log in so you can run tests or set up project-per-branch previews.
Copy link
Contributor

Choose a reason for hiding this comment

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

email, and password

is this comma supposed to be here?

to log in so you can run tests or set up project-per-branch previews.

Let's not mention this. Honestly, it's an unnecessary and confusing detail.


# Authentication {% #authentication %}

In non-interactive mode, you can authenticate using two methods: email and password or API keys. While using an [API key](/docs/advanced/platform/api-keys) does not require an email and password to authenticate, the API key needs to have its [scopes](/docs/advanced/platform/api-keys#scopes) configured and the API key does not work for project API calls that update project settings.
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we check this sentence with grammarly? Im not a native speaker and it reads hard.


# Authentication {% #authentication %}

In non-interactive mode, you can authenticate using two methods: email and password or API keys. While using an [API key](/docs/advanced/platform/api-keys) does not require an email and password to authenticate, the API key needs to have its [scopes](/docs/advanced/platform/api-keys#scopes) configured and the API key does not work for project API calls that update project settings.
Copy link
Contributor

Choose a reason for hiding this comment

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

Regarding API key limitation, we can say that administrative operations cannot be done such as changing organization billing, deleting a project, or creating a new project. For those you must email&password login

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