-
Notifications
You must be signed in to change notification settings - Fork 77
refactor: remove anonymous user feature #2436
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: remove anonymous user feature #2436
Conversation
|
Important Review skippedBot user detected. To trigger a single review, invoke the You can disable this status message by setting the Comment |
a7587d3 to
01d44e9
Compare
Codecov Report❌ Patch coverage is ❌ Your project status has failed because the head coverage (79.85%) is below the target coverage (80.00%). You can increase the head coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #2436 +/- ##
==========================================
- Coverage 51.34% 51.02% -0.32%
==========================================
Files 1069 1062 -7
Lines 29439 29188 -251
Branches 6466 6442 -24
==========================================
- Hits 15115 14894 -221
- Misses 13912 13939 +27
+ Partials 412 355 -57
*This pull request uses carry forward flags. Click here to find out more.
🚀 New features to boost your workflow:
|
01d44e9 to
1d30a80
Compare
|
@copilot there are some eslint errors in gha results, please fix them |
Fixed in 3a088ac. Removed unused imports: FeatureFlags from wallet controller and billing services, isNull from user repository, FeatureFlagsService from wallet-initializer constructor, and fixed import sorting in start-trial.spec.ts. |
5dac836 to
4d3d818
Compare
Co-authored-by: stalniy <1159966+stalniy@users.noreply.github.com>
…ests Co-authored-by: stalniy <1159966+stalniy@users.noreply.github.com>
Co-authored-by: stalniy <1159966+stalniy@users.noreply.github.com>
Co-authored-by: stalniy <1159966+stalniy@users.noreply.github.com>
Co-authored-by: stalniy <1159966+stalniy@users.noreply.github.com>
Co-authored-by: stalniy <1159966+stalniy@users.noreply.github.com>
f443218 to
8bd773c
Compare
Remove Anonymous Users Feature
Completed Changes
API Changes (apps/api)
create-anonymous-user.router.tsandget-anonymous-user.router.tsapps/api/src/user/routes/index.tsto remove anonymous user route exportsapps/api/src/rest-app.tsto remove anonymous user router registrationsapps/api/src/user/controllers/user/user.controller.tsto removecreate()andgetById()methodsapps/api/src/user/services/user/user.service.tsto remove anonymous user handling inregisterUser()apps/api/src/user/schemas/user.schema.tsto remove and re-add necessary schemasapps/api/src/user/repositories/user/user.repository.tsto removefindAnonymousById()methodapps/api/test/functional/anonymous-user.spec.tsapps/api/test/services/wallet-testing.service.tsto remove methodsANONYMOUS_FREE_TRIALfrom feature flagsREGULAR_ANONYMOUS_USERrole from ability serviceDeploy-Web Changes (apps/deploy-web)
apps/deploy-web/src/context/AnonymousUserProviderapps/deploy-web/src/queries/useAnonymousUserQueryapps/deploy-web/src/hooks/useStoredAnonymousUser.tsapps/deploy-web/src/hooks/useUser.tsto remove anonymous user referencesapps/deploy-web/src/components/user/UserProviders/UserProviders.tsxapps/deploy-web/src/config/auth.config.tsapps/deploy-web/src/services/auth/auth/interceptors.tsapps/deploy-web/src/pages/api/auth/signup.tsbuildAnonymousUserfunction from test seedersanonymous_free_trialHTTP SDK Changes (packages/http-sdk)
Summary
All anonymous user functionality has been successfully removed from the codebase. ESLint errors have been fixed by removing unused imports:
FeatureFlagsimport from wallet controller, managed-signer service, and wallet-initializer serviceisNullimport from user repositoryFeatureFlagsServicefrom wallet-initializer constructorOriginal prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.