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

feat(next): init and add sign in route handler #339

Merged
merged 1 commit into from
Jul 18, 2022

Conversation

wangsijie
Copy link
Contributor

@wangsijie wangsijie commented Jul 13, 2022

Summary

Implement "Sign In" handler, when visiting "/api/sign-in", the user will be redirect to generated sign in uri (to Logto).

check "next-sample" for usage, if you are not familar with NextJS.

Testing

UTs

@linear
Copy link

linear bot commented Jul 13, 2022

@wangsijie wangsijie force-pushed the sijie-log-3588-node-sdk branch 3 times, most recently from cfcfed2 to 81481ea Compare July 14, 2022 06:17
Base automatically changed from sijie-log-3588-node-sdk to master July 14, 2022 06:21
@wangsijie wangsijie force-pushed the sijie-log-3580-init-next branch 2 times, most recently from 43f811d to 792ae85 Compare July 14, 2022 07:46
@wangsijie wangsijie marked this pull request as ready for review July 14, 2022 07:46
@wangsijie wangsijie force-pushed the sijie-log-3580-init-next branch 3 times, most recently from f4460a8 to 87822f9 Compare July 15, 2022 06:04
@wangsijie wangsijie changed the title feat(next): init and sign in route feat(next): init and add sign in route handler Jul 15, 2022
packages/next-sample/.eslintrc Show resolved Hide resolved
packages/next/package.json Show resolved Hide resolved
packages/next/src/index.ts Show resolved Hide resolved
packages/next/tsconfig.json Outdated Show resolved Hide resolved
packages/next-sample/libraries/logto.ts Outdated Show resolved Hide resolved
packages/next-sample/libraries/logto.ts Outdated Show resolved Hide resolved
packages/next-sample/tsconfig.json Outdated Show resolved Hide resolved
packages/next/package.json Outdated Show resolved Hide resolved
packages/next/src/storage.ts Outdated Show resolved Hide resolved
packages/next/src/storage.ts Show resolved Hide resolved
Copy link
Member

@gao-sun gao-sun left a comment

Choose a reason for hiding this comment

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

good @demonzoo can double check if needed.

@wangsijie wangsijie merged commit f17364a into master Jul 18, 2022
@wangsijie wangsijie deleted the sijie-log-3580-init-next branch July 18, 2022 07:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants