Skip to content

Conversation

github-actions[bot]
Copy link
Contributor

If this change should result in new package versions please add a changeset before merging. You can do so by clicking the link provided by changeset bot below.

## Reason

In order to enforce name matching in the MX onboarding flow, we need to add a pending page and different landing pages depending on the name matching result. This PR adds the pending page when the GK is enabled.

Design:
https://www.figma.com/design/hb8mqsg0n3QKJHpJA0ahkI/MXN-Grid-Switch?node-id=17-15925&m=dev

Screenshots below:

![Screenshot 2025-08-21 at 2.24.16 PM.png](https://app.graphite.dev/user-attachments/assets/555e2954-9675-4222-a33c-d91bd164cf3e.png)

![Screenshot 2025-08-21 at 2.24.32 PM.png](https://app.graphite.dev/user-attachments/assets/070c7fbb-4f54-4c13-a3da-dbb51d0077d2.png)

GitOrigin-RevId: 16f902a477add267300b15f73f2420bbadccbd53
@github-actions github-actions bot requested a review from a team as a code owner August 21, 2025 22:31
Copy link
Contributor Author

github-actions bot commented Aug 21, 2025

The following public packages have changed files:

Changed Current version
@lightsparkdev/lightspark-sdk 1.9.8
@lightsparkdev/core 1.4.3
@lightsparkdev/ui 1.1.9

There are 3 existing changesets for this branch. If additional changes have been made that require different versions for any package simply add another changeset. Any packages that depend on the planned releases will be updated and released automatically in a separate PR.

Each changeset corresponds to an update in the CHANGELOG for the packages listed in the changeset. Therefore, you should add a changeset for each noteable package change that this PR contains. For example, if a PR adds two features - one feature for packages A and B and one feature for package C - you should add two changesets. One changeset for packages A and B and one changeset for package C, with a description of each feature. The feature description will end up being the CHANGELOG entry for the packages in the changeset.

The following releases are planned based on the existing changesets:

Planned releases Update
@lightsparkdev/lightspark-sdk patch
@lightsparkdev/ui patch
@lightsparkdev/core patch
@lightsparkdev/lightspark-cli patch
@lightsparkdev/crypto-wasm patch
@lightsparkdev/oauth patch

Last updated by commit 11e36a5

lightspark-ci-js-sdk bot and others added 9 commits August 21, 2025 22:57
Update public `js` sources with the latest code from the [public
repository](https://github.com/lightsparkdev/js-sdk) main branch.

This typically happens when new versions of the SDK are released and
version updates need to be synced. The PR should be merged as soon as
possible to avoid updates to webdev overwriting the changes in the
js-sdk develop branch.

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Lightspark Eng <engineering@lightspark.com>
Co-authored-by: Peng Ying <peng@lightspark.com>
GitOrigin-RevId: e30a45111187be96d3583658bd02bb271521e53f
## Reason

Add "fun facts" while users wait for their kyc to complete

## Overview

This re-uses the same fun facts component from LinkBankMX

## Test Plan

KYC a user - while their kyc is in pending state, we should see the following screen, complete with fun facts

![Screenshot 2025-08-24 at 6.21.54 PM.png](https://app.graphite.dev/user-attachments/assets/cf729f5e-7ec8-4fb8-ae33-de2e10c63231.png)

GitOrigin-RevId: fe6335c827bd3110d2915b288da66b6de04d3e11
![Screenshot 2025-08-18 at 5.25.07 PM.png](https://app.graphite.dev/user-attachments/assets/0fb72900-d9b6-4010-9274-9cfafa4245c0.png)

GitOrigin-RevId: 6388684ba3bcce5fad01df8fbbb3621f23691c04
## Reason

Typically we have this circular-deps check for JS packages but it was
never added to this one. I realized it's causing a downstream issue so
this adds the check and resolves circular deps.

GitOrigin-RevId: 1ea0bf0d6f3c7f48a38527f971cb9c32c90b6a7a
## Reason

Typically we have this circular-deps check for JS packages but it was
never added to this one. I realized it's causing a downstream issue so
this adds the check and resolves circular deps.

GitOrigin-RevId: cc9081a0419495600c89ebec1d9e4c8ff6b5c9b3
## Reason

It's not exactly clear when this became an issue for spark-sdk on React
Native because originally it was working fine with a dynamic import.
Somewhere along the way, possibly due to an updated dependency in
spark/js lock file, it started causing the following error:

<img width="479" height="377" alt="Screenshot 2025-08-21 at 11 27 57 PM"
src="https://github.com/user-attachments/assets/206d254b-b721-4c84-b93c-559f3039a042"
/>

We need to avoid referencing ws in React Native entrypoints which
resolves the issue.

GitOrigin-RevId: 8e9297ac7c1d096d90fd0f46096187cc92256c2b
@coreymartin coreymartin changed the title [penny test] Add pending state for name match ## Reason In order to enforce na Updates to core, ui, lightspark-sdkl Aug 26, 2025
@coreymartin coreymartin changed the title Updates to core, ui, lightspark-sdkl Updates to core, ui, lightspark-sdk Aug 26, 2025
@coreymartin coreymartin enabled auto-merge (squash) August 26, 2025 21:58
@coreymartin coreymartin merged commit 031eb1b into main Aug 26, 2025
8 checks passed
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.

4 participants