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

Update auth doc links for 1.3 #5300

Merged
merged 3 commits into from
Apr 11, 2023
Merged

Update auth doc links for 1.3 #5300

merged 3 commits into from
Apr 11, 2023

Conversation

gewenyu99
Copy link

@gewenyu99 gewenyu99 commented Mar 29, 2023

What does this PR do?

Updates links because we had to move rate limits in the auth guide.

Details in appwrite/docs#352

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Screenshots may also be helpful.)

Related PRs and Issues

Depends on:
appwrite/docs#352

Related to:
https://github.com/appwrite/homepage/pull/184

Checklist

  • Have you read the Contributing Guidelines on issues?
  • If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

@gewenyu99 gewenyu99 marked this pull request as ready for review March 29, 2023 19:13
@abnegate abnegate changed the base branch from master to 1.3.x April 5, 2023 15:44
@abnegate
Copy link
Contributor

abnegate commented Apr 5, 2023

@gewenyu99 I've changed the base branch to 1.3.x, looks like there is a conflict and some unrelated changes from main, please address

@gewenyu99
Copy link
Author

@gewenyu99 I've changed the base branch to 1.3.x, looks like there is a conflict and some unrelated changes from main, please address

It's just the composer lock file :)

@abnegate
Copy link
Contributor

@gewenyu99 I've changed the base branch to 1.3.x, looks like there is a conflict and some unrelated changes from main, please address

It's just the composer lock file :)

Yep but it needs to be resolved otherwise GitHub blocks merging the PR

@gewenyu99
Copy link
Author

@gewenyu99 I've changed the base branch to 1.3.x, looks like there is a conflict and some unrelated changes from main, please address

It's just the composer lock file :)

Yep but it needs to be resolved otherwise GitHub blocks merging the PR

Ooops! Didn't push 😅

# Conflicts:
#	composer.lock
@abnegate abnegate merged commit c077636 into 1.3.x Apr 11, 2023
@gewenyu99 gewenyu99 deleted the doc-auth-1.3 branch July 10, 2023 20:23
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.

None yet

3 participants