Skip to content

Allow overriding iTwin reality data URL for QA or Dev env#1372

Merged
azrogers merged 4 commits into
CesiumGS:mainfrom
ludovic-cheval-bentley-systems:override-itwin-realitydata-url
Jun 2, 2026
Merged

Allow overriding iTwin reality data URL for QA or Dev env#1372
azrogers merged 4 commits into
CesiumGS:mainfrom
ludovic-cheval-bentley-systems:override-itwin-realitydata-url

Conversation

@ludovic-cheval-bentley-systems
Copy link
Copy Markdown
Contributor

@ludovic-cheval-bentley-systems ludovic-cheval-bentley-systems commented May 19, 2026

Description

Adds support for overriding the iTwin base URL when accessing Reality Data, aligning behavior with the existing override mechanism used for iTwin curated content.

This change improves flexibility for environments that require custom endpoints (e.g., DEV, QA, ...), ensuring consistent configuration capabilities across both curated content and Reality Data access.

For backward compatibility, when no overridden URL is given, behavior remains unchanged by defaulting to the existing Reality Data base URL for production.

Issue number or link

Author checklist

  • I have submitted a Contributor License Agreement (only needed once).
  • I have done a full self-review of my code.
  • I have updated CHANGES.md with a short summary of my change (for user-facing changes).
  • I have added or updated unit tests to ensure consistent code coverage as necessary.
  • I have updated the documentation as necessary.

Reviewer checklist

Thank you for taking the time to review this PR. By approving a PR you are taking as much responsibility for these changes as the author.

As you review, please go through the checklist below:

  • Review and run all parts of the test plan on this branch and verify it matches expectations.
    • If the issue is a bug please make sure you can reproduce the bug in the main branch and then checkout this branch to make sure it actually solved the issue.
  • Review the code and make sure you do not have any remaining questions or concerns. You should understand the code change and the chosen approach. If you are not confident or have doubts about the code, please do not hesitate to ask questions.
  • Review the unit tests and make sure there are no missing tests or edge cases.
  • Review documentation changes and updates to CHANGES.md to make sure they accurately cover the work in this PR.
  • Verify that the Contributor License Agreement has been submitted, if needed.

@ludovic-cheval-bentley-systems ludovic-cheval-bentley-systems marked this pull request as ready for review May 20, 2026 20:49
@j9liu j9liu added this to the July 2026 Release milestone Jun 1, 2026
@j9liu j9liu requested a review from azrogers June 1, 2026 17:41
@azrogers azrogers merged commit 7a52c97 into CesiumGS:main Jun 2, 2026
14 checks passed
@azrogers
Copy link
Copy Markdown
Contributor

azrogers commented Jun 2, 2026

Thanks @ludovic-cheval-bentley-systems!

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.

3 participants