search

ejplatform / ej-server

EJ uses gamification and machine learning to promote discussions and participation in a virtual democracy
https://ejplatform.github.io/ej-server/
Other
38 stars 17 forks source link

Rotas de participação e moderação das conversas não estão incluindo o slug do mural #683

Closed ricardopoppi closed 5 years ago

ricardopoppi commented 5 years ago

Após criar uma conversa com um usuário comum, tanto o acesso à moderação (rota /moderate) quanto o acesso à participação na conversa não estão incluindo o slug do mural.

Rotas que não estão usando o slug do mural:

Rota que funciona tanto sem slug de mural quanto com slug de mural (estranho pode ser algum bug):

Para replicar:

1) Crie um mural e uma conversa com usuário comum 2) Acesse o mural e tente clicar no "participar agora" ou no "moderar conversa" 3) Observe que ambas as rotas não estão incluindo o /\<slug-do-mural>/ 4) Na tela de moderação, incluindo o \<slug-do-mural> manualmente, o botão de "editar conversa" também está com essa falha

fabiolamfleury commented 5 years ago

Esses problemas surgiram na refatoração para utilizar url reverse das conversas, como a relação da conversa com boards sai um pouco do padrão, o erro acontece, não consegui entender muito bem a implementação que gera as urls, então vou tentar conversar com o professor Fábio.

fabiolamfleury commented 5 years ago

O pr https://github.com/ejplatform/ej-server/pull/689 resolve esses problemas e alguns outros com essa nova implementação de url que eu acabei esbarrando :) já está em revisão