Skip to content

Instantly share code, notes, and snippets.

@mwickett
Last active December 20, 2022 23:00
Show Gist options
  • Star 62 You must be signed in to star a gist
  • Fork 10 You must be signed in to fork a gist
  • Save mwickett/2aecfdeea40daa07d39e11922ae1fe20 to your computer and use it in GitHub Desktop.
Save mwickett/2aecfdeea40daa07d39e11922ae1fe20 to your computer and use it in GitHub Desktop.
Formik + Apollo
import React from 'react'
import { withRouter, Link } from 'react-router-dom'
import { graphql, compose } from 'react-apollo'
import { Formik } from 'formik'
import Yup from 'yup'
import FormWideError from '../elements/form/FormWideError'
import TextInput from '../elements/form/TextInput'
import Button from '../elements/form/Button'
import { H2 } from '../elements/text/Headings'
import Wrapper from '../elements/layout/Wrapper'
import CurrentUser from '../queries/CurrentUser'
import signinUser from '../mutations/signinUser'
const handleSubmit = (payload, { props, setSubmitting, setErrors }) => {
const {email, password} = payload
props.signinUser({ variables: { email, password } })
.then((response) => {
window.localStorage.setItem('graphcoolToken', response.data.signinUser.token)
props.data.refetch()
props.history.push('/')
}).catch((e) => {
const errors = e.graphQLErrors.map(error => error.message)
console.log(errors)
setSubmitting(false)
setErrors({ email: ' ', password: ' ', form: errors })
})
}
const LoginForm = ({
handleSubmit,
errors,
touched,
values,
handleChange,
handleBlur,
isSubmitting
}) =>
<Wrapper>
<H2>Sign In</H2>
<FormWideError error={errors.form} />
<form onSubmit={handleSubmit}>
<TextInput
id='email'
type='email'
label='Email'
placeholder='you@yourdomain.com'
error={errors.email && touched.email && errors.email}
value={values.email}
onChange={handleChange}
onBlur={handleBlur}
/>
<TextInput
id='password'
type='password'
label='Password'
placeholder=''
error={errors.password && touched.password && errors.password}
value={values.password}
onChange={handleChange}
onBlur={handleBlur}
/>
<Button
primary
type='submit'
disabled={
isSubmitting ||
!!(errors.email && touched.email) ||
!!(errors.password && touched.password)
}
>
Sign In
</Button>
</form>
</Wrapper>
const LoginFormWithGraphQL = compose(
graphql(signinUser, {name: 'signinUser'}),
graphql(CurrentUser, { options: { fetchPolicy: 'network-only' } }),
Formik({
validationSchema: Yup.object().shape({
email: Yup.string()
.email('Invalid email address')
.required('Email is required'),
password: Yup.string()
.required('Password is required')
}),
mapPropsToValues: ({ variables }) => ({
...variables
}),
handleSubmit: handleSubmit,
displayName: 'Login'
})
)(LoginForm)
const LoginFormWithRouter = withRouter(LoginFormWithGraphQL)
class Login extends React.Component {
componentWillUpdate (nextProps) {
if (!this.props.data.user && nextProps.data.user) {
this.props.history.push('/dashboard')
}
}
render () {
if (this.props.data.loading) {
return (<div></div>)
}
return (
<div>
<LoginFormWithRouter variables={{ email: '', password: '' }} />
<p>Don't have an account? <Link to='/signup'>Create one now</Link></p>
</div>
)
}
}
// Set network fetch policy to network only for security reasons
export default graphql(CurrentUser, { options: { fetchPolicy: 'network-only' } })(withRouter(Login))
@jaredpalmer
Copy link

Wow! First, this is awesome. Super cool to see that apollo and compose works pretty much as expected with formik

Some thoughts / tips:

Formik-related

  • You don't need to pass down variables as a prop to Formik because there isn't any initial state (a login form should usually be empty to start). Instead, you can just set email and password to '' inside of your mapPropsToValues:
const mapPropsToValues: () => ({ email: '', password: '' })
  • Instead of a add custom key on your errors object (what you have as form), you might want to write a helper method that reduces the array of your graphql errors to an object whose keys match errors/values exactly. Keeping the same error shape is very useful when you inevitably need to work with custom inputs and share them across projects. If you need to set some arbitrary state, you can use Formik's setStatus, which will become available as props.status in your form.

General React stuff

  • You probably don't need anything other than history in your form, so no need to use withRouter() twice. You can either: pass history down as a prop: <LoginForm history={this.props.history} />. Or, better yet, lift up the route transition call up to <Login/> (maybe call it handleLoginSuccess() and then pass it back down to your form: <LoginForm onLoginSuccess={this.handleLoginSuccess}/>. Then you can call props.onLoginSuccess() inside of handleSubmit at the right moment. This removes React Router as a hard dependency of <LoginForm/>, making testing / snapshotting much much easier.

  • Last thing...an unfortunately not well-known fact about writing promises in React: using promise.catch(e => {}) can lead to inconsistent state especially in something like dispatch (which is what apollo uses under the hood).

    I cringe when I see .then(() => dispatch(...)).catch(...) in React projects.
    If a component throws during dispatch, you’ll get into catch.
    --Dan Abramov

    Instead of using .catch, you should try to always handle errors in promises by passing the error handler as the second argument to .then():

    So above it would be:

const handleSubmit = (payload, { props, setSubmitting, setErrors }) => {
  const {email, password} = payload
  props.signinUser({ variables: { email, password } })
    .then(
      (response) => {
        window.localStorage.setItem('graphcoolToken', response.data.signinUser.token)
        props.data.refetch()
        props.history.push('/')
      }, 
      (e) => {
        const errors = e.graphQLErrors.map(error => error.message)
        console.log(errors)
        setSubmitting(false)
        setErrors({ email: ' ', password: ' ', form: errors })
     }
   )

Formik does this internally, so you're probably fine as it is now, but I highly suggest using the aforementioned pattern whenever possible to be safe and ensure proper error handling / state updates. Note this may change when the new error boundary API ships in Fiber.

@mwickett
Copy link
Author

Wow. For some reason I completely missed seeing your response Jared. Lost in the pile of GH notifications.

Just wanted to express my sincerest appreciation for taking the time to write that up. I'm relatively new to React (clearly) and guidance and advice from you is really helpful. I had no idea about the error handler / catch thing being an issue. I'll definitely use that pattern.

@churcho
Copy link

churcho commented Oct 20, 2018

I have tried to use this method and was able to code my login. I have however hit a snag, I am trying to access the Mutation render prop update function to update my local cache but can't. Have you faced this and if so, how did you go about it?

<Mutation
      mutation={SIGN_IN_USER}
      update={(cache, { data: { token } }) => {
       ...
      }}
    >

@churcho
Copy link

churcho commented Oct 20, 2018

I found the solution, by adding an update function right after variables in the mutation.

.mutate({
      variables: { email, password },

      update: (cache, { data: { token } }) => {
        ...
      },
    })

@Pruxis
Copy link

Pruxis commented Dec 8, 2018

use withFormik instead of Formik if you are using the latest version of Formik as of now

@benbowler
Copy link

This was also a useful example for me working through the same kind of task: https://github.com/helabenkhalfallah/react-apollo-link-graphql/blob/master/src/app/users/components/UserAddForm.jsx

@gajus
Copy link

gajus commented Mar 8, 2019

@benbowler, in your example, how do you reset setSubmitting in case of an error?

This is the code that I have at the moment:

// @flow

import React, {
  useState
} from 'react';
import Textarea from 'react-textarea-autosize';
import gql from 'graphql-tag';
import {
  Query,
  Mutation
} from 'react-apollo';
import Modal from 'react-modal';
import {
  Formik,
  Field
} from 'formik';
import {
  createStyleName
} from '../factories';
import {
  CreateProbeMutation
} from '../mutations';
import styles from './ProbeListView.module.scss';
import ProbeMonitorCard from './../components/ProbeMonitorCard';
import PrimaryButton from './../components/PrimaryButton';
import FormField from './../components/FormField';
import InlineError from './../components/InlineError';

const styleName = createStyleName(styles);

const CreateProbeForm = (props) => {
  console.log('CreateProbeForm.props', props);

  return <form className={styleName('form-body')} onSubmit={props.handleSubmit}>
    <Field
      label='Description'
      type='textarea'
      name='description'
      description='Describe what the probe is checking, e.g. "There must be no queries running longer than 30 seconds." You can use bold, italic and inline-code markdown in the description.'
      component={FormField}
    />

    <Field
      label='Query'
      type='textarea'
      name='query'
      description='SQL query. Each row returned from the query is interpreted as a warning. Under normal working conditions, query must return no results.'
      component={FormField}
    />

    <div className={styleName('form-field button-group')}>
      <PrimaryButton
        type='submit'
        inProgress={props.isSubmitting}
      >Create Probe</PrimaryButton>
    </div>
  </form>;
};

export default () => {
  const [createProbeIsOpen, setCreateProbeIsOpen] = useState(false);
  const [queryFilter, setQueryFilter] = useState('');

  return <>
    <div className={styleName('query-filter')}>
      <input
        id='query-filter'
        placeholder='Search for a query'
        type='text'
        className={styleName('text-input')}
        onChange={(event) => {
          setQueryFilter(event.target.value);
        }}
      />
    </div>

    <Modal
      isOpen={createProbeIsOpen}
      onRequestClose={() => {
        setCreateProbeIsOpen(false);
      }}
      className={styleName('modal')}
      overlayClassName={styleName('overlay')}
      >
      <div className={styleName('modal-header')}>
        <div className={styleName('name')}>Create probe</div>
      </div>
      <div className={styleName('modal-body')}>
        <Mutation
          onCompleted={() => {
            this.props.history.push('/');
          }}
          mutation={CreateProbeMutation}>
          {(createProbe, {loading, error, data}) => {
            let errorElement;

            if (error) {
              

              console.log('error', error);

              errorElement = <InlineError
                error={error}
              />;
            }

            return <>
              {errorElement}
              <Formik
                initialValues={{
                  description: '',
                  query: ''
                }}
                onSubmit={(values) => {
                  createProbe({
                    variables: {
                      databaseConnectionId: 1,
                      description: values.description,
                      query: values.query,
                      executionInterval: 60,
                      executionTimeout: 60,
                      labelIds: []
                    }
                  })
                }}
                render={CreateProbeForm}
              />
            </>;
          }}
        </Mutation>
      </div>
    </Modal>

    <Query
        query={gql`
          {
            probes(first : 10) {
              edges {
                node {
                  id
                  description
                }
              }
            }
          }
        `}
      >{(props) => {
        if (props.loading) {
          return <div>Loading</div>;
        }

        const probeListItems = props.data.probes.edges
          .filter((edge) => {
            return edge.node.description.toLowerCase().includes(queryFilter.toLowerCase());
          })
          .map(({node}) => {
            return <li key={node.id}>
              <ProbeMonitorCard
                {...node}
              />
            </li>;
          });

        return <div>
          <ol className={styleName('probe-monitor-list')}>
            {probeListItems}
          </ol>
        </div>;
      }}</Query>

    <div className={styleName('navigation')}>
      <PrimaryButton onClick={() => { setCreateProbeIsOpen(true); }}>
        Create probe
      </PrimaryButton>
    </div>
  </>;
};

@gajus
Copy link

gajus commented Mar 8, 2019

I could just do this:

onSubmit={(values, {setSubmitting}) => {
  createProbe({
    variables: {
      databaseConnectionId: 1,
      description: values.description,
      query: values.query,
      executionInterval: 60,
      executionTimeout: 60,
      labelIds: []
    }
  })
  .catch((error) => {
    setSubmitting(false);
  })
}}

Not sure if thats the way to go, though.

@rigalpatel001
Copy link

Use the below code example. It works fine.

import React,{ useState } from "react";
import { StyleSheet, Image ,Text } from "react-native";
import * as Yup from "yup";
import { gql, useQuery,useMutation } from '@apollo/client';

import Screen from "../components/Screen";
import { ErrorMessage,Form, FormField, SubmitButton } from "../components/forms";
import ActivityIndicator from "../components/ActivityIndicator";
import useAuth from "../auth/useAuth";


const validationSchema = Yup.object().shape({
  email: Yup.string().required().email().label("Email"),
  password: Yup.string().required().min(4).label("Password"),
});

const LOG_IN = gql`
mutation SignInMutation($signInLogin: String!, $signInPassword: String!) {
  signIn(login: $signInLogin, password: $signInPassword) {
    token
  }
}
`;
function LoginScreen(props) {

  const auth = useAuth();
  const [loginFailed, setLoginFailed] = useState(false);
  const [isloading, setIsloading] = useState(false);

  const handleSubmit = async ({ email, password }) => {
    setIsloading(true);
   try {
    const result = await signIn({ variables: { signInLogin: email,signInPassword:password } });
    setIsloading(false);
    auth.logIn(result.data.signIn.token);
   } catch (error) {
    console.log("Error", error);
    setLoginFailed(true);
    setIsloading(false);
   }
  };

   const [signIn, { data, loading, error }] = useMutation(LOG_IN);
   //if (loading) return <Text>Submitting...</Text>
   //if (error) setLoginFailed(true);

  return (
    <>
    <ActivityIndicator visible={isloading} />
    <Screen style={styles.container}>
      <Image style={styles.logo} source={require("../assets/logo.png")} />
      <Form
        initialValues={{ email: "", password: "" }}
        onSubmit={handleSubmit}
        validationSchema={validationSchema}
      >
         <ErrorMessage
          error="Invalid email and/or password."
          visible={loginFailed}
        />
        <FormField
          autoCapitalize="none"
          autoCorrect={false}
          icon="email"
          keyboardType="email-address"
          name="email"
          placeholder="Email"
          textContentType="emailAddress"
        />
        <FormField
          autoCapitalize="none"
          autoCorrect={false}
          icon="lock"
          name="password"
          placeholder="Password"
          secureTextEntry
          textContentType="password"
        />
        <SubmitButton title="Login" />
      </Form>
    </Screen>
    </>
  );
}

const styles = StyleSheet.create({
  container: {
    padding: 10,
  },
  logo: {
    width: 200,
    height: 80,
    alignSelf: "center",
    marginTop: 50,
    marginBottom: 20,
  },
});

export default LoginScreen;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment