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

Remove /api/operations/:id #467

Closed
jrozner opened this issue Feb 21, 2022 · 0 comments · Fixed by #471
Closed

Remove /api/operations/:id #467

jrozner opened this issue Feb 21, 2022 · 0 comments · Fixed by #471
Labels
question Further information is requested triage Issue needs triage

Comments

@jrozner
Copy link
Member

jrozner commented Feb 21, 2022

It looks like we still provide this endpoint for the api server and are still returning the actual id when we request /api/operations rather than just the slug. There is a note that we should remove the id when the client no longer needs it. I doesn't seem like the client actually uses that endpoint and everything else is done via slug. Is this endpoint just safe to remove? If not, what code needs to be updated?

@jrozner jrozner added question Further information is requested triage Issue needs triage labels Feb 21, 2022
@JoelAtDeluxe JoelAtDeluxe linked a pull request Feb 22, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested triage Issue needs triage
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant