-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
console: show session argument input for computed fields #5610
console: show session argument input for computed fields #5610
Conversation
Deploy preview for hasura-docs ready! Built with commit a15c65d |
Review app for commit 3b872c8 deployed to Heroku: https://hge-ci-pull-5610.herokuapp.com |
✔️ Deploy preview for hasura-docs ready! 🔨 Explore the source changes: 98df2be 🔍 Inspect the deploy logs: https://app.netlify.com/sites/hasura-docs/deploys/5fca51be7460a6000752789f 😎 Browse the preview: https://deploy-preview-5610--hasura-docs.netlify.app |
Review app for commit eef0aad deployed to Heroku: https://hge-ci-pull-5610.herokuapp.com |
Review app for commit eaf7527 deployed to Heroku: https://hge-ci-pull-5610.herokuapp.com |
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.
Left a few, mostly minor, comments.
The functionality looks good.
console/src/components/Services/ApiExplorer/OneGraphExplorer/OneGraphExplorer.js
Outdated
Show resolved
Hide resolved
console/src/components/Services/Data/TableModify/ComputedFieldsEditor.tsx
Outdated
Show resolved
Hide resolved
console/src/components/Services/Data/TableModify/ComputedFieldsEditor.tsx
Outdated
Show resolved
Hide resolved
console/src/components/Services/Data/TableModify/ComputedFieldsEditor.tsx
Outdated
Show resolved
Hide resolved
console/src/components/Services/Data/TableModify/ComputedFieldsEditor.tsx
Outdated
Show resolved
Hide resolved
console/src/components/Services/Data/TableModify/ComputedFieldsEditor.tsx
Outdated
Show resolved
Hide resolved
console/src/components/Services/Data/TableModify/ComputedFieldsEditor.tsx
Outdated
Show resolved
Hide resolved
console/src/components/Services/Data/TableModify/ModifyTable.js
Outdated
Show resolved
Hide resolved
console/src/components/Services/Data/TableModify/ComputedFieldsEditor.tsx
Outdated
Show resolved
Hide resolved
…arsam/graphql-engine into feature/session-args-computed-field
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.
Console
closes #5154
Description
This PR mainly adds a new input field for including a session argument for computed fields. This PR also includes minor fixes and also TS migration of the
ComputedFieldsEditor
fileChangelog
CHANGELOG.md
is updated with user-facing content relevant to this PR. If no changelog is required, then add theno-changelog-required
label.Affected components
Function that you use to test this PR