Skip to content

Conversation

carlosmuvi-stripe
Copy link
Collaborator

@carlosmuvi-stripe carlosmuvi-stripe commented Aug 6, 2025

Summary

  • User signs up to Link and adds a PM
  • Then launches Link again
  • Before: we’d show add payment method again, because consumer just signed up.
  • Now: we show the wallet if the user selected a PM previously.
record_bug.webm

Motivation

Testing

  • Added tests
  • Modified tests
  • Manually verified

Screenshots

Before After
before screenshot after screenshot

Changelog

Copy link
Contributor

github-actions bot commented Aug 6, 2025

Diffuse output:

OLD: identity-example-release-base.apk (signature: V1, V2)
NEW: identity-example-release-pr.apk (signature: V1, V2)

          │          compressed           │         uncompressed         
          ├───────────┬───────────┬───────┼───────────┬───────────┬──────
 APK      │ old       │ new       │ diff  │ old       │ new       │ diff 
──────────┼───────────┼───────────┼───────┼───────────┼───────────┼──────
      dex │   2.1 MiB │   2.1 MiB │   0 B │   4.3 MiB │   4.3 MiB │  0 B 
     arsc │     1 MiB │     1 MiB │   0 B │     1 MiB │     1 MiB │  0 B 
 manifest │   2.3 KiB │   2.3 KiB │   0 B │     8 KiB │     8 KiB │  0 B 
      res │ 302.9 KiB │ 302.9 KiB │   0 B │   457 KiB │   457 KiB │  0 B 
   native │   7.9 MiB │   7.9 MiB │   0 B │  19.3 MiB │  19.3 MiB │  0 B 
    asset │   7.7 KiB │   7.7 KiB │   0 B │   7.4 KiB │   7.4 KiB │  0 B 
    other │  95.7 KiB │  95.7 KiB │ -18 B │ 183.5 KiB │ 183.5 KiB │  0 B 
──────────┼───────────┼───────────┼───────┼───────────┼───────────┼──────
    total │  11.4 MiB │  11.4 MiB │ -18 B │  25.3 MiB │  25.3 MiB │  0 B 

 DEX     │ old   │ new   │ diff      
─────────┼───────┼───────┼───────────
   files │     1 │     1 │ 0         
 strings │ 20684 │ 20684 │ 0 (+0 -0) 
   types │  6503 │  6503 │ 0 (+0 -0) 
 classes │  5267 │  5267 │ 0 (+0 -0) 
 methods │ 31502 │ 31502 │ 0 (+0 -0) 
  fields │ 18232 │ 18232 │ 0 (+0 -0) 

 ARSC    │ old  │ new  │ diff 
─────────┼──────┼──────┼──────
 configs │  164 │  164 │  0   
 entries │ 3648 │ 3648 │  0
APK
    compressed    │   uncompressed   │                        
──────────┬───────┼───────────┬──────┤                        
 size     │ diff  │ size      │ diff │ path                   
──────────┼───────┼───────────┼──────┼────────────────────────
 29.2 KiB │ -14 B │  64.6 KiB │  0 B │ ∆ META-INF/CERT.SF     
 25.9 KiB │  -3 B │  64.5 KiB │  0 B │ ∆ META-INF/MANIFEST.MF 
  1.2 KiB │  -1 B │   1.2 KiB │  0 B │ ∆ META-INF/CERT.RSA    
──────────┼───────┼───────────┼──────┼────────────────────────
 56.2 KiB │ -18 B │ 130.4 KiB │  0 B │ (total)

@carlosmuvi-stripe carlosmuvi-stripe marked this pull request as ready for review August 7, 2025 00:15
@carlosmuvi-stripe carlosmuvi-stripe requested review from a team as code owners August 7, 2025 00:15
@carlosmuvi-stripe carlosmuvi-stripe merged commit a3fbae5 into master Aug 7, 2025
13 checks passed
@carlosmuvi-stripe carlosmuvi-stripe deleted the carlosmuvi/do-not-start-with-new-pm-after-relaunching branch August 7, 2025 00:44
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