Skip to content
This repository has been archived by the owner on Sep 11, 2019. It is now read-only.

Extract separate prepare function #72

Merged
merged 3 commits into from
Jan 11, 2018
Merged

Extract separate prepare function #72

merged 3 commits into from
Jan 11, 2018

Conversation

ecwyne
Copy link
Collaborator

@ecwyne ecwyne commented Jan 11, 2018

Re #62 #63

import {prepare} from '@gramps/gramps';

const {schema, context, addContext} = prepare(...);
// do stuff with `schema`
app.use('/graphql', addContext) // req.gramps = {...[gramps context stuff]}

@ecwyne ecwyne requested a review from jlengstorf January 11, 2018 17:15
jlengstorf
jlengstorf previously approved these changes Jan 11, 2018
Copy link
Member

@jlengstorf jlengstorf left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks @kbrandwijk and @ecwyne!

@jlengstorf jlengstorf merged commit 3bec290 into master Jan 11, 2018
@jlengstorf jlengstorf deleted the feat/prepare-method branch January 11, 2018 22:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants