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

Support inline preview in auth flow v2 #4359

Merged
merged 31 commits into from
Jun 24, 2024

Conversation

cychiuae
Copy link
Contributor

No description provided.

@cychiuae
Copy link
Contributor Author

@louischan-oursky @tung2744

You can have a look at this PR if you have time. Please give feedback on what do you think.

@louischan-oursky louischan-oursky self-requested a review June 19, 2024 04:09
pkg/auth/handler/webapp/authflowv2/login.go Outdated Show resolved Hide resolved
pkg/auth/handler/webapp/authflowv2/login.go Outdated Show resolved Hide resolved
pkg/lib/config/identity.go Outdated Show resolved Hide resolved
@cychiuae cychiuae force-pushed the dev-1424-support-authui-preview-mode branch from 8f2e377 to d99f721 Compare June 19, 2024 06:08
@cychiuae cychiuae force-pushed the dev-1424-support-authui-preview-mode branch from f9f7652 to e506ece Compare June 19, 2024 14:41
@cychiuae cychiuae changed the title wip: Support inline preview in auth flow v2 Support inline preview in auth flow v2 Jun 20, 2024
@cychiuae
Copy link
Contributor Author

@tung2744 Can review now

@tung2744 tung2744 self-assigned this Jun 20, 2024
@cychiuae cychiuae force-pushed the dev-1424-support-authui-preview-mode branch from 5b8e399 to b1c9177 Compare June 21, 2024 04:11
@louischan-oursky louischan-oursky self-requested a review June 21, 2024 11:35
@louischan-oursky louischan-oursky self-requested a review June 24, 2024 03:47
@louischan-oursky
Copy link
Collaborator

@cychiuae
Copy link
Contributor Author

@louischan-oursky @tung2744 Updated. THanks!

@tung2744 tung2744 merged commit 9e660b1 into authgear:main Jun 24, 2024
7 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