appwrite icon indicating copy to clipboard operation
appwrite copied to clipboard

Fix docs showing endpoint for client

Open Meldiron opened this issue 3 years ago • 5 comments

What does this PR do?

  • Fix bug when docs were showing Retry Build to the client
  • Re-ordered endpoints. First entity, then action (create,list,get,update,delete). Action from least detailed to the most
  • Renamed some endpoint descriptions to make them more consistent
  • Fix some types, so homepage doesn't fail to start

Test Plan

  • [x] Test with homepage repo. Screenshots are provided as a comment on this PR

Related PRs and Issues

x

Have you read the Contributing Guidelines on issues?

Yes 😇

Meldiron avatar Jun 06 '22 11:06 Meldiron

Screenshots:

CleanShot 2022-06-07 at 10 06 53@2x CleanShot 2022-06-07 at 10 06 37@2x CleanShot 2022-06-07 at 10 06 31@2x CleanShot 2022-06-07 at 10 06 19@2x CleanShot 2022-06-07 at 10 05 49@2x CleanShot 2022-06-07 at 10 05 32@2x CleanShot 2022-06-07 at 10 05 23@2x CleanShot 2022-06-07 at 10 08 34@2x CleanShot 2022-06-07 at 10 08 30@2x

Meldiron avatar Jun 07 '22 08:06 Meldiron

The diff makes it super hard to review, will take some time 😁

TorstenDittmann avatar Jun 07 '22 22:06 TorstenDittmann

@Meldiron And now there are merge conflicts. Perhaps it's possible to break down this PR?

stnguyen90 avatar Jul 22 '22 23:07 stnguyen90

We have a lot of naming inconsistencies when creating a session. Also, email shouldn't be treated as the default way.

eldadfux avatar Aug 06 '22 07:08 eldadfux

Ensure we fix the naming convention of the Create Account Session endpoints

  1. Create Email Session
  2. Create Phone Session
  3. Create Anonymous session etc.

christyjacob4 avatar Aug 09 '22 09:08 christyjacob4

Due to the merge conflicts,  #3667 has been created to supplant this PR.

PineappleIOnic avatar Aug 12 '22 14:08 PineappleIOnic

closing this in favour of #3667

christyjacob4 avatar Aug 13 '22 13:08 christyjacob4