FE/IN Feature add messaging - Githubissues

hatchways / team-scallops

1 stars 1 forks source link

FE/IN Feature add messaging #82

Closed navjot-chahal closed 3 years ago

navjot-chahal commented 3 years ago

What this PR does:

Added messaging page.
Modified previously made BE messaging controller to match auth controller.
Integrated the with the BE.

Screenshots / Videos:

Messaging example when test user is logged-in.

test

Messaging example when test2 user is logged-in.

test 2

Any information needed to test this feature:

Simply run the Express server with npm run dev and populate the database with user and conversation manually.

Any issues with the current functionality (optional):

The online user functionality is hardcoded now.

closes #38

navjot-chahal commented 3 years ago

Thanks for your comments @ezeilo-su. I have addressed your concerns in Hotfix - 1.

navjot-chahal commented 3 years ago

@rajivtitus Thanks for your detailed review of the PR. See Hotfix - 2 for the changes requested.

navjot-chahal commented 3 years ago

@rajivtitus @ugur-akin Guys, If everything is alright here, can you please approve this PR. 😀