Software Público Brasileiro

@leandronunes @vfcosta estamos precisando de ajuda aqui, @kanashiro subiu uma instancia do pairwise-api (versão do upstream sem as modificações de vocês) e não funcionou, precisamos saber se a implementação do lado do noosfero depende das mudanças que vocês fizeram no pairwise-api, precisamos também definir como iremos disponibilizar essas mudanças feitas no pairwise-api para outras pessoas que queiram subir um noosfero com o pairwise habilitado, penso que será preciso incorporar as mudanças ao upstream do pairwise-api.

Hoje consegui reproduzir parcialmente o pairwise + noosfero localmente, segue o status abaixo:

Inicialmente tentei instalar o plugin do pairwise (master) + pairwise-api oficial (upstream - master), nesse caso o plugin não funcionou. Gostaria de saber se tem como o @leandronunes e/ou @vfcosta me dizer quais as funcionalidades que foram implementadas no fork do pairwise-api do participa e não foram retornadas ao upstream.

Em seguida, tentei instalar o plugin do pairwise (master) + o fork do pairwise (participa - master), nesse caso a funcionalidade do pairwise funcionou parcialmente. Durante a instalação foi necessária a adição da gem 'will_paginate' que é utilizada pelo plugin, entretanto não foi adicionada ao Gemfile. Após a instalação e configuração do plugin via interface de administração, o teste inicial de criar uma questão pairwise ('Pairwise question') foi finalizado com sucesso. Ao tentar votar em uma questão tudo ocorreu bem também, mas ao tentar acessar as outras abas ('Result', 'Edit' e 'Manage Ideas') ouve um erro, que está sendo apresentado abaixo:

TypeError (can't convert Symbol into Integer):
  app/controllers/choices_controller.rb:64:in `[]'
  app/controllers/choices_controller.rb:64:in `index'
  vendor/ruby/1.9.1/gems/actionpack-2.3.18/lib/action_controller/base.rb:1333:in `perform_action'
  vendor/ruby/1.9.1/gems/actionpack-2.3.18/lib/action_controller/filters.rb:617:in `call_filters'
  vendor/ruby/1.9.1/gems/actionpack-2.3.18/lib/action_controller/filters.rb:610:in `perform_action_with_filters'

Gostaria de saber do pessoal do serpro se eles têm conhecimento do erro acima.

Por enquanto foi isso :)

@joenio @kanashiro o pairwise que estamos rodando é o oficial + algumas modificações: https://softwarepublico.gov.br/gitlab/noosferogov/pairwise-api

O ideal seria submeter as modificações para o master do projeto sim. Na verdade do ideal seria pegar o projeto original e verificar se essas modificações ainda são necessárias. Se você observar os últimos commits não é tanta coisa a ser feita do jeito certo. Na época que isso foi feito não tinhamos condições de submeter o projeto para a comunidade naquele momento. O plugin funciona no branch staging do software público e essa gem will_paginate já existe neste branch. Lucas baixa essa versão do pairwise e verifica se tudo funciona.

@leandronunes na verdade o pairwise-api do participa é um fork porque possui modificações e não está atualizado com upstream, se não me engano já foram lançadas 2 ou 3 versões do pairwise-api depois de feito o fork do participa.

Leandro, eu tive que instalar o 'will_paginate' na instalação do pairwise-api e não na do plugin pairwise do noosfero. Com o plugin mesmo não tive problema com a master, mas de qualquer forma vou testar depois com esse branch staging.

@leandronunes nós vamos precisar que vocês façam um esforço para enviar essas modificações ao upstream do pairwise-api, não temos condição de entrar nisso. Enviar essas mudanças ao pairwise-api é bem importante, esse é o primeiro passo para criarmos um caminho sustentável para que outras pessoas consigam subir suas próprias instâncias de pairwise-api.

@kanashiro sei que é um fork :) (oficial + algumas modificações = fork) Sei que já foram lançadas novas versões, mas ainda não tivemos condições de priorizar a atualização, pois isso implica também em analisar o que foi modificado e incorporar as modificações que fizemos. @joenio não teremos condições de fazer este esforço neste momento para enviar as modificações para o upstream também.

@ricardopoppi você pode nos ajudar aqui? é importante finalizar as mudanças feitas no pairwise-api e incorporar ao upstream, é possível viabilizar para que a equipe do serpro coloque esforços nisso? do contrário corremos o risco de não incorporar o pairwise ao noosfero.

@joenio como priorizamos a demanda de frontend ao serpro estamos com previsão de que eles consigam colocar energia disso a partir do dia 14 (daqui a duas semanas), isso inviabiliza a incorporação de algum jeito? (cc @leandronunes )

@ricardopoppi não inviabiliza não.

maravilha!

@ricardopoppi não invialbilza não. Inclusive o pessoal pode fazer esse trabalho de envio do que foi modificado para o upstream se acharem conveniente. Só precisa ver se ainda faz sentido, pois "em tese" pode ser que as modificações já tenham sido implementadas lá no upstream.

@leandronunes @ricardopoppi não podemos entrar no desenvolvimento do pairwise-api, estas mudanças não são apenas conveniente, elas são necessárias para que a solução seja sustentável ao longo do tempo, ou ao menos, devemos remover as dependências do lado do noosfero e trabalhar com a versão oficial do upstream, talvez este seja o melhor caminho.

@joenio @kanashiro @leandronunes temos documentado em algum lugar essas modificações? imagino que seria o primeiro passo pra avaliar se elas podem ser "abandonadas" e passarmos a trabalhar com a versão do upstream.

mentioned in issue #143

Status changed to closed

Vamos retomar este trabalho a partir da Sprint 7 na issue #143, que é quando o Serpro poderá nos ajudar