Data graph permissions decoding -- removing redundancies (#27665)
This work removes a spec and a p.schema for data graph permissions. There are still some specs but they are for execution, and collection graph permissions. Those should be targeted next. * decoded + verified FIXME * still good * some cleanup * a little more cleaning up * bit more cleaning * converted update-group-permissions! -> mu/defn * use the correct shape for new-group-perms * remove bunch of pschemas * linter appeasement * update test for setting dissallowed schema * respond to review comments * fix some tests + improve mu/defn error messages * a lint fix * lint fix * read and query are optional * update expected error message to the improved one * stop calling trs at the top level * fix function signature * improve test
Showing
- deps.edn 1 addition, 1 deletiondeps.edn
- enterprise/backend/test/metabase_enterprise/advanced_permissions/models/permissions/block_permissions_test.clj 2 additions, 2 deletions...permissions/models/permissions/block_permissions_test.clj
- src/metabase/api/permission_graph.clj 96 additions, 80 deletionssrc/metabase/api/permission_graph.clj
- src/metabase/api/permissions.clj 15 additions, 7 deletionssrc/metabase/api/permissions.clj
- src/metabase/models/permissions.clj 15 additions, 42 deletionssrc/metabase/models/permissions.clj
- src/metabase/util/malli.clj 16 additions, 12 deletionssrc/metabase/util/malli.clj
- test/metabase/models/permissions_test.clj 2 additions, 2 deletionstest/metabase/models/permissions_test.clj
Loading
Please register or sign in to comment