Skip to content
This repository has been archived by the owner on Jun 27, 2019. It is now read-only.

Max#mensagens nao lidas #115

Merged
merged 19 commits into from
May 14, 2019
Merged

Max#mensagens nao lidas #115

merged 19 commits into from
May 14, 2019

Conversation

Mex978
Copy link
Collaborator

@Mex978 Mex978 commented May 11, 2019

Issues que foram resolvidas: #99 e #116

O que foi feito

  • Alterações na tela de conversas para que possa recuperar as informações de:
    • Número de mensagens não lidas
    • Última mensagem enviada com data adaptativa
      • Caso a mensagem tenha sido enviada no mesmo dia de visualização, o que será mostrado é o horário
      • Caso tenha sido enviada no dia anterior, será mostrado "Ontem"
      • Caso tenha sido enviada antes disso, a data com dia/mês/ano será mostrada
  • Alterações no Chat.js para que, quando um usuário mande uma mensagem, o documento conversa do usuário destino terá os seguintes campos atualizados:
    • unreadMsgs (bool): Indica se tem mensagens não lidas (true) ou não (false)
    • numUnreadMsgs (num): Indica o número de mensagens não lidas
    • lastMessage (string): Conteúdo da última mensagem enviada na conversa
    • dateLastMessage (timestamp): Data da última mensagem enviada na conversa
  • Além disso, ainda no Chat.js, quando um usuário manda uma mensagem, no seu proprio documento daquela conversa, os campos lastMessage e dateLastMessage serão atualizados

@Mex978 Mex978 added needs review pull request needs a review PR: unreviewed labels May 11, 2019
@Mex978 Mex978 requested review from netochaves and sosolidkk May 11, 2019 17:17
Copy link
Collaborator

@sosolidkk sosolidkk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bugs

  1. Ao se apagar uma conversa, ocorre um warning referente a um documento não encontrado. Ele aparece diversas vezes podendo ocasionar crash no app.
  2. Ao se entrar em uma conversa que possui mensagens não lidas e voltar para tela de conversas, se o usuário recebe uma nova mensagem, o contador de mensagens não lidas não aparece.
  3. Vez ou outra pode ocorrer um erro com uma função toDate(), não sei a causa exata e os passos pra reprodução, porém o warning do item número 1 pode ocasionar esse error.

@netochaves netochaves self-requested a review May 14, 2019 16:06
@sosolidkk sosolidkk self-requested a review May 14, 2019 17:23
Copy link
Collaborator

@sosolidkk sosolidkk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Testada por @netochaves @sosolidkk .
Aprovada por @sosolidkk @netochaves .

@sosolidkk sosolidkk removed the needs review pull request needs a review label May 14, 2019
@sosolidkk sosolidkk merged commit 43b78f9 into dev May 14, 2019
@netochaves netochaves deleted the max#mensagensNaoLidas branch May 14, 2019 20:34
@netochaves netochaves removed needs review pull request needs a review labels May 22, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants