Skip to content

Comments

add NEXTAUTH_URL to environment variables#25

Merged
pumfleet merged 1 commit intocalcom:mainfrom
mfts:pr-env-variables
Apr 16, 2021
Merged

add NEXTAUTH_URL to environment variables#25
pumfleet merged 1 commit intocalcom:mainfrom
mfts:pr-env-variables

Conversation

@mfts
Copy link
Contributor

@mfts mfts commented Apr 15, 2021

When I'm running it locally I got an error from NEXTAUTH_URL that it's not present in the .env file.

https://next-auth.js.org/configuration/options#nextauth_url

@vercel
Copy link

vercel bot commented Apr 15, 2021

@mfts is attempting to deploy a commit to the calendso Team on Vercel.

A member of the Team first needs to authorize it.

@rittmang
Copy link

As per https://github.com/calendso/calendso/issues/4#issuecomment-818747886, shouldn't this be <URL>/api/auth instead of just <URL> ? Or can it be both?

@mfts
Copy link
Contributor Author

mfts commented Apr 16, 2021

As per next-auth docs, it can be both. You would only add the long url i.e. <BASE-URL>/api/auth if you use a different specialty directory like <BASE-URL>/custom-route/api/auth.

That's because next-auth already knows the default directory so you can just the BASE-URL.

@pumfleet pumfleet merged commit bd84803 into calcom:main Apr 16, 2021
@pumfleet
Copy link
Contributor

Thanks for your contribution!

@rittmang
Copy link

As per next-auth docs, it can be both. You would only add the long url i.e. <BASE-URL>/api/auth if you use a different specialty directory like <BASE-URL>/custom-route/api/auth.

That's because next-auth already knows the default directory so you can just the BASE-URL.

Oh ok cool! TIL 😅

KATT added a commit that referenced this pull request Sep 3, 2021
zomars pushed a commit that referenced this pull request May 23, 2023
emrysal pushed a commit that referenced this pull request Feb 18, 2026
* perf: remove Sentry span from getPeriodStartDatesBetween

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>

* perf: remove Sentry prismaIntegration to reduce span volume

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>

---------

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
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.

3 participants