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.

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

FR-15371 - Example app makeover #1118

Merged
merged 3 commits into from
Mar 3, 2024
Merged

FR-15371 - Example app makeover #1118

merged 3 commits into from
Mar 3, 2024

Conversation

doregg
Copy link
Contributor

@doregg doregg commented Feb 29, 2024 •
edited
Loading

Most of the files just copied form the admin box example app

Main changes compared to the admin box:

  • Full stepped up page replaced by the HOC page
  • React is 17, the main render method is like before

Screenshot 2024-02-29 at 14 29 02

yuvalotem1
yuvalotem1 reviewed Feb 29, 2024
View reviewed changes
packages/demo-saas/src/adminBoxPreview/AdminBoxPreviewContext.tsx Outdated Show resolved Hide resolved
packages/demo-saas/src/customizationOptions/authOptions.ts Outdated Show resolved Hide resolved
packages/demo-saas/src/customizationOptions/themeOptions/splitMode.ts Outdated Show resolved Hide resolved
packages/demo-saas/src/loginBoxPreview/LoginBoxIframe.tsx Outdated Show resolved Hide resolved
packages/demo-saas/src/loginBoxPreview/LoginBoxInstance.tsx Outdated Show resolved Hide resolved
packages/demo-saas/src/loginBoxPreview/LoginBoxPreview.tsx Outdated Show resolved Hide resolved
packages/demo-saas/src/loginBoxPreview/LoginBoxPreviewContext.tsx Outdated Show resolved Hide resolved
packages/demo-saas/src/loginBoxPreview/index.ts Outdated Show resolved Hide resolved
packages/demo-saas/src/stepUp/Wallet/MyBalance.tsx Outdated Show resolved Hide resolved
packages/demo-saas/src/stepUp/Wallet/QPWallet.tsx Outdated Show resolved Hide resolved
@doregg doregg merged commit 0b026f0 into master Mar 3, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cycode-security cycode-security[bot] cycode-security[bot] left review comments

@yuvalotem1 yuvalotem1 yuvalotem1 approved these changes

@aviadmizrachi aviadmizrachi Awaiting requested review from aviadmizrachi aviadmizrachi is a code owner

@frontegg-david frontegg-david Awaiting requested review from frontegg-david frontegg-david is a code owner

@rotemzif1 rotemzif1 Awaiting requested review from rotemzif1

@amirjaron amirjaron Awaiting requested review from amirjaron amirjaron is a code owner

@MaxArnautFrontegg MaxArnautFrontegg Awaiting requested review from MaxArnautFrontegg MaxArnautFrontegg is a code owner

@TomerFrontegg TomerFrontegg Awaiting requested review from TomerFrontegg TomerFrontegg is a code owner

@mariavlasov mariavlasov Awaiting requested review from mariavlasov mariavlasov is a code owner

@isra-frontegg isra-frontegg Awaiting requested review from isra-frontegg

@AtaliaRefua AtaliaRefua Awaiting requested review from AtaliaRefua

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@doregg @yuvalotem1