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

Boa noite #7

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

Boa noite #7

wants to merge 51 commits into from

Conversation

diogorg
Copy link

@diogorg diogorg commented Aug 1, 2020

Pessoal da T10, boa noite. Como estão?
Agradeço muito a oportunidade de criar esta pequena aplicação.
Fiz de coração e pelo desafio/diversão.
Caprichei contudo há muitos pontos de melhoria.
Vou depois expandir alguns pontos. Identificar outros padrões de cartas do Poker eu acho que é uma estratégia bem legal.
Permitir criar a lista de cartas de forma aleatória.
User o local storage para manter o visual das cartas e as opções selecionados pelo usuário.

Acredito que só vi depois que não era para usar um framework CSS. Usei o Material-ui não porque estou acostumado, mas justamente pelo contrário, para aprender ele.

Mudei a forma de salvar na API. Colocando as cartas lá, mas o pivot deixei na URL. O que é interessante para testar uma mesma configuração e mudar o pivot dinamicamente.

A parte de testes esta crú. Vou melhorar este ponto também. Há diversos pontos do software que tenho certeza que posso fazer melhor e de forma mais performática.

Anyway. Um bom final de semana para vocês. Vou subir a aplicação para testar rodando e depois coloco o link.
Um super abraço, fiquem com Deus.

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