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

Atualizar perfil de usuário #122

Closed LeoSilvaGomes-zz closed 4 years ago

LeoSilvaGomes-zz commented 5 years ago

Descrição

Criação de endpoint para enviar e atualizar os dados do perfil do usuário

Resolve (Issues)

Issue [#73]()

PRs relacionados, se houver

branch	PR
feature/73-update-user	link

Tarefas gerais realizadas

Criação do endpoint que provê as informações do perfil do usuário autenticado;
Criação da view para retornar e atualizar dados do perfil

codecov-io commented 5 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@d6bbc21). Click here to learn what that means. The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop     #122   +/-   ##
==========================================
  Coverage           ?   93.64%           
==========================================
  Files              ?       32           
  Lines              ?      991           
  Branches           ?        0           
==========================================
  Hits               ?      928           
  Misses             ?       63           
  Partials           ?        0

Flag	Coverage Δ
#unittests	`93.64% <ø> (?)`

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 d6bbc21...5c0cd8a. Read the comment docs.

durvalcarvalho commented 4 years ago

Foi criado testes para as novas funcionalidades, e estamos com um cobertura de 96% no app users.

durvalcarvalho commented 4 years ago

Pessoal, as tarefas ditas realizadas nesse pr não são as únicas, tem bastante coisa acontecendo nele que não tá documentado, @durvalcarvalho @RenatoBrittoAraujo vocês podem me ajudar a entender o que tá acontocendo nesse pr?

Eu não sei explicar o motivo dessa união de commits de diversas branches...

Em algum momento foi necessário 'pegar' commits de outras branches em andamento e acabou resultando nessa bagunça toda...

fabiolamfleury commented 4 years ago

Beleza, por segurança eu vou deixar esse pr "bloqueado", para avaliarmos esses commits em suas branches de origem e termos o minimo de problemas o possível

fabiolamfleury commented 4 years ago

Outra observação: a mudança da rota de harvest_months pra months quebra a criação de árvores no frontend

durvalcarvalho commented 4 years ago

Outra observação: a mudança da rota de harvest_months pra months quebra a criação de árvores no frontend

Eu devo consertar isso nessa branch (#73 Atualizar perfil de usuário) ou na branch referente as propriedades (#120 Visualização de Propriedades) ? Ou em ambas?

durvalcarvalho commented 4 years ago

Outra observação: a mudança da rota de harvest_months pra months quebra a criação de árvores no frontend

Eu devo consertar isso nessa branch (#73 Atualizar perfil de usuário) ou na branch referente as propriedades (#120 Visualização de Propriedades) ? Ou em ambas?

Eu mudei no frontend, para que se utilize a nova rota renomeada

fabiolamfleury commented 4 years ago

@jps12 vc pode arrumar os conflitos pra gente conseguir mergear essa branch? :)

durvalcarvalho commented 4 years ago

@jps12 vc pode arrumar os conflitos pra gente conseguir mergear essa branch? :)

Done

fabiolamfleury commented 4 years ago

Devido o problema que foi citado anteriormente, de várias branches com commits repetidos, essa issue acabou sendo resolvida em outros prs