Skip to content
Snippets Groups Projects
user avatar
Ngoc Khuat authored
* add is_group_manager and hydrate it

* update test title

* fix namespaces and 1 test

* update docstring and make sure the is-group-manager? is converted to boolean in any db

* even more docstring

* fix ns

* Fix a test appeared when merge with master

* add tests and more api enforcement

* appease linter and make sure the test can run in ee

* update by membership id

* one dot

* update docs

* make is_group_manager optional

* hydrate is_group_manager when get single group

* - Split the `check-group-manager` into 2 functions
- Address Noah's comments

* Update enterprise/backend/test/metabase_enterprise/advanced_permissions/api/group_manager_test.clj

Co-authored-by: default avatarNoah Moss <32746338+noahmoss@users.noreply.github.com>

* change is_group_manager in to bool instead of boolstring

* remove debug code

Co-authored-by: default avatarNoah Moss <32746338+noahmoss@users.noreply.github.com>
d3807aff
History
Name Last commit Last update