Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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 .

@JNSPK JNSPK requested a review from douglasduteil August 26, 2024 10:59
@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 😅

# for free to join this conversation on GitHub. Already have an account? # 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