[PR #210] [MERGED] Use Vite's way of using env variables #218

New issue

Closed

opened 2026-02-27 04:59:35 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 04:59:35 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/guyzyl/whatsapp-contact-sync/pull/210
Author: @MRDGH2821
Created: 7/10/2025
Status: ✅ Merged
Merged: 7/18/2025
Merged by: @guyzyl

Base: main ← Head: main

📝 Commits (5)

0572f99 Directly run main script with nodemon
f0b0ce0 Use env variables to set client id & api key
d6e2db0 Fix typos
be061e8 Add sample.env files & update instructions
bdf36c6 Rename environment variables for clarity

📊 Changes

6 files changed (+32 additions, -11 deletions)

View changed files

📝 .dockerignore (+1 -0)
📝 README.md (+19 -8)
📝 server/package.json (+1 -1)
➕ server/sample.env (+4 -0)
➕ web/sample.env (+5 -0)
📝 web/src/pages/GoogleAuth.vue (+2 -2)

📄 Description

This PR removes hard coded secrets & instead uses Vite's way of using env variables.
I have been using it this way for multiple times as it avoids the hassle of constantly renaming hardcoded secrets it I git pull

Additionally nodemon can directly watch & run typescript files, so the npm dev script for launching server was also updated.

_{🔄 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/guyzyl/whatsapp-contact-sync/pull/210 **Author:** [@MRDGH2821](https://github.com/MRDGH2821) **Created:** 7/10/2025 **Status:** ✅ Merged **Merged:** 7/18/2025 **Merged by:** [@guyzyl](https://github.com/guyzyl) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (5) - [`0572f99`](https://github.com/guyzyl/whatsapp-contact-sync/commit/0572f9963f9e4bb5b500f1827f65143b387a5243) Directly run main script with nodemon - [`f0b0ce0`](https://github.com/guyzyl/whatsapp-contact-sync/commit/f0b0ce03c89d79eb6a5ae6b80688e321703648d9) Use env variables to set client id & api key - [`d6e2db0`](https://github.com/guyzyl/whatsapp-contact-sync/commit/d6e2db0bc1d4e5e4914b76144966a2f167a0caee) Fix typos - [`be061e8`](https://github.com/guyzyl/whatsapp-contact-sync/commit/be061e878a192d74e544c494da9cabb9d209e56e) Add sample.env files & update instructions - [`bdf36c6`](https://github.com/guyzyl/whatsapp-contact-sync/commit/bdf36c69219d37c2ca2b01ea8b5ed16d2990b7fe) Rename environment variables for clarity ### 📊 Changes **6 files changed** (+32 additions, -11 deletions) <details> <summary>View changed files</summary> 📝 `.dockerignore` (+1 -0) 📝 `README.md` (+19 -8) 📝 `server/package.json` (+1 -1) ➕ `server/sample.env` (+4 -0) ➕ `web/sample.env` (+5 -0) 📝 `web/src/pages/GoogleAuth.vue` (+2 -2) </details> ### 📄 Description This PR removes hard coded secrets & instead uses Vite's way of using env variables. I have been using it this way for multiple times as it avoids the hassle of constantly renaming hardcoded secrets it I `git pull` Additionally nodemon can directly watch & run typescript files, so the npm dev script for launching server was also updated. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>