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

Add Central Sandbox URLs for NA and EU #76

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

justinappler
Copy link

@justinappler justinappler commented Dec 10, 2024

Description of change

The current Zuora integration supports API sandboxes but not the Central Sandboxes. This change updates the set of possible URLs to include all URLs from this documentation.

Manual QA steps

  • Verify integrating with a central sandbox works as well

Risks

  • No obvious risks here as all Zuora environments use the same API scheme and the code is designed to handle failures of the URLs and iteration through them gracefully.

Rollback steps

  • revert this branch

AI generated code

https://internal.qlik.dev/general/ways-of-working/code-reviews/#guidelines-for-ai-generated-code

  • this PR has been written with the help of GitHub Copilot or another generative AI tool

@singer-bot
Copy link

Hi @justinappler, thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes.

@singer-bot
Copy link

You did it @justinappler!

Thank you for signing the Singer Contribution License Agreement.

@justinappler
Copy link
Author

@eskaaren @dsprayberry Do changes/improvements to this repo get folded into the Stitch integrations at some cadence? Was trying with this PR to address a breakage in our integration as a result of us moving to a Zuora Central Sandbox which requires the additional URLs added in this PR, and I'm wondering if this is the place to do it?

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