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

refactor(core): remove session api #2878

Merged
merged 3 commits into from
Jan 10, 2023

Conversation

simeng-li
Copy link
Contributor

Summary

remove session APIs

Testing

test locally

@simeng-li simeng-li requested a review from wangsijie as a code owner January 9, 2023 11:15
@linear
Copy link

linear bot commented Jan 9, 2023

@simeng-li simeng-li requested review from a team and charIeszhao and removed request for a team January 9, 2023 11:15
@github-actions
Copy link

github-actions bot commented Jan 9, 2023

COMPARE TO master

Total Size Diff 📉 -157.84 KB

Diff by File
Name Diff
.github/CODEOWNERS 📉 -56 Bytes
packages/core/src/middleware/koa-audit-log-legacy.test.ts 📉 -4.63 KB
packages/core/src/middleware/koa-audit-log-legacy.ts 📉 -2.94 KB
packages/core/src/middleware/koa-log-session-legacy.test.ts 📉 -2.37 KB
packages/core/src/middleware/koa-log-session-legacy.ts 📉 -814 Bytes
packages/core/src/routes/init.ts 📉 -506 Bytes
packages/core/src/routes/session/continue.test.ts 📉 -7.69 KB
packages/core/src/routes/session/continue.ts 📉 -5.35 KB
packages/core/src/routes/session/forgot-password.test.ts 📉 -8.16 KB
packages/core/src/routes/session/forgot-password.ts 📉 -2.03 KB
packages/core/src/routes/session/index.test.ts 📉 -7.24 KB
packages/core/src/routes/session/index.ts 📉 -3.67 KB
packages/core/src/routes/session/middleware/koa-guard-session-action.ts 📉 -1.7 KB
packages/core/src/routes/session/middleware/passwordless-action.ts 📉 -6.75 KB
packages/core/src/routes/session/password.test.ts 📉 -9.92 KB
packages/core/src/routes/session/password.ts 📉 -5.33 KB
packages/core/src/routes/session/passwordless.test.ts 📉 -31.36 KB
packages/core/src/routes/session/passwordless.ts 📉 -5.44 KB
packages/core/src/routes/session/social.bind-social.test.ts 📉 -6.13 KB
packages/core/src/routes/session/social.test.ts 📉 -13.79 KB
packages/core/src/routes/session/social.ts 📉 -8.93 KB
packages/core/src/routes/session/types.ts 📉 -2.65 KB
packages/core/src/routes/session/utils.test.ts 📉 -11.86 KB
packages/core/src/routes/session/utils.ts 📉 -8.11 KB
packages/core/src/routes/types.ts 📉 -259 Bytes
packages/ui/src/apis/social.ts 📉 -212 Bytes

@gao-sun
Copy link
Member

gao-sun commented Jan 9, 2023

CI failed

@simeng-li simeng-li force-pushed the simeng-log-5027-unload-the-session-apis-from-router branch from 6e9d065 to 8984bda Compare January 10, 2023 02:56
@simeng-li simeng-li force-pushed the simeng-log-5026-clean-up-session-api-integration-tests branch from 5b703f3 to de8582b Compare January 10, 2023 04:01
@simeng-li simeng-li force-pushed the simeng-log-5027-unload-the-session-apis-from-router branch from 8984bda to 20b61e7 Compare January 10, 2023 04:03
Base automatically changed from simeng-log-5026-clean-up-session-api-integration-tests to master January 10, 2023 04:18
@simeng-li simeng-li force-pushed the simeng-log-5027-unload-the-session-apis-from-router branch from 20b61e7 to 8ff5fcb Compare January 10, 2023 04:20
@simeng-li simeng-li force-pushed the simeng-log-5027-unload-the-session-apis-from-router branch from d7ddef7 to 9d5fdda Compare January 10, 2023 05:42
@simeng-li simeng-li enabled auto-merge (squash) January 10, 2023 05:42
@simeng-li simeng-li merged commit 39f15ac into master Jan 10, 2023
@simeng-li simeng-li deleted the simeng-log-5027-unload-the-session-apis-from-router branch January 10, 2023 05:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants