Modernize db model relationships, auth checks #109

bdewilde · 2024-05-06T01:54:07Z

tidies up api v1 init module
updates legacy "dynamic" db model relationships to equivalent "write_only" relationships, and updates their usage in api resources accordingly
updates various other db model relationships to be more performant and clear
adds User.collaborators method, to get all users who work together on shared reviews
consolidates some is-allowed checks for api resources... this was more awkward than hoped for, so tbd on if this will expand
allows admins to delete users, and updates unit tests accordingly
un-skips citation import unit tests

and allow admins to delete users if necessary

bdewilde added 14 commits May 5, 2024 13:36

refactor: Move api def, tidy resource imports

712d010

feat: Add empty api v2 def

5546196

refactor: Share auth logic in user api resources

0702d4a

and allow admins to delete users if necessary

tests: Update user delete test to work for admins

984bf96

refactor: Share access logic in review api

40457e6

tests: Stop skipping citation import tests

269151a

feat: Update many lazy model relationships

5b8dfaf

fix: Update model usage for lazy rel api change

bb88d15

feat: Update more lazy db relationships

c606714

refactor: Use write-only rel for study screenings

6afd76d

feat: Update user review assoc lazy db rel

f747a78

fix: Update rel usage for lazy api change

59e1a49

feat: Add collaborators method to user model

bf657a4

feat: Kill api v2, flask app barfed on it hard

fa87616

bdewilde changed the base branch from main to develop May 6, 2024 01:54

bdewilde marked this pull request as ready for review May 6, 2024 02:01

bdewilde merged commit 9d97f2c into develop May 6, 2024
3 checks passed

bdewilde deleted the burton-lay-groundwork-api-v2 branch May 6, 2024 02:08

Provide feedback