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

removed the default validate status instances #595

Merged

ashleysmithTTD merged 1 commit into main from ans-UID2-2664-remove-validate-status-axios-config

Feb 18, 2025

Contributor

ashleysmithTTD commented Feb 18, 2025 •

edited

Loading

What Changed:

Removed the two instances of specifying validateStatus that match the default, so they are not necessary ( >= 200 && <= 300)

Test Plan:

This only happens in two functions related to sharing. Test that sharing still works fine in the portal


          removed the default validate status instances

e910287

ssundahlTTD approved these changes

View reviewed changes

ashleysmithTTD merged commit 42dbf2f into main

3 checks passed

ashleysmithTTD deleted the ans-UID2-2664-remove-validate-status-axios-config branch

February 18, 2025 19:37

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

Labels

None yet