Skip to content

Conversation

@tiwarishubham635
Copy link
Contributor

Fixes

Use this branch to test against the new changes. A dummy api account.v1.service is added to test this flow. Follow these steps for testing:

  1. Start a local server to return error response as expected by new standards
  2. Run it on a public url using ngrok
  3. Copy the public url and place in src/Twilio/Rest/AccountsBase.php as marked
  4. Run the example/test_new_error_response.php to play around

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, or create a GitHub Issue in this repository.

@tiwarishubham635 tiwarishubham635 changed the title [DO NOT MERGE] PR to test new error response chore: [DO NOT MERGE] PR to test new error response Dec 11, 2025
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.

2 participants