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

feat(form): location field on exchange edit #1736

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

JNSPK
Copy link
Collaborator

@JNSPK JNSPK commented Aug 26, 2024

${\color{lightgreen}FIX}$

fix #1735

@JNSPK
Copy link
Collaborator Author

JNSPK commented Aug 26, 2024

HELP !
Moi pas comprendre pourquoi il ne trouve pas le module. Lorsque j'essaye d'importer TRPC React depuis :trpc/client il me dit qu'il trouv epas le module. Je met le chemin entier y'a pas de souci en dev mais apparemment ça ne marche pas pendant le build .

@douglasduteil
Copy link
Member

C'est normal, mon cher @JNSPK, pour forcer la séparation des couches.

C'est l'heure du du du du du du course d'architecture logiciel 📇
image

Je te laisse faire quelques recherches mais sur ce projet j'essaie de tirer partie de l'archi N-Layer https://medium.com/@iamprovidence/backend-side-architecture-evolution-n-layered-ddd-hexagon-onion-clean-architecture-643d72444ce4

On cherche pour chaque context d'isolé les couches a travers pnpm qui n'install pas les liens entre les packages. LA règle en général c'est d'avoir une couche de présentation agnostique (notre @1.ui/react et nos @1.modules/*.ui) de même pour l'infra (nos @1.infra/*) car la caractéristique des couches applicatives c'est qu'elle sont le lien entre la données et le mise en page ;)

@JNSPK
Copy link
Collaborator Author

JNSPK commented Aug 26, 2024

Comment je fais dans mon cas alors ? J'ai besoin de faire la query via trpc mais je peux pas l'appeler ici ?
Parfois je comprend rien, le location field va me rendre fou 😅

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.

feat(form): location field on exchange edit
2 participants