-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
Bump graphql@14 #58974
Bump graphql@14 #58974
Conversation
@elasticmachine merge upstream |
@elasticmachine merge upstream |
LGTM - I only pulled the PR locally and verified I could |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
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.
Sweet, thanks for the info and I'm happy to see that removing the nohoist
doesn't cause any issues and actually prevents issues as we fiddle a little less with the yarn resolution. LGTM
# Conflicts: # package.json # x-pack/legacy/plugins/siem/public/mock/timeline_results.ts # x-pack/legacy/plugins/siem/public/pages/timelines/timelines_page.tsx # yarn.lock
@@ -89,35 +96,6 @@ export class KibanaBackendFrameworkAdapter implements FrameworkAdapter { | |||
} | |||
} | |||
); | |||
|
|||
if (!this.isProductionMode) { |
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.
there is no separate endpoint for graphiql
anymore
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.
@patrykkopycinski does that mean that this functionality exists within graphql now, or that we're losing this functionality entirely? /cc @XavierM since I know he was most familiar with usage of this endpoint.
@@ -38,6 +38,7 @@ import { SettingFormRow } from './SettingFormRow'; | |||
import { getOptionLabel } from '../../../../../../../../../../plugins/apm/common/agent_configuration/all_option'; | |||
|
|||
function removeEmpty<T>(obj: T): T { | |||
// @ts-ignore |
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.
Could you add a description of why you're ignoring TS warning, please?
# Conflicts: # yarn.lock
💔 Build Failed
Failed CI StepsTest FailuresKibana Pipeline / kibana-oss-agent / Accessibility Tests.test/accessibility/apps/management·ts.Management Create Index pattern wizardStandard Out
Stack Trace
History
To update your PR or re-run it, just comment with: |
Summary
After #51926 was reverted I've tried to split the changes into smaller pieces and make sure there are stable.
The next step will be #58988
Checklist
Delete any items that are not applicable to this PR.
For maintainers