Skip to content
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

add search to localhost #6083

Closed
wants to merge 2 commits into from
Closed

add search to localhost #6083

wants to merge 2 commits into from

Conversation

mirnawong1
Copy link
Contributor

@mirnawong1 mirnawong1 commented Sep 18, 2024

this pr adds search to our localhost env for easier testing

asana task

tested and it works!
Screenshot 2024-09-18 at 10 45 42

@mirnawong1 mirnawong1 requested a review from a team as a code owner September 18, 2024 09:49
Copy link

vercel bot commented Sep 18, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
docs-getdbt-com ✅ Ready (Inspect) Visit Preview Sep 18, 2024 9:56am

@github-actions github-actions bot added Docs team Authored by the Docs team @dbt Labs size: x-small This change will take under 3 hours to fix. labels Sep 18, 2024
@JKarlavige
Copy link
Collaborator

Hey @mirnawong1, .env files are intended on being used locally without being pushed to Git. In most cases they are used for storing private API keys locally. Then once the code is being served in production, it pulls the env variables from the hosting provider (Vercel in our case).

Although we don't have any sensitive API keys here, I would prefer we continue to keep the .env files out of Git.

.env files are often marked in the .gitignore file, which prevents certain files from accidentally being pushed to Git.

@mirnawong1 mirnawong1 closed this Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Docs team Authored by the Docs team @dbt Labs size: x-small This change will take under 3 hours to fix.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants