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

fix: Support 204 response in axios #1428

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

lopenchi
Copy link

@lopenchi lopenchi commented Feb 18, 2025

Fixes

Related PR: #1327

fix: Support 204 response in axios

Checklist

  • I acknowledge that all my contributions will be made under the project's license
  • I have made a material change to the repo (functionality, testing, spelling, grammar)
  • I have read the Contribution Guidelines and my PR follows them
  • I have titled the PR appropriately
  • I have updated my branch with the main branch
  • I have added tests that prove my fix is effective or that my feature works
  • I have added the necessary documentation about the functionality in the appropriate .md file
  • I have added inline documentation to the code I modified

If you have questions, please file a support ticket.

Sorry, something went wrong.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@lopenchi lopenchi changed the title Support 204 response in axios fix: Support 204 response in axios Feb 18, 2025

Verified

This commit was signed with the committer’s verified signature.
gustavocbritto Gustavo Britto
lopenchi and others added 2 commits February 18, 2025 11:42

Verified

This commit was signed with the committer’s verified signature.
gustavocbritto Gustavo Britto

Verified

This commit was signed with the committer’s verified signature.
gustavocbritto Gustavo Britto
@lopenchi lopenchi closed this Mar 21, 2025
@lopenchi lopenchi reopened this Mar 21, 2025

Verified

This commit was signed with the committer’s verified signature.
gustavocbritto Gustavo Britto

Verified

This commit was signed with the committer’s verified signature.
gustavocbritto Gustavo Britto
…nto DOCSPLAT-1860
@@ -152,7 +152,7 @@ describe('test_access_settings_whitelist_delete', () => {
};
request.method = 'DELETE';
request.url = '/v3/access_settings/whitelist';
xit('should have the correct response code', () => {
it('should have the correct response code', () => {
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In all cases, we should probably also add tests for expected failure state since we are now adding that in (anything that's not 200-299), rather than only the success state?

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

Successfully merging this pull request may close these issues.

None yet

3 participants