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

Conversation

@arvi3411301
Copy link
Member

Description

What component does this PR affect?

  • Server
  • Console
  • CLI
  • Docs
  • Community Content
  • Build System

Requires changes from other components? If yes, please mark the components:

  • Server
  • Console
  • CLI
  • Docs
  • Community Content
  • Build System

Related Issue

Solution and Design

Type

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Docs update
  • Community content

Checklist:

  • I have read the contributing guide and my code conforms to the guidelines.
  • This change requires a change in the documentation.
  • I have updated the documentation accordingly.
  • I have added required tests.

@arvi3411301 arvi3411301 added the c/console Related to console label Nov 28, 2018
@arvi3411301 arvi3411301 requested a review from wawhal November 28, 2018 11:55
@hasura-bot
Copy link
Contributor

Review app for commit e5d5636 deployed to Heroku: https://hge-ci-pull-1134.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1134-e5d5636

@shahidhk shahidhk requested a review from praveenweb November 30, 2018 09:29
Copy link
Contributor

@wawhal wawhal left a comment

Choose a reason for hiding this comment

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

LGTM

@shahidhk
Copy link
Member

shahidhk commented Dec 2, 2018

package-lock.json is changed. @praveenweb

@praveenweb
Copy link
Member

@shahidhk - yeah the semver package version has changed in package.json (we have freezed to a particular version) and hence package-lock needs to be updated. LGTM

@praveenweb praveenweb added the s/ok-to-merge Status: This pull request can be merged to master label Dec 3, 2018
@hasura-bot
Copy link
Contributor

Review app for commit 5502621 deployed to Heroku: https://hge-ci-pull-1134.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1134-5502621

@shahidhk shahidhk merged commit 0281a1b into hasura:master Dec 3, 2018
@hasura-bot
Copy link
Contributor

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

@arvi3411301 arvi3411301 deleted the issue-1133 branch January 3, 2019 10:55
hasura-bot pushed a commit that referenced this pull request Sep 25, 2024
…1134)

<!-- The PR description should answer 2 important questions: -->

### What

<!-- What is this PR trying to accomplish (and why, if it's not
obvious)? -->

<!-- Consider: do we need to add a changelog entry? -->

<!-- Does this PR introduce new validation that might break old builds?
-->

<!-- Consider: do we need to put new checks behind a flag? -->
- Introduce a brand new `graphql-ws` crate that implements
  - WebSocket request handling by `WebSocketServer` struct
-
[graphql-ws](https://github.com/enisdenjo/graphql-ws/blob/master/PROTOCOL.md)
protocol and handles subscriptions in async tokio tasks.
- OSS engine now handles GraphQL websockets through `GET /graphql`
handshakes.

### How

Refer to added
[architecture.md](https://github.com/hasura/v3-engine/blob/rakeshkky/graphql-ws-crate/crates/graphql-ws/architecture.md)
file.

<!-- How is it trying to accomplish it (what are the implementation
steps)? -->

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

Labels

c/console Related to console s/ok-to-merge Status: This pull request can be merged to master

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants