search

nextauthjs / next-auth

Authentication for the Web.
https://authjs.dev
ISC License
24.11k stars 3.34k forks source link

MongoDB adapter "Add the MongoDB client" is out of date from NextJS #11255

Closed nalenb closed 2 weeks ago

nalenb commented 2 months ago

What is the improvement or update you wish to see?

On this adapter page MongoDB Adapter the example code for connecting in NextJS is out of date. MongoDB team says not to cache the promise. NextJS has updated their example documentation

Is there any context that might help us understand?

Response from MongoDB

MongoDB Jira ticket

Does the docs page already exist? Please link to it.

https://authjs.dev/getting-started/adapters/mongodb

github-actions[bot] commented 1 month ago

This issue was marked with the good first issue label by a maintainer.

This means that it is a good candidate for someone interested in contributing to the project, but does not know where to start.

Have a look at the Contributing Guide first.

This will help you set up your development environment to get started. When you are ready, open a PR, and link back to this issue in the form of adding Fixes #1234 to the PR description, where 1234 is the issue number. This will auto-close the issue when the PR gets merged, making it easier for us to keep track of what has been fixed.

Please make sure that - if applicable - you add tests for the changes you make.

If you have any questions, feel free to ask in the comments below or the PR. Generally, you don't need to @mention anyone directly, as we will get notified anyway and will respond as soon as we can)

[!NOTE]
There is no need to ask for permission "can I work on this?" Please, go ahead if there is no linked PR :slightly_smiling_face:

odeta939 commented 2 weeks ago

This was updated in #11393, so the issue can be closed.

balazsorban44 commented 2 weeks ago

Closing per above comment.