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

Conversation

@rakeshkky
Copy link
Member

Description

Affected components

  • Server
  • Tests

Related Issues

close #2054

Solution and Design

Fetch dependant columns from OpExpG a for column comparison operators.

Steps to test and verify

Create permission with column operators and try to delete a column involved in permission rule through the console or run_sql query. This should throw a dependency error.

@rakeshkky rakeshkky added s/ok-to-merge Status: This pull request can be merged to master c/server Related to server labels Apr 24, 2019
@rakeshkky rakeshkky requested a review from 0x777 as a code owner April 24, 2019 05:06
@rakeshkky rakeshkky self-assigned this Apr 24, 2019
@netlify
Copy link

netlify bot commented Apr 24, 2019

Deploy preview for hasura-docs ready!

Built with commit bd15cf9

https://deploy-preview-2055--hasura-docs.netlify.com

@hasura-bot
Copy link
Contributor

Review app for commit facef2f deployed to Heroku: https://hge-ci-pull-2055.herokuapp.com
Docker image for server: hasura/graphql-engine:pull2055-facef2f

@hasura-bot
Copy link
Contributor

Review app for commit bd15cf9 deployed to Heroku: https://hge-ci-pull-2055.herokuapp.com
Docker image for server: hasura/graphql-engine:pull2055-bd15cf9

@0x777 0x777 merged commit 733101b into hasura:master Apr 24, 2019
@hasura-bot
Copy link
Contributor

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

polRk pushed a commit to polRk/graphql-engine that referenced this pull request Feb 12, 2020
hasura-bot pushed a commit that referenced this pull request Jul 17, 2025
<!-- The PR description should answer 2 important questions: -->

### What

We have the same pattern for models, commands and object types used in
an output context, so let's do the same for objects in an input context.
Essentially a no-op that makes implementing input type permissions for
authorization rules simpler.

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

Labels

c/server Related to server s/ok-to-merge Status: This pull request can be merged to master

Projects

None yet

Development

Successfully merging this pull request may close these issues.

evaluate dependencies for column operators in permissions

3 participants