Skip to content

Address virtual currencies with zero units in customer center #5500

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Aug 26, 2025
Merged

Address virtual currencies with zero units in customer center #5500

merged 3 commits into from
Aug 26, 2025

Conversation

facumenzella
Copy link
Contributor

Motivation

We've noticed that there's an issue with virtual currencies and the empty state of customer center

Description

  • Consider the balance of VCs instead of the presence of a currency
  • Fix ui glitch for empty state

@facumenzella facumenzella requested a review from vegaro August 26, 2025 09:32
@facumenzella facumenzella changed the title Address virtual currencies with zero units Address virtual currencies with zero units in customer center Aug 26, 2025
ekurutepe
ekurutepe approved these changes Aug 26, 2025
View reviewed changes
Copy link
Contributor

@ekurutepe ekurutepe left a comment

Choose a reason for hiding this comment

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

lgtm

@emerge-tools Emerge Tools
Copy link

emerge-tools bot commented Aug 26, 2025 •
edited
Loading

📸 Snapshot Test

27 modified, 666 unchanged

Name Added Removed Modified Renamed Unchanged Errored Approval
RevenueCat
com.revenuecat.PaywallsTester		 0 0 9 0 222 0 ⏳ Needs approval
RevenueCat
com.revenuecat.PaywallsTester.mac-catalyst-optimized-for-mac		 0 0 9 0 222 0 ⏳ Needs approval
RevenueCat
com.revenuecat.PaywallsTester.mac-catalyst-scaled-to-match-ipad		 0 0 9 0 222 0 ⏳ Needs approval

🛸 Powered by Emerge Tools

@facumenzella facumenzella merged commit 37d8f4d into main Aug 26, 2025
12 checks passed
@facumenzella facumenzella deleted the fix/virtual-currencies-empty branch August 26, 2025 13:53
@facumenzella facumenzella mentioned this pull request Aug 27, 2025
Merged
This was referenced Aug 27, 2025
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ekurutepe ekurutepe ekurutepe approved these changes

@vegaro vegaro vegaro approved these changes

@fire-at-will fire-at-will Awaiting requested review from fire-at-will

Assignees
No one assigned
Labels
feat:Customer Center pr:fix A bug fix pr:RevenueCatUI
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@facumenzella @ekurutepe @vegaro