search

coopera / teamtracker

0 stars 0 forks source link

Pull Requests Feed #29

Closed brnrdog closed 9 years ago

brnrdog commented 9 years ago

Construção da feed de pull requests.

Rota: /feed/pull_request

Observações

Eu estou "mockando" objetos e os usando na view. A ideia é que depois estes objetos sejam instanciados com dados reais. Dei o nome de "stubs" já que eles funcionam como um esboço/esqueleto dos objetos que serão usados nas views. Eles se encontram na pasta stubs. Também criei factories só para instanciar esses stubs com fake data e usar por enquanto, estão na pasta factories. Outro ponto é que criei a pasta presenters, onde estarão as classes presenters responsáveis por alguns métodos usados nas views – o objetivo aqui é isolar a lógica na view.

No mais é isso.

IgorMarques commented 9 years ago

:+1:

IgorMarques commented 9 years ago

Posso merjar? É bem simples... (ou @bernardogfilho vai fazer mais commits ainda?)

brnrdog commented 9 years ago

Vou fazer mais commits sim, por isso o in progress. Eu pensei em fazer toda a feature da feed de PR aí, mas é melhor quebrar em features bem modulares mesmo.

Esse PR vai ser a base do escopo das features de feed de PR. Quando eu terminar, eu boto a label ready.

IgorMarques commented 9 years ago

blzinha

brnrdog commented 9 years ago

Estado atual da feed:

image

brnrdog commented 9 years ago

Eu farei uma refatorada na view, e em seguida acredito que pode ser feito merge neste PR.

IgorMarques commented 9 years ago

ok, @bernardogfilho. Avisa quando terminar que damos uma olhada

brnrdog commented 9 years ago

Pois certo :+1:

brnrdog commented 9 years ago

Só uma observação: to usando uma api pra pegar usuários aleatórios, com fotos, nome, login, etc. Então, a página pode demorar um ou dois segundos, pq tá fazendo a requisição à api :kissing_cat:

brnrdog commented 9 years ago

E testes são bem-vindos :heart:

IgorMarques commented 9 years ago

:+1: