Skip to content

feat(adapter): Use createAdapter utility #9

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

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

octet-stream
Copy link
Owner

@octet-stream octet-stream commented Apr 21, 2025

Details

This PR adds use of Better Auth utilities for database adapters following the documentation: https://www.better-auth.com/docs/guides/create-a-db-adapter.

Changes

  • Use createAdapter utility for adapter definition;
  • Add config argument for adapter to support additional parameters. Though, most of the parameters will be inherited from Mikro ORM instance passed as the first argument, so at this point the only supported option is debugLogs.
  • Use schema utilities as needed (I will add more details on this one as I progress);
  • Run adapter against the tests from better-auth;

Checklist

  • I have self-reviewed my changes before asking for a review from maintainers
  • I have added changesets
  • I have brought tests
  • I have updated the documentation

@octet-stream octet-stream added the enhancement New feature or request label Apr 21, 2025
@octet-stream octet-stream self-assigned this Apr 21, 2025
Copy link

changeset-bot bot commented Apr 21, 2025

⚠️ No Changeset found

Latest commit: 9ea7ace

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant