-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Schema and event triggers page update #1630
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Review app for commit 04858b7 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com |
|
Review app for commit 7d79c8a deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com |
|
Review app for commit 3770890 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com |
|
Review app for commit 63766b0 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com |
|
Review app for commit d4804a6 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com |
|
Review app for commit 107d36a deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com |
|
@surendran82 can you close this DRAFT PR and open a normal one? |
|
Review app for commit 0ab82f6 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com |
…last line text change
|
Review app for commit d1a7173 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com |
|
Review app for commit e3851ff deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com |
|
Review app for commit c405ff4 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com |
| const { dispatch, migrationMode } = this.props; | ||
| const { dispatch, migrationMode, customResolverList } = this.props; | ||
| const showFirstSection = customResolverList.resolvers.length ? false : true; | ||
| console.log(showFirstSection); |
There was a problem hiding this comment.
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'); |
There was a problem hiding this comment.
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
praveenweb
left a comment
There was a problem hiding this 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
|
Review app for commit 5f6a0d7 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com |
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
|
Review app for commit b7980f0 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com |
|
Review app https://hge-ci-pull-1630.herokuapp.com is deleted |
|
Review app for commit d951423 deployed to Heroku: https://hge-ci-pull-1630.herokuapp.com |
<!-- 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
fix int-#76
Description
Updated landing for remote schema and event trigger
What component does this PR affect?
Requires changes from other components? If yes, please mark the components:
Related Issue
Solution and Design
Type
Checklist: