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

React native SDK #794

Merged
merged 21 commits into from
Apr 24, 2024
Merged

React native SDK #794

merged 21 commits into from
Apr 24, 2024

Conversation

lohanidamodar
Copy link
Member

What does this PR do?

  • React Native SDK

Test Plan

  • Need to work on tests, still figuring things out on testing side

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?

YES

```bash
npm install react-native-appwrite react-native-fs react-native-url-polyfill --save
```

Choose a reason for hiding this comment

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

Do we need

For iOS apps, you will need to install cocoapod dependencies.

```sh
cd iOS && pod install && cd ..
```

Copy link
Member Author

Choose a reason for hiding this comment

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

We have it in the getting started section

npm install react-native-appwrite react-native-fs react-native-url-polyfill --save
```

{% if sdk.gettingStarted %}

Choose a reason for hiding this comment

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

Where did we add this? I can't find it.

Copy link
Member Author

Choose a reason for hiding this comment

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

In appwrite/appwrite

templates/react-native/.github/workflows/publish.yml.twig Outdated Show resolved Hide resolved
templates/react-native/src/id.ts.twig Outdated Show resolved Hide resolved
templates/react-native/src/client.ts.twig Outdated Show resolved Hide resolved
templates/react-native/src/client.ts.twig Show resolved Hide resolved
templates/react-native/src/services/template.ts.twig Outdated Show resolved Hide resolved
templates/react-native/src/services/template.ts.twig Outdated Show resolved Hide resolved
templates/react-native/src/client.ts.twig Outdated Show resolved Hide resolved
Comment on lines +307 to +321
* - account
* - collections
* - collections.[ID]
* - collections.[ID].documents
* - documents
* - documents.[ID]
* - files
* - files.[ID]
* - executions
* - executions.[ID]
* - functions.[ID]
* - teams
* - teams.[ID]
* - memberships
* - memberships.[ID]
Copy link
Contributor

Choose a reason for hiding this comment

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

I think there's lots of missing and incorrect channels here

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm keeping this sync with web SDK, if that's the case we should probably update there as well. Where do we have the latest updated channels?

Copy link
Contributor

Choose a reason for hiding this comment

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

List is here: https://appwrite.io/docs/apis/realtime#channels looks like mostly databases/storage ones need updated

templates/react-native/src/client.ts.twig Outdated Show resolved Hide resolved
templates/react-native/src/client.ts.twig Outdated Show resolved Hide resolved
templates/react-native/src/enums/enum.ts.twig Outdated Show resolved Hide resolved
templates/react-native/src/index.ts.twig Show resolved Hide resolved
templates/react-native/src/permission.ts.twig Show resolved Hide resolved
@lohanidamodar lohanidamodar merged commit 5beae76 into master Apr 24, 2024
34 checks passed
@lohanidamodar lohanidamodar deleted the feat-react-native-sdk branch May 1, 2024 08:56
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

4 participants