search

SharebookBR / sharebook-backend

Projeto backend de código livre para o app Sharebook.
117 stars 75 forks source link

Novo endpoint com a lista de interessados. #217

Closed raffacabofrio closed 5 years ago

raffacabofrio commented 5 years ago

Estou mandando essa PR pra dar visibilidade. Por favor não aprovem ainda.

Preciso de ajuda na parte de colocar o apelido. DomainToViewModelMappingProfile.cs >> linha 45

AppVeyorBot commented 5 years ago

:white_check_mark: Build Sharebook-Backend: PR Check 1.0.47-estimwxw completed (commit https://github.com/SharebookBR/sharebook-backend/commit/57e14eb247 by @raffacabofrio)

marcios commented 5 years ago

Neste ponto as seguintes possibilidades:

1 - A cada novo interessado, gravar o apelido na tabela de bookuser (lembro que um tempo atras não achamos necessário essa abordagem mas acredito que agora é a melhor solução)

2 - criar método que recebe o requestUser e faça o calculo para saber em que posição de interesse ele está, este é uma mudança que não requer alteração física no banco mas ao meu ver vai gerar um processamento a mais.

raffacabofrio commented 5 years ago

Obrigado pela ajuda @marcios.

Vou atualizar essa PR indo pelo caminho 1 então. Tmj.

AppVeyorBot commented 5 years ago

:white_check_mark: Build Sharebook-Backend: PR Check 1.0.47-ngeowgfn completed (commit https://github.com/SharebookBR/sharebook-backend/commit/d4c1245ba3 by @raffacabofrio)

raffacabofrio commented 5 years ago

@marcios @bhenn

Adicionei dois commits pra persistir o campo NickName. Agora é hora de dar uma voadora no peito do code review. Bora!

Caso concordem, podem aprovar e mesclar, pfv.

AppVeyorBot commented 5 years ago

:x: Build Sharebook-Backend: PR Check 1.0.48-oeqpvolm failed (commit https://github.com/SharebookBR/sharebook-backend/commit/5420b3c8f4 by @raffacabofrio)

AppVeyorBot commented 5 years ago

:white_check_mark: Build Sharebook-Backend: PR Check 1.0.48-xmnljcbl completed (commit https://github.com/SharebookBR/sharebook-backend/commit/ad3807db3c by @raffacabofrio)

wantero commented 5 years ago

Caso mantenha a retrocompatibilidade do endpoint que existia, eu aprovo :) Acho arriscado aprovar e subir quebrando o que temos.

AppVeyorBot commented 5 years ago

:white_check_mark: Build Sharebook-Backend: PR Check 1.0.48-wmboyciy completed (commit https://github.com/SharebookBR/sharebook-backend/commit/87a795ae41 by @raffacabofrio)

AppVeyorBot commented 5 years ago

:white_check_mark: Build Sharebook-Backend: PR Check 1.0.48-ooiakqrc completed (commit https://github.com/SharebookBR/sharebook-backend/commit/7b4393c386 by @raffacabofrio)