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

Conversation

@wawhal
Copy link
Contributor

@wawhal wawhal commented Mar 31, 2020

Description

Single row mutations were introduced in v1.2.0-beta.2. The UI however was not updated to allow customising root fields for these single row mutations. That's what this PR does.

Changelog

  • CHANGELOG.md is updated with user-facing content relevant to this PR.

Affected components

  • Server
  • Console
  • CLI
  • Docs
  • Community Content
  • Build System
  • Tests
  • Other (list it)

Related Issues

#4203

Steps to test and verify

Try customising root fields for single row mutations

@wawhal wawhal added k/enhancement New feature or improve an existing feature c/console Related to console labels Mar 31, 2020
@wawhal wawhal requested a review from a team as a code owner March 31, 2020 12:04
@wawhal wawhal requested a review from a team as a code owner March 31, 2020 12:08
@hasura-bot
Copy link
Contributor

Review app for commit e89b1db deployed to Heroku: https://hge-ci-pull-4254.herokuapp.com
Docker image for server: hasura/graphql-engine:pull4254-e89b1db6

}}
deleteByPkOnChange={e => {
onChange('delete_by_pk', e.target.value);
}}
Copy link
Contributor

Choose a reason for hiding this comment

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

It definitely looks like it needs some abstraction on onChange handlers 🙈

@rikinsk
Copy link
Member

rikinsk commented Mar 31, 2020

Added changelog in #4253

@hasura-bot
Copy link
Contributor

Review app for commit b674527 deployed to Heroku: https://hge-ci-pull-4254.herokuapp.com
Docker image for server: hasura/graphql-engine:pull4254-b674527f

@rikinsk rikinsk merged commit 0abb6c2 into hasura:master Mar 31, 2020
@hasura-bot
Copy link
Contributor

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

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

Labels

c/console Related to console k/enhancement New feature or improve an existing feature no-changelog-required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants