Skip to content
This repository was archived by the owner on Apr 13, 2023. It is now read-only.

Flow fixes #952

Merged
merged 3 commits into from
Aug 4, 2017
Merged

Flow fixes #952

merged 3 commits into from
Aug 4, 2017

Conversation

dozoisch
Copy link
Contributor

@dozoisch dozoisch commented Aug 4, 2017

Fixes: #931

See each commit message for description!

- because fix is an interface and has two types, it needs to be set as
covariant (using read-only notation)
- see facebook/flow#4084
- this allows for containers to have state or default props.
@apollo-cla
Copy link

@dozoisch: Thank you for submitting a pull request! Before we can merge it, you'll need to sign the Meteor Contributor Agreement here: https://contribute.meteor.com/

@dozoisch
Copy link
Contributor Author

dozoisch commented Aug 4, 2017

Done

@jbaxleyiii jbaxleyiii merged commit 1c66c65 into apollographql:master Aug 4, 2017
@dozoisch dozoisch deleted the flow_fixes branch August 4, 2017 21:16
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.

3 participants