smswithoutborders/SMSWithoutBorders.com

Rename ENV vars for better identification

Closed this issue · 4 comments

https://github.com/smswithoutborders/smswithoutborders.com/blob/9acf563244ca06cccdfbd57e7771e785b6c707bf/src/pages/Profile.jsx#L77

to enhance devs ability to understand why they need each variable, it's best we rename variables to match the functionalities they are for.
Suggesting we rename variable VITE_ROUTER_URL to VITE_GATEWAY_SERVER

We no longer use Vite but the variable is still router could change to Gateway

Sorry I don't understand. Currently looking at the naming convention - not sure about the tooling

I see. The build tools define the variable naming conventions. When we used Vite every env variable had to be prefixed with VITE_. Now we prefix them with REACT_APP_. i will update ROUTER_URL to GATEWAY_SERVER so it matches current spec

Thanks, I will go ahead and close this as soon you make the commit 👍