Skip to content

Refresh duck.ai when subscription state changes #6395

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

Open
wants to merge 1 commit into
base: feature/cristian/subscriptions/show_duck_ai_pro_settings
Choose a base branch
from

Conversation

cmonfortep
Copy link
Contributor

@cmonfortep cmonfortep commented Jul 14, 2025

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

Description

Signals FE to refresh Duck.ai when subscription state changes

Steps to test this PR

Feature 1

  • apply patch attached into task
  • Fresh install
  • Visit https://euw-serp-dev-testing10.duck.co/ on a normal tab and log in
  • Now open duck.ai (you shouldn't be asked to login now)
  • Ensure you are in free mode
  • From there, find a subscription upsell (e.g: open models selector and pick an advanced model, or inside duck.ai settings -> purchase)
  • Finish the purchase
  • Go back to duck.ai, it should be "subscriber mode" now

UI changes

Before After
!(Upload before screenshot) (Upload after screenshot)

@cmonfortep
Copy link
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

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

@cmonfortep cmonfortep requested a review from lmac012 July 14, 2025 22:01
@cmonfortep cmonfortep force-pushed the feature/cristian/duckai/refresh_on_subscription_changes branch from f1f47aa to d19380a Compare July 14, 2025 22:05
@cmonfortep cmonfortep force-pushed the feature/cristian/subscriptions/show_duck_ai_pro_settings branch from 1b3df67 to 8af6f45 Compare July 15, 2025 10:38
@cmonfortep cmonfortep force-pushed the feature/cristian/duckai/refresh_on_subscription_changes branch from d19380a to f08c8bf Compare July 15, 2025 10:38
Copy link
Contributor

@lmac012 lmac012 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

subscriptions.getSubscriptionStatusFlow()
.distinctUntilChanged()
.onEach { _ ->
logcat { "CRIS: SUBSCRIPTION STATUS CHANGED" }
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like a debugging leftover.

subscriptions.getSubscriptionStatusFlow()
.distinctUntilChanged()
.onEach { _ ->
logcat { "CRIS: SUBSCRIPTION STATUS CHANGED" }
Copy link
Contributor

Choose a reason for hiding this comment

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

Looks like a debugging leftover.

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