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

Subs #256

Merged
merged 3 commits into from
Oct 8, 2016
Merged

Subs #256

merged 3 commits into from
Oct 8, 2016

Conversation

jbaxleyiii
Copy link
Contributor

@jbaxleyiii jbaxleyiii commented Oct 8, 2016

Integrated subscriptions!

@graphql(gql`subscription { user { name } }`)
class MyComponentWithSubscription extends Component {
  //
}

TODO:

  • Make sure all of the significant new logic is covered by tests
  • Rebase your changes on master so that they can be merged easily
  • Make sure all tests and linter rules pass
  • Update CHANGELOG.md with your change
  • If this was a change that affects the external API, update the docs and post a link to the PR in the discussion

going to quietly ship this and test it in production before adding to the api docs

@jbaxleyiii jbaxleyiii merged commit a23883f into master Oct 8, 2016
@jbaxleyiii jbaxleyiii deleted the subs branch October 8, 2016 01:21
@jbaxleyiii jbaxleyiii mentioned this pull request Oct 8, 2016
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.

1 participant