Display Chat between users

MrinalVarshney commented 1 year ago

Description

Some bug fixes in backend for retrieving messages
Added profile Image field on necessary places , so that it can be shown while chat is going on
During register and login no userId was being sent , which is giving error during display messages, So i have included userId in the the response json from backend while login and signup activities.

Related Issue

No live chat is displayed earlier while users are online.

Fixes # (issue)

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Motivation and Context

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Screenshots (if appropriate):

screen-capture (4).webm

Checklist:

[x] I have registered myself at Contrihub website.
[x] My code follows the code style of this project.
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] Any dependent changes have been merged and published in downstream modules
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

MrinalVarshney commented 1 year ago

Moreover some styling and animations are left , will do it asap !

MrinalVarshney commented 1 year ago

UI interface modified , screen-capture (5).webm

silkie261001 commented 1 year ago

can you decrease that lag between pressing button and displaying of message

MrinalVarshney commented 1 year ago

Shall I display something like "loading messages..." till they are not fetched ?

silkie261001 commented 1 year ago

Yes, you can do that.

On Thu, Oct 19, 2023, 9:43 PM Mrinal @.***> wrote:

Shall I display something like "loading messages..." till they are not fetched ?

— Reply to this email directly, view it on GitHub https://github.com/ContriHUB/PlayTopia/pull/15#issuecomment-1771308975, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASRKYIZN6PO6KC2K47FHFQTYAFGSTAVCNFSM6AAAAAA6DF4VYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZRGMYDQOJXGU . You are receiving this because you commented.Message ID: @.***>

MrinalVarshney commented 1 year ago

ok mam

Yes, you can do that. … On Thu, Oct 19, 2023, 9:43 PM Mrinal @.> wrote: Shall I display something like "loading messages..." till they are not fetched ? — Reply to this email directly, view it on GitHub <#15 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASRKYIZN6PO6KC2K47FHFQTYAFGSTAVCNFSM6AAAAAA6DF4VYWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZRGMYDQOJXGU . You are receiving this because you commented.Message ID: @.>

MrinalVarshney commented 1 year ago

It's done ! screen-capture (6).webm

ContriHUB / PlayTopia