这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@wawhal
Copy link
Contributor

@wawhal wawhal commented May 7, 2019

Add React Native tutorial

Description

Affected components

  • Community Content

Related Issues

Solution and Design

Steps to test and verify

Limitations, known bugs & workarounds

@wawhal wawhal added the s/do-not-merge Do not merge this pull request to master label May 7, 2019
@wawhal wawhal requested review from rikinsk-zz and shahidhk as code owners May 7, 2019 08:52
@netlify
Copy link

netlify bot commented May 7, 2019

Deploy preview for hasura-docs ready!

Built with commit d93ca89

https://deploy-preview-2136--hasura-docs.netlify.com

@wawhal wawhal force-pushed the learn-hasura-react-native branch from 72feb0f to b0f95f1 Compare May 7, 2019 08:56
@praveenweb praveenweb added the c/learn Related to learn.hasura.io label May 10, 2019
Copy link
Contributor

@vnovick vnovick left a comment

Choose a reason for hiding this comment

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

Please change inconsistencies in the docs and between sections
And most importantly please create a new engine instance for react-native and mobile in general with the same custom auth you are using because we need to run graphiql on that engine.


```graphql
- query {
+ query ($is_public: Boolean) {
Copy link
Contributor

Choose a reason for hiding this comment

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

later on you use $isPublic - please change to that

+ created_at: desc
+ },
- where: { is_public: { _eq: false} }
+ where: { is_public: { _eq: $is_public} }
Copy link
Contributor

Choose a reason for hiding this comment

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

isPublic

id
title
created_at
is_completed
Copy link
Contributor

Choose a reason for hiding this comment

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

add here user and basically all data you get in the next section.

@shahidhk shahidhk changed the title add react native tutorial learn: add react native tutorial May 13, 2019
@wawhal wawhal removed the s/do-not-merge Do not merge this pull request to master label May 13, 2019
@shahidhk shahidhk merged commit 5f903e4 into hasura:master May 13, 2019
@hasura-bot
Copy link
Contributor

Review app https://hge-ci-pull-2136.herokuapp.com is deleted

polRk pushed a commit to polRk/graphql-engine that referenced this pull request Feb 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c/learn Related to learn.hasura.io

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants