Issue #43
Extrair melhorias da API implementadas pelo Serpro no branch staging e incorporar ao master do Noosfero
Esta issue é uma continuidade do trabalho feito em #19.
-
Reassigned to @marcosronaldo
-
Link para o branch staging:
-
@marcosronaldo conseguiu fechar isso?
-
@marcosronaldo beleza, amanhã discutimos isso em detalhes. Obrigado!
-
Conversarmos com terceiro, ele vai empacotar a gem e @marcosronaldo irá incorporar as modificações da API que usam ela.
-
Enquanto a master do noosfero oficial não descongela, alguém poderia dar uma olhada nesse commit: link. Acredito que peguei tudo que era possível (algumas coisas que dependem de outras estruturas no backend não foram incorporadas). Criei uns testes para endpoints novos que não continham teste.
@leandronunes @vfcosta podem dar uma olhada e ver se tem alguma sugestão?
-
Revisei lá e a única reclamação que eu tenho é que o commit tem várias coisas diferentes juntas. Em relação às alterações, tá massa!
Resolvendo a questão do empacotamento da gem, por mim podemos incluir no master.
-
@marcosronaldo não olhei o commit por completo mas não achei o método
followed_by_me
: https://softwarepublico.gov.br/gitlab/noosferogov/noosfero/blob/staging/lib/noosfero/api/v1/articles.rb#L42 -
como vai ficar então? será feito um MR adicional depois?
-
@marcosronaldo são esses marge-requests aqui?
- https://gitlab.com/noosfero/noosfero/merge_requests/725
- https://gitlab.com/noosfero/noosfero/merge_requests/738
vamos então revisar esse MR e incorporar ao noosfero já que é uma dependência para esta issue aqui. correto?
-
Status changed to closed