[PR #476] [MERGED] fix: change dontRemeberMe to remeberMe #20218

Closed
opened 2026-04-15 19:33:04 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/better-auth/better-auth/pull/476
Author: @Bekacru
Created: 11/9/2024
Status: Merged
Merged: 11/10/2024
Merged by: @Bekacru

Base: mainHead: fix/remember-me


📝 Commits (3)

📊 Changes

11 files changed (+20 additions, -19 deletions)

View changed files

📝 demo/nextjs/app/dashboard/organization-card.tsx (+2 -2)
📝 demo/nextjs/components/sign-in.tsx (+1 -1)
📝 docs/content/docs/authentication/email-password.mdx (+1 -1)
📝 examples/astro-example/src/components/sign-in.tsx (+1 -1)
📝 examples/expo-example/package.json (+1 -1)
📝 examples/nextjs-example/components/sign-in.tsx (+2 -2)
📝 examples/nuxt-example/package.json (+2 -2)
📝 examples/remix-example/app/components/sign-in-card.tsx (+1 -1)
📝 packages/better-auth/src/api/routes/session-api.test.ts (+1 -1)
📝 packages/better-auth/src/api/routes/sign-in.ts (+5 -5)
📝 packages/better-auth/src/plugins/username/index.ts (+3 -2)

📄 Description

The sign-in endpoint currently accepts a dontRememberMe argument, which is not ideal. First, dontRememberMe isn’t grammatically correct it should be don’tRememberMe which isn't valid syntax. And more importantly, it's not intuitive. This PR renames it to rememberMe, which defaults to true.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/better-auth/better-auth/pull/476 **Author:** [@Bekacru](https://github.com/Bekacru) **Created:** 11/9/2024 **Status:** ✅ Merged **Merged:** 11/10/2024 **Merged by:** [@Bekacru](https://github.com/Bekacru) **Base:** `main` ← **Head:** `fix/remember-me` --- ### 📝 Commits (3) - [`d7f4095`](https://github.com/better-auth/better-auth/commit/d7f40956dd1b2ed6dcb18a7cd58b48c481210922) fix: change dontRemeberMe to remeberMe - [`a321b09`](https://github.com/better-auth/better-auth/commit/a321b0902f6a59bbfd129ca52178e1a761b37d37) fix: demo - [`9b29e18`](https://github.com/better-auth/better-auth/commit/9b29e184a474197a8dc9ae00c6ba8f8b5150183c) fix: username plugins ### 📊 Changes **11 files changed** (+20 additions, -19 deletions) <details> <summary>View changed files</summary> 📝 `demo/nextjs/app/dashboard/organization-card.tsx` (+2 -2) 📝 `demo/nextjs/components/sign-in.tsx` (+1 -1) 📝 `docs/content/docs/authentication/email-password.mdx` (+1 -1) 📝 `examples/astro-example/src/components/sign-in.tsx` (+1 -1) 📝 `examples/expo-example/package.json` (+1 -1) 📝 `examples/nextjs-example/components/sign-in.tsx` (+2 -2) 📝 `examples/nuxt-example/package.json` (+2 -2) 📝 `examples/remix-example/app/components/sign-in-card.tsx` (+1 -1) 📝 `packages/better-auth/src/api/routes/session-api.test.ts` (+1 -1) 📝 `packages/better-auth/src/api/routes/sign-in.ts` (+5 -5) 📝 `packages/better-auth/src/plugins/username/index.ts` (+3 -2) </details> ### 📄 Description The sign-in endpoint currently accepts a `dontRememberMe` argument, which is not ideal. First, `dontRememberMe` isn’t grammatically correct it should be `don’tRememberMe` which isn't valid syntax. And more importantly, it's not intuitive. This PR renames it to `rememberMe`, which defaults to `true`. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-15 19:33:04 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/better-auth#20218