Skip to content

UN-2376 refactor: semplificare gestione file .env#1519

Open
sinatragianpaolo wants to merge 6 commits intodevelopfrom
UN-2376
Open

UN-2376 refactor: semplificare gestione file .env#1519
sinatragianpaolo wants to merge 6 commits intodevelopfrom
UN-2376

Conversation

@sinatragianpaolo
Copy link
Contributor

@sinatragianpaolo sinatragianpaolo commented Feb 17, 2026

Summary

  • Aggiornato .env.template con tutte le variabili necessarie e placeholder descrittivi
  • Sostituito il README boilerplate CRA con documentazione specifica del progetto (setup locale, comandi, pattern build-time vs runtime)
  • Eliminato .env.test (non veniva mai caricato da nessuno: Playwright usa mode development, non esiste vitest/jest)
  • Rimosso REACT_ZAPIER_WEBHOOK da .env.development e .env.production (Vite carica solo variabili con prefisso REACT_APP_, quindi non veniva mai letta)
  • Rimosso REACT_APP_CROWD_WP_URL= vuoto da .env.development e .env.production
  • Rimosso .env.test.local da .gitignore (non piu' necessario)
  • Aggiunto file environment mancanti a .dockerignore

Requisiti

  • Nessuna regression sulla creazione e sull'uso dei file .env in tutti gli ambienti (locale, Docker, CI)

Verifiche effettuate

  • npm run build passa senza errori
  • npm start avvia il dev server senza errori

Riferimento

UN-2376

@sinatragianpaolo sinatragianpaolo marked this pull request as ready for review February 18, 2026 09:50
@Kariamos
Copy link
Contributor

negli script del package.json aggiungerei anche:

"local-schema": ". ./.env.local; API_URL=$REACT_APP_API_URL/reference/; npx @appquality/fixture-generator -u $API_URL && -u $API_URL && npx prettier --write 'tests/api/**/*.json'",
    "local-api": ". ./.env.local; API_URL=$REACT_APP_API_URL/reference/ npx @rtk-query/codegen-openapi src/features/api/config.ts",

se vuoi provare qualcosa con le api in locale (ovviamente .env.local deve avere gli url configurati in localhost

Copy link
Collaborator

@marcbon marcbon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

questa va prima deployata e testata per verificare che non si rompa niente sugli ambienti staging/prod

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants