Skip to content

Feat server sdks quickstarts - #533

Merged
gewenyu99 merged 18 commits into
mainfrom
feat-server-sdks
Jan 29, 2024
Merged

Feat server sdks quickstarts#533
gewenyu99 merged 18 commits into
mainfrom
feat-server-sdks

Conversation

@gewenyu99

@gewenyu99gewenyu99 commented Jan 18, 2024

Copy link
Copy Markdown
Contributor

What does this PR do?

Adds these to /docs/quick-starts

  • swift
  • ruby
  • deno
  • kotlin
  • php

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.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

@render

renderBot commented Jan 18, 2024

Copy link
Copy Markdown

Your Render PR Server URL is https://appwrite-io-docker-pr-533.onrender.com.

Follow its progress at https://dashboard.render.com/web/srv-cmkqte6g1b2c738bjds0.

lohanidamodar
lohanidamodar requested changes Jan 22, 2024
View reviewed changes
Comment threadsrc/routes/docs/quick-starts/deno/+page.markdoc Outdated
Comment threadsrc/routes/docs/quick-starts/kotlin/+page.markdoc Outdated
```kotlin
import io.appwrite.Client
import io.appwrite.services.Databases
import io.appwrite.models.Database as DatabaseModel

@lohanidamodarlohanidamodarJan 22, 2024

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why are we importing as DatabaseModel as service is Databases there shouldn't be conflict?

@gewenyu99gewenyu99Jan 22, 2024

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Fixed

Comment threadsrc/routes/docs/quick-starts/kotlin/+page.markdoc Outdated
Comment threadsrc/routes/docs/quick-starts/kotlin/+page.markdoc Outdated
Comment on lines +141 to +144
databases.createDocument(todoDatabase?.id!!, todoCollection?.id!!, UUID.randomUUID().toString(), testTodo1)
databases.createDocument(todoDatabase?.id!!, todoCollection?.id!!, UUID.randomUUID().toString(), testTodo2)
databases.createDocument(todoDatabase?.id!!, todoCollection?.id!!, UUID.randomUUID().toString(), testTodo3)
}

@lohanidamodarlohanidamodarJan 22, 2024

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Same here we should be using ID.unique()

@gewenyu99gewenyu99Jan 22, 2024

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Oooooops haha

Comment threadsrc/routes/docs/quick-starts/node/+page.markdoc Outdated
Vincent (Wen Yu) Geand others added 2 commits January 22, 2024 17:25
Vincent (Wen Yu) Ge@lohanidamodar
Update src/routes/docs/quick-starts/kotlin/+page.markdoc
9feebbc
Co-authored-by: Damodar Lohani <lohanidamodar@users.noreply.github.com>
Vincent (Wen Yu) Ge@lohanidamodar
Update src/routes/docs/quick-starts/node/+page.markdoc
daaa259
Co-authored-by: Damodar Lohani <lohanidamodar@users.noreply.github.com>
abnegate
abnegate reviewed Jan 23, 2024
View reviewed changes
Comment threadsrc/routes/docs/quick-starts/+page.svelte
Comment threadsrc/routes/docs/quick-starts/deno/+page.markdoc Outdated
Comment threadsrc/routes/docs/quick-starts/deno/+page.markdoc
Comment threadsrc/routes/docs/quick-starts/deno/+page.markdoc Outdated
Comment threadsrc/routes/docs/quick-starts/deno/+page.markdoc Outdated
Comment threadsrc/routes/docs/quick-starts/swift/+page.markdoc Outdated
Comment threadsrc/routes/docs/quick-starts/swift/+page.markdoc Outdated
Comment threadsrc/routes/docs/quick-starts/swift/+page.markdoc Outdated
Comment threadsrc/routes/docs/quick-starts/swift/+page.markdoc Outdated
Comment threadsrc/routes/docs/quick-starts/swift/+page.markdoc Outdated
Vincent (Wen Yu) Geand others added 6 commits January 25, 2024 00:53
Vincent (Wen Yu) Ge@abnegate
Update src/routes/docs/quick-starts/deno/+page.markdoc
b6606f7
Co-authored-by: Jake Barnby <jakeb994@gmail.com>
Vincent (Wen Yu) Ge@abnegate
Update src/routes/docs/quick-starts/swift/+page.markdoc
27facc4
Co-authored-by: Jake Barnby <jakeb994@gmail.com>
Vincent (Wen Yu) Ge@abnegate@lohanidamodar
Apply suggestions from code review
cf1b9a7
Co-authored-by: Jake Barnby <jakeb994@gmail.com>
Co-authored-by: Damodar Lohani <lohanidamodar@users.noreply.github.com>
Vincent (Wen Yu) Ge
Fix Ruby + Fix permission tables
4834ffd
Vincent (Wen Yu) Ge
Merge branch 'main' into feat-server-sdks
a5c7797
Vincent (Wen Yu) Ge
Use named params + address review comments
7a77930
abnegate
abnegate reviewed Jan 26, 2024
View reviewed changes
Comment threadsrc/routes/docs/quick-starts/deno/+page.markdoc Outdated
Comment threadsrc/routes/docs/quick-starts/kotlin/+page.markdoc
Comment threadsrc/routes/docs/quick-starts/kotlin/+page.markdoc Outdated
Comment threadsrc/routes/docs/quick-starts/php/+page.markdoc
Comment threadsrc/routes/docs/quick-starts/php/+page.markdoc Outdated
Comment threadsrc/routes/docs/quick-starts/swift/+page.markdoc
Comment threadsrc/routes/docs/quick-starts/swift/+page.markdoc Outdated
Comment threadsrc/routes/docs/quick-starts/swift/+page.markdoc
Comment threadsrc/routes/docs/quick-starts/swift/+page.markdoc Outdated
Comment threadsrc/routes/docs/quick-starts/swift/+page.markdoc Outdated
Vincent (Wen Yu) Geand others added 4 commits January 26, 2024 10:08
lohanidamodar
lohanidamodar approved these changes Jan 29, 2024
View reviewed changes

@lohanidamodarlohanidamodar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

rest is good.

Comment threadsrc/routes/docs/quick-starts/kotlin/+page.markdoc Outdated
Vincent (Wen Yu) Ge@lohanidamodar
Update src/routes/docs/quick-starts/kotlin/+page.markdoc
7d39791
Co-authored-by: Damodar Lohani <lohanidamodar@users.noreply.github.com>
@gewenyu99
gewenyu99 merged commit 3afc5cd into mainJan 29, 2024
This was referenced Feb 1, 2024
Closed
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@abnegateabnegateabnegate approved these changes
@lohanidamodarlohanidamodarlohanidamodar approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gewenyu99@abnegate@lohanidamodar