Possible Bug: Computed Field Permission is not being inferred

Hi there,

I have a computed field, order, for which I believe the inferred permission is incorrect. If my understanding is correct, the permission box is meant to be checked and disabled if the computed field permissions are inferred from the table from which it is derived. However, it is unchecked and not disabled.

As can be seen below, the permission for the value returned by the sql function is enabled on the table that the computed field is derived from:

I am doing metadata programatically so I have to come to the console afterwards and check this box in order for things to work at all!

I have seen this pull request, https://github.com/hasura/graphql-engine/pull/5741, maybe this issue I am having relates to this?

We are using 1.3.2.

Thanks!

[LydiaF]

Dear Gavin,

Thank you so much for replying so quickly and indeed for helping me fix the issue!!

It was all my fault.

Instead of doing this :

  permission:  {
    computed_fields: [
      'card_order'
    ],
    columns: [...],
    ...
  }

  permission:  {
    columns: ['card_order', ...],
  }

Basically I didn’t know permission.computed_fields was thing. Had a quick check of the docs there and can only see “For scalar computed fields, permissions are managed similar to the columns permissions of the table.” but maybe I’m missing it… I was at the time certainly.

In any case I am glad as now I understand how to find bugs a bit better!

A BIG THANK YOU!

Lydia

[GavinRay97]

That’s awesome to hear! Glad it was something able to be figured out =D