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

Frontend update 623ab16 #181

Merged
merged 10 commits into from
Jun 20, 2024
Merged

Frontend update 623ab16 #181

merged 10 commits into from
Jun 20, 2024

Conversation

rv2931
Copy link
Collaborator

@rv2931 rv2931 commented May 19, 2024

Mise à jour du frontend à partir du dépôt de @tanega (https://github.com/tanega/trailwatch-front-kickoff/)
Version courante : main/commit 623ab16
Ajout d'un ignore frontend/public/data pour éviter d'indexer/visualiser des data

L'idée étant d'intégrer une ultime version permettant au front de travailler directement dans le dépôt commun à partir de maintenant

Pour la partie dev embarquée au sein du conteneur Docker, je pense qu'il va falloir éclaircir une meilleure manière de faire
En effet j'ai l'impression que les dev actuels développent hors docker et une app JS devant plutôt être compilée, le montage d'un volume de développement au sein du conteneur nécessite une recompilation au sein du conteneur.

Ce point d'un conteneur docker permettant et le déploiement et le développement est à rediscuter sur la manière de faire mais ne fait pas partie de cette PR

@rv2931
Copy link
Collaborator Author

rv2931 commented May 27, 2024

@tanega J'ai fait une version plus adaptée au dev pour le front

  • montage du code complet
  • génération du node_modules (npm install) et du .next (npm run build) vu que le montage de volume les supprimes par défaut
  • exécution du nom run dev
    Si t'as moyen de me valider que ça fonctionne pour le dev

@rv2931 rv2931 force-pushed the frontend_update_623ab16 branch 5 times, most recently from 4597483 to 10f7d1c Compare June 3, 2024 16:45
@rv2931 rv2931 requested a review from ComeTiss June 13, 2024 19:08
@rv2931 rv2931 force-pushed the frontend_update_623ab16 branch 2 times, most recently from a17848c to e4f8c94 Compare June 20, 2024 12:35
herve.le-bars and others added 10 commits June 20, 2024 15:04
…les missing

Because frontend needs frontend build that is done during building docker, mounting volume for dev removes this building
Entrytpoint could be modified to detect that frontend needs to be rebuild but for the moment, removing volume mounting to assure frontend works
# Conflicts:
#	docker-compose-load-data.yaml
…les missing

Because frontend needs frontend build that is done during building docker, mounting volume for dev removes this building
Entrytpoint could be modified to detect that frontend needs to be rebuild but for the moment, removing volume mounting to assure frontend works

# Conflicts:
#	docker-compose.yaml
@rv2931
Copy link
Collaborator Author

rv2931 commented Jun 20, 2024

Procédure pour le lancement d'une stack complète from scratch

@rv2931 rv2931 merged commit 3059bb9 into main Jun 20, 2024
4 of 7 checks passed
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.

1 participant