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

Return void on 204 #1021

Merged
merged 1 commit into from
Jan 11, 2024
Merged

Return void on 204 #1021

merged 1 commit into from
Jan 11, 2024

Conversation

bsian03
Copy link
Collaborator

@bsian03 bsian03 commented Sep 21, 2020

We don't check if the string is empty or not. This can cause vague errors when returning a 204 endpoint in a command generator for instance.

@bsian03 bsian03 force-pushed the response-fix branch 2 times, most recently from 0c95fe1 to 0478ee4 Compare November 7, 2020 23:27
Flicksie pushed a commit to PolestarLabs/polaris that referenced this pull request Jan 30, 2021
@abalabahaha abalabahaha marked this pull request as draft March 25, 2021 00:52
@bsian03 bsian03 marked this pull request as ready for review January 4, 2024 17:41
@bsian03 bsian03 merged commit c3e1b73 into abalabahaha:dev Jan 11, 2024
flav-code added a commit to Flavibot/eris that referenced this pull request Jan 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants