Skip to content

Conversation

@ieow
Copy link

@ieow ieow commented May 14, 2025

Description

1.3
Update New SRP onboarding flow as per figma design
https://www.figma.com/design/pViOUcmjwhEzFsdrwknpNc/Onboarding-redesign?node-id=434-55200&t=wmEoQSF9g9lFtkwp-0

This PR is part 3 of 7 for the new SRP ui update

This PR cover

  • new ui for welcome page

Related issues

Fixes:

Manual testing steps

  1. Go to this page...

Screenshots/Recordings

Before

After

Screen.Recording.2025-05-21.at.8.49.18.AM.mov

Pre-merge author checklist

Pre-merge reviewer checklist

  • I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
  • I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

@github-actions
Copy link

github-actions bot commented May 14, 2025

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

@ieow ieow mentioned this pull request May 14, 2025
7 tasks
@smgv
Copy link

smgv commented May 14, 2025

I have read the CLA Document and I hereby sign the CLA

@ieow ieow marked this pull request as ready for review May 21, 2025 14:17
StyleSheet.create({
scroll: {
flex: 1,
// marginTop: 100,

Choose a reason for hiding this comment

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

Can we remove unneeded comment?

Copy link
Author

Choose a reason for hiding this comment

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

removed!

@tommasini
Copy link

Can we have screenshots at the PR description? It would be awesome!

@ieow
Copy link
Author

ieow commented May 22, 2025

added screenshot !

ieow and others added 2 commits May 23, 2025 13:36
@ieow ieow force-pushed the feat/srp-onboarding-carousel-tnc branch from 2afe811 to 2ace86b Compare May 23, 2025 05:45
@ieow ieow force-pushed the feat/new-srp-welcom branch from 2a3adb3 to fc8632c Compare May 23, 2025 06:51
@ieow ieow force-pushed the feat/new-srp-welcom branch from fc8632c to 756db9a Compare May 23, 2025 08:59
@ieow ieow force-pushed the feat/srp-onboarding-carousel-tnc branch from e111fcf to ceab2f4 Compare May 23, 2025 09:19
const innerStyles = StyleSheet.create({
headerStyle: {
backgroundColor: themeColors.background.default,
backgroundColor: color || themeColors.background.default,

Choose a reason for hiding this comment

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

Would rename color to backgroundColor be more accurate?

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