Software Público Brasileiro

Reassigned to @diguliu

@diguliu esse aqui você incorporou?

@joenio não incorporei pq ainda tinha que confirmar com terceiro se a gem api-pagination estava empacotada. Já confirmei com ele e irei incorporá-la em breve.

massa, valeu!!!

@diguliu MR 801

Ainda está pendente o empacotamento da gem 'grape-entity' versão 0.4.8 para incorporação desse MR. Estou vendo isso com @terceiro.

Beleza, vamos adiar isso para a Sprint-6 então.

@diguliu como ficou o empacotamento dessa gem? já incorporou as melhorias da API que estavam esperando ela?

Estava esperando terceiro voltar de viagem pra me ajudar com isso, mas kanashiro se ofereceu para fazer isso. Acredito que em alguns dias isso estará resolvido.

@diguliu como o lançamento da próxima versão do noosfero ainda está distante, será que não poderíamos antecipar e incorporar logo esse merge request?

@diguliu somente para reforçar que isso tá trazendo impacto sobre aquele trabalho da refatoração do frontend já que volta e meia precisamos mexer na api.

@diguliu @joenio @daniela o branch não poderia ser incorporado e depois ser feito o pacote debian? Há essa necessidade de só incorporar quando o pacote debian existir? Hoje isso está virando blocker para o nosso trabalho aqui. O que vai acabar acontecendo é que vamos ter que evoluir a api do branch staging e teremos que revisar todo o código novamente :(

@leandronunes o blocker não é mais o pacote debian. Como falei a 2 dias atrás, Kanashiro resolveu esse problema. Ainda não incorporei esse MR porque gostaria de resolver a situação de testes unitários quebrando intermitentemente e testes de plugins quebrados por conta do https://gitlab.com/noosfero/noosfero/merge_requests/803. Como esse MR é muito grande, quero evitar incorporar muitas coisas novas e ter que depois reverter ele. Estamos quase com uma solução pronta (que não involve reverter o MR) que está sendo avaliada aqui: https://gitlab.com/noosfero/noosfero/merge_requests/812. Se vocês puderem opinar a respeito, vai agilizar bastante o processo.

Parece que @joenio já incorporou a solução proposta por @manzo. Vou incorporar o merge da api.

sim @diguliu, e também estou trabalhando nos testes unitários falhando.

@diguliu @joenio tinha entendido diferente. foi mau :)

@diguliu @joenio Qual o status dessa incorporação?

@leandronunes está pendente apenas a correção de um teste unitário. Veja em: https://gitlab.com/noosfero/noosfero/merge_requests/801

@diguliu @marcosronaldo atualizei o MR em relação ao master e fiz 2 pequenas correções, criei um novo MR em:

• https://gitlab.com/noosfero/noosfero/merge_requests/815

Status changed to closed