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

Conversation

@surendran82
Copy link
Contributor

@surendran82 surendran82 commented Feb 20, 2019

fix int-#76

Description

Updated landing for remote schema and event trigger

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.

@dsandip dsandip changed the title Schema and event triggers page update Schema and event triggers page update (closes hasura/graphql-engine-internal#76) Feb 20, 2019
@dsandip dsandip changed the title Schema and event triggers page update (closes hasura/graphql-engine-internal#76) Schema and event triggers page update ( closes hasura/graphql-engine-internal#76 ) Feb 20, 2019
@dsandip dsandip changed the title Schema and event triggers page update ( closes hasura/graphql-engine-internal#76 ) Schema and event triggers page update Feb 20, 2019
@hasura-bot
Copy link
Contributor

Review app for commit 04858b7 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1630-04858b7

@hasura-bot
Copy link
Contributor

Review app for commit 7d79c8a deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1630-7d79c8a

@hasura-bot
Copy link
Contributor

Review app for commit 3770890 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1630-3770890

@hasura-bot
Copy link
Contributor

Review app for commit 63766b0 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1630-63766b0

@hasura-bot
Copy link
Contributor

Review app for commit d4804a6 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1630-d4804a6

@hasura-bot
Copy link
Contributor

Review app for commit 107d36a deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1630-107d36a

@tirumaraiselvan
Copy link
Contributor

@surendran82 can you close this DRAFT PR and open a normal one?

@hasura-bot
Copy link
Contributor

Review app for commit 0ab82f6 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1630-0ab82f6

@surendran82 surendran82 marked this pull request as ready for review March 5, 2019 07:26
@hasura-bot
Copy link
Contributor

Review app for commit d1a7173 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1630-d1a7173

@coco98
Copy link
Contributor

coco98 commented Mar 5, 2019

Suggestions:

  1. Remove first section on what is remote schema if there's >=1 remote schema. Same for event triggers
  2. The glitch page leads to the readme, which is empty. Let's put some instructions there.

screenshot 2019-03-05 at 3 37 03 pm

@hasura-bot
Copy link
Contributor

Review app for commit e3851ff deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1630-e3851ff

@hasura-bot
Copy link
Contributor

Review app for commit c405ff4 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1630-c405ff4

const { dispatch, migrationMode } = this.props;
const { dispatch, migrationMode, customResolverList } = this.props;
const showFirstSection = customResolverList.resolvers.length ? false : true;
console.log(showFirstSection);
Copy link
Member

Choose a reason for hiding this comment

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

Remove console.log

const node = require('./Node.svg');
const Rectangle = require('./Rectangle.svg');

// const lightGrayArrow = require('./light-gray-arrow.svg');
Copy link
Member

Choose a reason for hiding this comment

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

remove source image if its part of code

Copy link
Member

@praveenweb praveenweb left a comment

Choose a reason for hiding this comment

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

Remove images, console.log

@hasura-bot
Copy link
Contributor

Review app for commit 5f6a0d7 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1630-5f6a0d7

rikinsk added 4 commits March 7, 2019 16:35
 Conflicts:
	console/src/components/Common/Common.scss
	console/src/components/Common/TextAreaWithCopy/TextAreaWithCopy.js
	console/src/components/Common/TextAreaWithCopy/TextAreaWithCopy.scss
	console/src/components/Services/CustomResolver/CustomResolver.scss
	console/src/components/Services/CustomResolver/Landing/CustomResolver.js
	console/src/components/Services/EventTrigger/Add/AddTrigger.js
	console/src/components/Services/EventTrigger/Schema/Schema.js
@hasura-bot
Copy link
Contributor

Review app for commit b7980f0 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1630-b7980f0

tirumaraiselvan
tirumaraiselvan previously approved these changes Mar 7, 2019
tirumaraiselvan
tirumaraiselvan previously approved these changes Mar 7, 2019
@rikinsk-zz rikinsk-zz merged commit 6a4a33f into hasura:master Mar 7, 2019
@hasura-bot
Copy link
Contributor

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

@hasura-bot
Copy link
Contributor

Review app for commit d951423 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com
Docker image for server: hasura/graphql-engine:pull1630-d951423

hasura-bot pushed a commit that referenced this pull request Feb 13, 2025
<!-- The PR description should answer 2 important questions: -->

### What

In GraphQL websockets, client headers from the handshake request are
ignored. It made subscriptions unusable in private DDN setup where
gateway sets `x-ddn-*` headers useful to authenticate the requests using
PAT in webhooks. This PR address this issue by involving all client
headers received from handshake in webhook request. These headers also
included in other upstream requests (plugins, NDC etc.).

### How
- Persist handshake headers in websocket connection context. Use them
while authenticating `connection_init` message.
- Add tests to confirm this behavior.

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

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants