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: domain-functions #3

Open
wants to merge 4 commits into
base: remix-forms
Choose a base branch
from

Conversation

gustavoguichard
Copy link

@gustavoguichard gustavoguichard commented Dec 16, 2022

This is just to show how we've been using domain-functions at Seasoned and maybe raise some curiosity.
It gets better as the composition of DFs increase in complex routes.
Another benefit is not having to maintain the types as everything is parsed in runtime and then inferred.

When not inferred and checked in runtime the types could be misleading as code or even the cart fetch response could change.

Feel free to accept or reject this PR. I just wanted to show this architecture ;)

@gustavoguichard gustavoguichard changed the title Domain functions feat: domain-functions Dec 16, 2022
@franklinjavier
Copy link
Owner

Ficou muito bom, thanks. Vou deixar o PR como referência de um advanced approach (vou referenciar no readme). Por enquanto vou manter mais simples pra exemplificar pra galera do GB.

Vlw pela disponibilidade Guga, o brabo!

@franklinjavier
Copy link
Owner

#2

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.

2 participants