ButterCMS / vuejs-starter-buttercms

Drop-in proof-of-concept VueJs app, fully integrated with your ButterCMS account
https://vuejs-starter-buttercms.vercel.app
5 stars 9 forks source link

Rename ENV variables to be prefixed with "VUE_" instead of "VITE_" #4

Closed prokopsimek closed 2 years ago

prokopsimek commented 2 years ago

Is there any reason to have the name prefixed with "VITE"?

gerzitom commented 2 years ago

Yes, there is. Ony VITE_ prefixes are exposed for the app. It is security reason from Vite engine: https://vitejs.dev/guide/env-and-mode.html.

prokopsimek commented 2 years ago

@ViolanteCodes Please, take a look if these var names are ok for you.

ViolanteCodes commented 2 years ago

@ViolanteCodes Please, take a look if these var names are ok for you.

Yes - if it's necessary in order for the Vite engine to work, we are okay with the one-off change here. Thanks :) !