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

Implementacao do Prototipo do front-end #2

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

Implementacao do Prototipo do front-end #2

wants to merge 9 commits into from

Conversation

fhagner
Copy link
Contributor

@fhagner fhagner commented Jan 31, 2014

Implementei um prototipo de front-end para melhor visualizar o resultado.

@laurocaetano
Copy link

Hey bro, it needs rebase 😃

@laurocaetano
Copy link

Precisa ser feito em um html mesmo?
Por que não um arquivo php com os testes e a implementação em uma classe separada?

@gpedro
Copy link
Member

gpedro commented Jan 31, 2014

o correto é só o código puro, mas com essa front ficou até legalzinha, engraçada e mais compreensível. De qualquer forma, os testes funcionam (segundo o Travis-CI.org), talvez com a implementação da front pode ocorrer erros. Por mim, está de bom tamanho. (nem consegui dar merge).

Ai tem que ver com @dadoce.

@dadoce
Copy link
Member

dadoce commented Jan 31, 2014

Massa :)

Como eu disse lá na thread do grupo, os PR vão sendo aceitos pra gente ir trocando figurinhas.
Minha ideia é como a do @laurocaetano: fazemos a API bonitinha (conforme o Curinga orientou) com os testinhos e, separadamente, consumimos.

O que acham?

@fhagner
Copy link
Contributor Author

fhagner commented Jan 31, 2014

A idéia do front foi só pra exemplificar caso alguém fique perdido quanto ao resultado.
Talvez se tivesse colocado como um 'exemplo de uso' ficaria melhor.

@dadoce
Copy link
Member

dadoce commented Jan 31, 2014

A ideia ta ótima, @fhagner, vamos continuar nela sim ;)

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.

4 participants