Skip to content
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

Add custom Uncaught Exception Handler for FC #8061

Draft
wants to merge 121 commits into
base: tillh/workmanager-for-analytics
Choose a base branch
from

Conversation

tillh-stripe
Copy link
Collaborator

Summary

Motivation

Testing

  • Added tests
  • Modified tests
  • Manually verified

Screenshots

Before After
before screenshot after screenshot

Changelog

carlosmuvi-stripe and others added 30 commits November 10, 2023 11:22
* [skip ci] Start PR

* Adds new typefaces.

* Updates screenshots.

* Style fixes.

* Adds new colors.

* Generates screenshots.

* Updates colors.

* Adds background to previews.

* api dump.
* [skip ci] Start PR

* WIP

* WIP

* Updates loading.

* Removes resources.

* Adds screenshots.

* Reverts loading.

* Style fixes.

* Removes resource.
* [skip ci] Start PR

* Updates models.

* Updates UI.

* Adds legal details and data access styles.

* Updates layout to support modal mode.

* Updates list items.

* Updates screenshots.

* Ktlint.

* Detekt.

* Uses items indexed and last index.

* Removes row.

* Uses animated content to transition between image states.

* Adds debug resources.

* Ktlint fixes.

* Adds divider on content scroll and elevation in consent.

* Adds divider on content scroll and elevation in consent.

* Adds top shadow when content is scrollable.

* Documents Layout.

* Ktlint fixes.
* [skip ci] Start PR

* WIP

* Adds loading / disabled states.

* Removes search mode.

* Removes search mode.

* Extracts institutionIcon and uses plain lazy column.

* Updates shadows on institution icons.

* Updates api dump.

* Adds clear search button.

* Ktlint fixes.
* [skip ci] Start PR

* Updates screen.

* Adds preview.

* Adds loading shimmer screenshots.

* Updates screenshots.
* [skip ci] Start PR

* Updates exit style.

* Adds icon.

* Generates screenshots.

* Generates screenshots.

* Updates bottom padding.
* [skip ci] Start PR

* Updates manual entry content.

* Updates API.
* [skip ci] Start PR

* Updates account view.

* Updates merchant data access box.

* Updates loading and error screens.

* Updates screenshots.

* Adds haptic feedback.

* Updates balance badge.

* Regenerates screenshots.

* Pascal case for enum.

* ContentAlpha.

* Adds spacedBy.

* Improves preview.

* Uses content alpha.

* Detekt fixes.

* Reverts content alpha.

* Updates API.
* [skip ci] Start PR

* Updates signup pane.

* Adds legal details notice.

* Updates selected color and elevation.

* Updates screenshots.

* Ktlint fixes.
* [skip ci] Start PR

* Adds success pane.

* Improves animation.

* Adds animation constants.

* Updates texts.

* Adds preview parameter provider.

* Adds preview parameter provider.

* Updates booleans for readability.

* Removes api field.

* Apidump.

* (WIP) updates manual entry success.

* Uses success content in manual entry success.

* Ktlint fixes.

* Updates screenshots.

* Uses delegation and method reference.

* uses graphic layer.

* removes count.

* Adds comment.

* Uses last4 directly.

* Moves preview parameter to composable.

* Updates screenshots.

* Updates api.

* Adds link messaging.
* [skip ci] Start PR

* Uses institution icon from partner account response.

* Uses layout.

* Updates loading view

* Updates screenshots.

* Adds missing screenshots.

* Adds click handler.

* Use clickable handler in Consent.

* Unifies mainContent.

* Updates API.

* Uses shared modal bottom sheet.

* Updates screenshots.

* Just navigates on successful attach.
* Make tweaks to Link inline signup

* Address code review feedback

- Make names consistent
- Add test for case where prefill is desired
Co-authored-by: runner <runner@Mac-1706055070232.local>
…eral-style-updates

# Conflicts:
#	CHANGELOG.md
#	README.md
#	VERSION
#	docs/link/com.stripe.android.link/-link-configuration/-customer-info/-customer-info.html
#	docs/link/com.stripe.android.link/-link-configuration/-customer-info/index.html
#	docs/link/com.stripe.android.link/-link-configuration/-link-configuration.html
#	docs/link/com.stripe.android.link/-link-configuration/customer-info.html
#	docs/link/com.stripe.android.link/-link-configuration/index.html
#	docs/link/com.stripe.android.link/index.html
#	docs/payments-core/com.stripe.android/-payment-controller/register-launchers-with-activity-result-caller.html
#	docs/scripts/pages.json
#	gradle.properties
#	stripe-core/src/main/java/com/stripe/android/core/version/StripeSdkVersion.kt
tillh-stripe and others added 8 commits March 5, 2024 13:01
* Initial approach

* Revert "Initial approach"

This reverts commit e06d860.

* Use NavController-based approach

Changed `tryNavigateTo` from taking `popUpToCurrent: Boolean` to taking `popUpTo: PopUpToBehavior?)`. For situations where we want to pop two destinations (the bottom sheet and its referrer), we can now use `PopUpToBehavior.Route(String)`.

* Fix rebase issue

* Address code review feedback

Add comments for clarity
* Align exit modal confirm label with Web

* Update screenshots
* [skip ci] Start PR

* Add 8dp padding underneath the institution picker search bar.

* Removes comment.

* Regenerates screenshots

* Regenerates screenshots

* Regenerates screenshots

* Regenerates screenshots

* Extracts to method.
Copy link
Contributor

github-actions bot commented Mar 6, 2024

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 MiB │     2 MiB │  0 B │   4.1 MiB │   4.1 MiB │  0 B 
     arsc │   1.4 MiB │   1.4 MiB │  0 B │   1.4 MiB │   1.4 MiB │  0 B 
 manifest │   2.3 KiB │   2.3 KiB │  0 B │   8.1 KiB │   8.1 KiB │  0 B 
      res │ 363.2 KiB │ 363.2 KiB │  0 B │ 490.6 KiB │ 490.6 KiB │  0 B 
   native │   6.2 MiB │   6.2 MiB │  0 B │  15.8 MiB │  15.8 MiB │  0 B 
    asset │  67.8 KiB │  67.8 KiB │  0 B │  87.8 KiB │  87.8 KiB │  0 B 
    other │ 114.7 KiB │ 114.7 KiB │  0 B │ 270.3 KiB │ 270.3 KiB │  0 B 
──────────┼───────────┼───────────┼──────┼───────────┼───────────┼──────
    total │  10.2 MiB │  10.2 MiB │  0 B │  22.2 MiB │  22.2 MiB │  0 B 

 DEX     │ old   │ new   │ diff      
─────────┼───────┼───────┼───────────
   files │     1 │     1 │ 0         
 strings │ 20979 │ 20979 │ 0 (+0 -0) 
   types │  6538 │  6538 │ 0 (+0 -0) 
 classes │  5312 │  5312 │ 0 (+0 -0) 
 methods │ 30706 │ 30706 │ 0 (+0 -0) 
  fields │ 17862 │ 17862 │ 0 (+0 -0) 

 ARSC    │ old  │ new  │ diff 
─────────┼──────┼──────┼──────
 configs │  262 │  262 │  0   
 entries │ 5504 │ 5504 │  0

@tillh-stripe tillh-stripe force-pushed the tillh/workmanager-for-analytics branch from d62e094 to 86deb95 Compare March 7, 2024 15:25
@tillh-stripe tillh-stripe changed the title [WIP] FC Uncaught Exception Handler [WIP] Add Uncaught Exception Handler for FC Mar 7, 2024
@tillh-stripe tillh-stripe changed the title [WIP] Add Uncaught Exception Handler for FC [WIP] Add custom Uncaught Exception Handler for FC Mar 7, 2024
@tillh-stripe tillh-stripe changed the title [WIP] Add custom Uncaught Exception Handler for FC Add custom Uncaught Exception Handler for FC Mar 7, 2024
@tillh-stripe tillh-stripe force-pushed the tillh/workmanager-for-analytics branch 3 times, most recently from 8b2486b to fc90e2c Compare March 19, 2024 19:34
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.

None yet

4 participants