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

expose remote_id, source field on /ingredientinfo/ api #1713

Closed
Dieterbe opened this issue Jul 4, 2024 · 2 comments
Closed

expose remote_id, source field on /ingredientinfo/ api #1713

Dieterbe opened this issue Jul 4, 2024 · 2 comments
Assignees
@rolandgeider

Comments

@Dieterbe
Copy link
Contributor

Dieterbe commented Jul 4, 2024

goal: display OFF url's and USDA urls for an ingredient in the UI
@Dieterbe
Copy link
Contributor Author

Dieterbe commented Jul 4, 2024

actually, why do we even need 2 api's, ingredientinfo and ingredient ? they are very similar, can we just simplify and have just 1 endpoint?

@Dieterbe Dieterbe mentioned this issue Jul 4, 2024
Merged
@rolandgeider
Copy link
Member

I had already commited this, didn't even open a ticket: bb99d6e

The *info endpoints are all read-only, while the others that map directly to db objects are writable

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rolandgeider rolandgeider

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Dieterbe @rolandgeider