Skip to content

Conversation

@saenyakorn
Copy link
Member

Why did you create this PR

  • Login always return success even the password is wrong
  • The error code is scattered. So, I tried to use the same error code as possible while the remaining the meaning.

What did you do

  • Fix bugs
  • Simplify error code
  • Better error handling on throwable function
  • Change delete many method to use updateMany instead
  • Change the verification identity from refCode to token

Screenshots / Recordings

Checklist

  • Self-reviewed your code
  • Wrote coverage tests
  • Added screenshots or recordings if applicable

@changeset-bot
Copy link

changeset-bot bot commented Sep 8, 2025

🦋 Changeset detected

Latest commit: 43c81b0

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 8 packages
Name Type
@example/erp Patch
@genseki/plugins Patch
@genseki/react Patch
@genseki/next Patch
@genseki/rest Patch
@genseki/react-query Patch
@genseki/prisma-generator Patch
@example/ui-playground Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link
Contributor

@miello miello left a comment

Choose a reason for hiding this comment

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

LGTM 🥳

@saenyakorn saenyakorn merged commit 0d7d89e into main Sep 8, 2025
1 check passed
@saenyakorn saenyakorn deleted the yu/fix/phone-plugin branch September 8, 2025 10:24
@github-actions github-actions bot mentioned this pull request Sep 8, 2025
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