search

braziljs / braziljs.org

BrazilJS website
http://braziljs.org
106 stars 50 forks source link

Fix lista de eventos na home. #147

Closed thyagorafaell closed 10 years ago

zenorocha commented 10 years ago

Não entendi @thyagorafaell, pode explicar melhor o porquê disso?

thyagorafaell commented 10 years ago

@zenorocha desculpe, realmente faltou uma descrição. Na listagem de eventos na home, o loop espera três eventos, recentemente foi adicionado o evento "Siga em Front", então não está mais ocorrendo o erro. Quando há menos de três eventos futuros cadastrados, o loop tem sucesso em pegar os índices 0 e 1 do array de eventos, porém o índice 2 não está definido. Portando o laço deve percorrer: caso tenha três ou mais eventos: 3, caso tenha menos: o tamanho do array.

zenorocha commented 10 years ago

Just started reviewing :)

:octocat: Sent from GH.

zenorocha commented 10 years ago

Thank you, pull request merged!

:octocat: Sent from GH.

zenorocha commented 10 years ago

Perfeito, saquei. Valeu!