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

Update electronic manifest service #678

Open
dpgraham4401 opened this issue Jan 10, 2024 · 0 comments
Open

Update electronic manifest service #678

dpgraham4401 opened this issue Jan 10, 2024 · 0 comments
Labels
enhancement New feature or enhancement question/discussion Further information is requested server Related to backend workings (Django/Rest API) test Unit tests are, or need to be, included

Comments

@dpgraham4401
Copy link
Member

🚀 Feature Request

Add a service to the haztrak back end for updating a manifest in e-Manifest.

Designing the REST endpoint, I'm not sure what is a better approach.

  1. /api/.../manifest/{mtn}
  2. /api/.../manifest/emanifest/{mtn}

We are currently implementing the 2nd option for our emanifest create (POST) service. If we implemented the same for the update service the endpoint design would looks as follows:

/api/.../manifest/{mtn} would update the haztrak database but not e-Manifest.
The opposite would be true for /api/.../manifest/emanifest/{mtn}. This would be nice because it would provide a clear separation of duties. The endpoints would always returns the same schema/resource definition (/manifest/{mtn} would return a Manifest JSON object, /manifest/emanifest/{mtn} would return a information about an asynchronous task launched to update e-Manifest).

further research required

@dpgraham4401 dpgraham4401 added the enhancement New feature or enhancement label Jan 10, 2024
@github-project-automation github-project-automation bot moved this to 🆕 ToDo in Haztrak Jan 10, 2024
@dpgraham4401 dpgraham4401 added question/discussion Further information is requested server Related to backend workings (Django/Rest API) test Unit tests are, or need to be, included labels Jan 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or enhancement question/discussion Further information is requested server Related to backend workings (Django/Rest API) test Unit tests are, or need to be, included
Projects
No open projects
Status: 🆕 ToDo
Development

No branches or pull requests

1 participant