Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[ENH] add delete operation to all endpoints #165

Merged
merged 35 commits into from
Dec 8, 2021
Merged

[ENH] add delete operation to all endpoints #165

merged 35 commits into from
Dec 8, 2021

Conversation

jdkent
Copy link
Member

@jdkent jdkent commented Dec 2, 2021 •
edited
Loading

allow users to delete their studies/analyses, etc.

dependent on #163

  • test basic functionality for crud on all endpoints

ref #161

analyses/ POST
Not currently working. Need this to create analyses

should be working now

analyses/ DELETE
Not currently implemented. Need this to delete analyses

implemented

points/ POST
Not currently working. Need this to create points

implemented

points/ DELETE
Not currently implemented. Need this to delete points

implemented

datasets/ POST
Working. Successfully creates object but there is no way to feed in a name and description when creating the object.

fixed response body

datasets/ DELETE
not currently implemented. Need this to delete datasets
implemented

@jdkent
bump version of postgres
b943a8a
@jdkent
change pip to pip3
3e0a3a6
@jdkent
reorganize and comment
fbd241d
@jdkent
remove files
2789c8c
@jdkent
do not use globals
b63b9fa
@jdkent
tmp commit
b3fa350
@jdkent
fix merge
894c1b1
@jdkent
add annotation at correct level
4d29d13
@jdkent
add concept of _parent and _linked
76ce684 
_parent: when "deep" cloning, do not want _parent attributes to be set since they will have a new reference
_linked: not necessarily a _parent or _nested within another resource, but otherwise related and need to maintain this relationship
@jdkent
change string or nested to return None when a "parent attribute" is b…
d117f6b 
…eing copied so a new object can be the parent
@jdkent
modify tests to represent new ownership requirements
a021cd6
@jdkent
add post points api
6abea4c
@jdkent
add delete operations to openapi
325d941
@jdkent
fix study and analysis table so that deletes cascade to subcomponents
a455ea8
@jdkent
add delete function and fix parent on imagelistview
4b0b7a7
@jdkent
fix image schema
b7db381
@jdkent
wip: add complete tests for endpoints
d8c18cd
@jdkent
remove model inits
7f4719a
@jdkent
do not assume dataset_id exists
6abdb63
@jdkent
replace user_studies with user_data
7151a26
@jdkent
wip: test crud operations in bulk
9c69b34
@jdkent
update tests
449aefa
@jdkent
move relationships to Condition and Analysis tables
dc71e4b
@jdkent
fix some bugs
2c68dbc
@jdkent
fix nested representation for DatasetView
000771d
@jdkent
add data_key and remove preload and postdump
ae1426c
@jdkent
test crud operations across multiple endpoints
24ce2b3
@jdkent
flake8 fix
cf55cd8
@jdkent
do not delete conditions/analyses if analyses/conditions are deleted
d109105
@jdkent
import db from database not from core
dde3501
@jdkent jdkent changed the title [ENH, wip] add delete operation to all endpoints [ENH] add delete operation to all endpoints Dec 8, 2021
@jdkent jdkent merged commit 50c5c07 into master Dec 8, 2021
This was referenced Dec 8, 2021
Closed
Closed
Closed
@jdkent jdkent mentioned this pull request Jan 12, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jdkent