search

fga-eps-mds / 2019.2-Acacia

Projeto de desenvolvimento de uma aplicação para auxiliar a colheita colaborativa. Ambiente de Homologação: http://45.55.46.19:8080 Ambiente de Produção: http://45.55.46.19:8081
https://fga-eps-mds.github.io/2019.2-Acacia/#/
GNU Affero General Public License v3.0
9 stars 17 forks source link

Ajuste das rotas do backend (develop) #194

Closed durvalcarvalho closed 4 years ago

durvalcarvalho commented 4 years ago

Descrição

As rotas que antes só funcionavam com a barra ('/') no final agora também funcionam sem a barra.

Resolve (Issues)

125 Ajuste das rotas do backend

Tarefas gerais realizadas

codecov-io commented 4 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@f6fab99). Click here to learn what that means. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##             develop     #194   +/-   ##
==========================================
  Coverage           ?   93.52%           
==========================================
  Files              ?       36           
  Lines              ?     1034           
  Branches           ?        0           
==========================================
  Hits               ?      967           
  Misses             ?       67           
  Partials           ?        0
Flag Coverage Δ
#unittests 93.52% <100%> (?)
Impacted Files Coverage Δ
src/users/urls.py 100% <ø> (ø)
src/acacia/urls.py 100% <ø> (ø)
src/property/urls.py 100% <100%> (ø)
src/tree/urls.py 100% <100%> (ø)
src/harvest/urls.py 100% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update f6fab99...25491af. Read the comment docs.

durvalcarvalho commented 4 years ago

Solução simples e top :)

Algumas mudanças sugeridas apenas:

  • Você deixou as coisas do banco de dados comentados no settings (shame virtualenv)
  • Não precisa dessas linhas em branco entre as rotas no src/acacia/urls (inclusive diminuiu a cobertura de testes por conta delas.

Corrigido! :)

vitorcx commented 4 years ago

image

@durvalcarvalho @fabiolamfleury Qual é o impacto desses warnings nesse pr?

durvalcarvalho commented 4 years ago

image

@durvalcarvalho @fabiolamfleury Qual é o impacto desses warnings nesse pr?

Nenhum! Esse warnings estão surgindo pq criei 2 rotas com o mesmo "apelido", essas duas rotas são a versão com e sem a barra '/' no final da url...

Como ambas redirecionam para a mesma view, não irá gerar nenhum problema :)