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.

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

Partially update an RDA DCS-compatible plan via API #278

Open
Tracked by #273
hmpf opened this issue Aug 26, 2022 · 0 comments
Open
Tracked by #273

Partially update an RDA DCS-compatible plan via API #278

hmpf opened this issue Aug 26, 2022 · 0 comments

Comments

@hmpf
Copy link
Owner

hmpf commented Aug 26, 2022

Does not depend on any of the other issues sorting under #273, but the solution will have to be updated to work with multiple RDA DCS-compatible templates if it is finished before the rest of #273.

MVP is to PUT an entire json-blob, storing the values in the existing plan instead of creating a new plan.

It would be better to PATCH a partial RDA DCS to an existing plan in order to update it. If the plan is locked, make a new version, otherwise, update the plan.

The main problem is that RDA DCS depends solely on order for identifying certain subparts, for instance costs.

For any of the subtrees that includes an identifier-subtree, the PATCH must include the identifier.

Another problem is how to remove subtrees.

The ideal solution is probably to expose any subtree as its own endpoint but this would take a lot of work since it goes against the nature of django-rest-framework.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant