Changed domain services URL.#3453
Conversation
🧪 Unit Test Results📊 Summary
Last updated: Wed, 17 Dec 2025 13:24:50 GMT |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
Review Summary
This PR updates API base URLs for Pine, Cedar, and Redwood services, migrating from .nonprod.inseng.io and .temp.prod.inseng.io domains to the new .core.inseng.io domain pattern.
Positive Observations
✅ The URL updates are straightforward and focused
✅ Both debug and release configurations are updated consistently for Pine, Cedar, and Redwood
✅ The changes follow a clear migration pattern to the new domain structure
Issues to Address
-
Naming Inconsistency (libs/canvas-api-2/build.gradle:26): Production URLs now drop the
-productionsuffix while debug URLs retain-dev. This creates an asymmetric naming pattern that could be confusing. Consider standardizing the approach across environments. -
Journey URLs Not Updated (libs/canvas-api-2/build.gradle:34): The Journey service URLs still use the old domain patterns (
.nonprod.inseng.ioand.temp.prod.inseng.io). If this is intentional due to different migration timeline or hosting, please clarify in the PR description.
Recommendations
- Verify that the new URLs are accessible and properly configured in the respective environments
- Consider adding a test deployment to ensure connectivity with the new endpoints
- If Journey URLs are intentionally not migrated yet, document the reasoning
Testing Considerations
- Ensure debug builds can connect to the new dev endpoints
- Verify release builds work with the new production endpoints
- Test authentication flows with the updated base URLs
tamaskozmer
merged commit d5f7555
into
release/student-8.4.2-285
2 participants
No description provided.