aboutsummaryrefslogtreecommitdiffstats
path: root/rust
diff options
context:
space:
mode:
authorBryan Newbold <bnewbold@robocracy.org>2018-09-09 10:06:49 -0700
committerBryan Newbold <bnewbold@robocracy.org>2018-09-09 10:06:49 -0700
commita29beab0683d77086cc1b431779d0540dc5a9b49 (patch)
tree893982496701b750f1f693ea846883ca4a7f1558 /rust
parent109f82169ef2a141b5d859e23d3ba052cec99fcb (diff)
downloadfatcat-a29beab0683d77086cc1b431779d0540dc5a9b49.tar.gz
fatcat-a29beab0683d77086cc1b431779d0540dc5a9b49.zip
update TODO
Diffstat (limited to 'rust')
-rw-r--r--rust/TODO20
1 files changed, 11 insertions, 9 deletions
diff --git a/rust/TODO b/rust/TODO
index 05ad76d8..ad4b1241 100644
--- a/rust/TODO
+++ b/rust/TODO
@@ -1,23 +1,26 @@
+finish refactor:
- database_entity_crud -> api_entity_crud
-- merge autoaccept branch in with http-verbs branch
+x merge autoaccept branch in with http-verbs branch
+- direct CRUD calls from api_wrappers (except, maybe, batch?)
+ => generally, standardize "edit" actions
+- FatCatId and edit context between wrappers and handlers
+- review editgroup accept code
+
+verbs:
+- enforce "previous_rev" required in updates
+
- fatcat_api -> fatcat_api_schema (or spec? models? types?)
- fatcat -> fatcat-api-server
-- refactor rev creation (from an entity) into it's own function, across the board
- => attach to database module structs?
- => should make cockroachdb compatible (single use of CTE)
- => usable from both "update" and "create"
- editgroup param to update
=> also for creation? for consistency
-
- editor_id vs. editor username; return editor_id (in addition to name?)
later:
+- have editgroup_id be a request-level param everywhere (not entity-level; for batch)
- editgroup: state to track review status?
- re-implement old python tests
- enforce "no editing if editgroup accepted" behavior
-- refactor entity creation/editing (DRY)
-- refactor to allow Result<> in all handlers
- real auth
- metrics, jwt, config, sentry
- ansible/deployment/DNS story
@@ -29,5 +32,4 @@ schema/api questions:
- "types"
- define release field stuff
- what should entity POST return? include both the entity and the edit?
-- PUTs (updates) to entities