Skip to content

WFL-303 | Add fallback URL for workflows#3608

Merged
RosieWatson merged 3 commits into
mainfrom
rosie/wfl-330-extend-workflows-fallback-to-8-lives-cloudflare-for-full-2
Jun 16, 2026
Merged

WFL-303 | Add fallback URL for workflows#3608
RosieWatson merged 3 commits into
mainfrom
rosie/wfl-330-extend-workflows-fallback-to-8-lives-cloudflare-for-full-2

Conversation

@RosieWatson

@RosieWatson RosieWatson commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

Motivation

Add Fortress fallback URLs for Workflow endpoints

iOS counterpart: RevenueCat/purchases-ios#7008


Note

Low Risk
Localized networking path logic with tests; failover only affects workflow API calls when fallback base URLs are configured.

Overview
Adds Fortress fallback path support for GetWorkflow and GetWorkflows, matching the existing pattern used by offerings.

When getPath(useFallback = true) is used (e.g. on HTTP client retry to alternate base URLs), single-workflow requests go to /workflows/v1/workflows/{workflowId} instead of the subscriber-scoped path, and list requests use /workflows/v1/workflows, still appending ?type= when a type filter is set. Primary paths are unchanged.

Unit tests cover fallback paths with and without the type query parameter.

Reviewed by Cursor Bugbot for commit 0a55033. Bugbot is set up for automated code reviews on this repo. Configure here.

@RosieWatson RosieWatson requested a review from a team as a code owner June 15, 2026 16:36

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 4e9445e. Configure here.

Comment thread purchases/src/main/kotlin/com/revenuecat/purchases/common/networking/Endpoint.kt Outdated
@facumenzella

Copy link
Copy Markdown
Member

Can we link the iOS PR in the pr description :) ?

@codecov

codecov Bot commented Jun 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 75.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.25%. Comparing base (39862fb) to head (0a55033).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...revenuecat/purchases/common/networking/Endpoint.kt 75.00% 0 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3608      +/-   ##
==========================================
- Coverage   80.26%   80.25%   -0.01%     
==========================================
  Files         378      378              
  Lines       15449    15455       +6     
  Branches     2144     2146       +2     
==========================================
+ Hits        12400    12404       +4     
  Misses       2190     2190              
- Partials      859      861       +2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@RosieWatson RosieWatson added this pull request to the merge queue Jun 16, 2026
Merged via the queue into main with commit b80506f Jun 16, 2026
38 checks passed
@RosieWatson RosieWatson deleted the rosie/wfl-330-extend-workflows-fallback-to-8-lives-cloudflare-for-full-2 branch June 16, 2026 07:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants