-
Notifications
You must be signed in to change notification settings - Fork 0
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
Deploy to Vercel with mocks #17
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Love it <3 One comment about commenting out code.
.env.sample
Outdated
#Optional | ||
#NEXT_PUBLIC_MOCK_API=TRUE |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we really need this kind of comments left out? Just remove it, if we want this constant we can add it. And if we want it configurable then we should make a way to configure it I think...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, I'll remove the comments but keep the variables there. This file is only used to easily generate a .env.local
by copying it
c40213d
to
4820f0a
Compare
72c87a6
to
099616f
Compare
No description provided.