Skip to content

Remove v1/offer-status request#8218

Merged
nalcalag merged 1 commit intodevelopfrom
feature/noelia/remove_offer_status_request
Apr 9, 2026
Merged

Remove v1/offer-status request#8218
nalcalag merged 1 commit intodevelopfrom
feature/noelia/remove_offer_status_request

Conversation

@nalcalag
Copy link
Copy Markdown
Contributor

@nalcalag nalcalag commented Apr 8, 2026

Task/Issue URL: https://app.asana.com/1/137249556945/project/1209991789468715/task/1213982969677093?focus=true

Description

Remove backend check for hadTrial since we limited usage on Play Console

Steps to test this PR (subscriptions smoke test)

Pre steps

Free trial eligible

  • Go to Play Billing Lab and enable introductory offer in Configuration Settings
  • Fresh install from branch
  • Once in the browser, open the menu in the new tab page and check you se VPN menu item with 'Try for Free' yellow pill

Subscription expired

  • Purchase a test subscription
  • Remove it from device
  • Add it back with 'I have a subscription flow'
  • Wait until it expires and check expired status looks correct in Settings screen and Subscription Settings screen

VPN menu item

  • Remove Free trial eligibility in Play Billing Lab
  • Open a new tap page
  • Check VPN menu doesnt have yellow pill
  • Go to Subscription Settings (still in expired status)
  • Tab on View plans
  • Verify no free trial is offered
  • Purchase a new plan with no free trial

No UI changes


Note

Low Risk
Low risk: removes a redundant backend call and simplifies free-trial eligibility gating; behavior changes only in cases where the backend previously reported hadTrial but Play Console still exposes an intro offer.

Overview
Removes the v1/offer-status API from SubscriptionsService and deletes the associated OfferStatusResponse model.

RealSubscriptionsManager.isFreeTrialEligible no longer queries the backend for hadTrial; free-trial eligibility is now determined solely by the privacyProFreeTrial feature flag and whether a matching free-trial offer is available from Google Play.

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

Copy link
Copy Markdown
Contributor Author

nalcalag commented Apr 8, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@nalcalag nalcalag marked this pull request as ready for review April 8, 2026 15:49
Copy link
Copy Markdown
Contributor

@cmonfortep cmonfortep left a comment

Choose a reason for hiding this comment

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

LGTM

@nalcalag nalcalag merged commit 51f8d77 into develop Apr 9, 2026
40 of 42 checks passed
@nalcalag nalcalag deleted the feature/noelia/remove_offer_status_request branch April 9, 2026 06:31
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