Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature:Sending-media-via-messaging implementation #146

Merged
merged 2 commits into from
Jul 22, 2024

Conversation

Emmyfrank
Copy link
Contributor

@Emmyfrank Emmyfrank commented Jul 16, 2024

Discription

This implementation enables customers to communicate with a doctor appointment with media services, chatting with a doctor by using media image and voice, audio record and messaging
1: If you don't have an account 1st register ,if you aleady have an account navigate too
2: Book appointment or navigate to appointment if you have already booked and choose messaging appointment.
3: From there you can user text message , image,audio, and record, as a patient if your session has already started , if your session is not started they will show you a message to to describe that your session not yet started , and if your swssion has already reached on the time you scheduled , you can start chating with your doctor, if time has ended they will also show you that the session has ended .
4: the message or audio and record you sent or what you received will be different according to their color and possitions.
5:If you want to test reply , you can go in database and open messages table, check the spesfic message that you sent, and click on appointment Id, and after copy the appointment id of your message , insert new row and follow the screenshoot below to fill it after save the message will be sent to your emirator as reply.
Screenshot 2024-07-17 at 12 20 31

Screenshot 2024-07-17 at 12 20 47

screenshoot

Screenshot 2024-07-17 at 00 12 42

@Emmyfrank Emmyfrank changed the title Feature:Sending audio,image,recode and ending session when it is necess… Feature:Sending-media-via-messaging Jul 17, 2024
@Emmyfrank Emmyfrank self-assigned this Jul 17, 2024
@Emmyfrank Emmyfrank added the Feature This is a project feature label Jul 17, 2024
@Emmyfrank Emmyfrank changed the title Feature:Sending-media-via-messaging Feature:Sending-media-via-messaging implementation Jul 17, 2024
@Emmyfrank Emmyfrank added Chore Define what a chore is Need Review labels Jul 17, 2024
@fredshema
Copy link
Contributor

@Emmyfrank try to rename the pr and add description on how we can test the feature. I tried to go through the feature and I noticed that messages are not differentiated. Both the doctor and patient messages looks the same and you can't tell who sent which message. I saw that messages are being auto updated that is nice. Also i don't think you noticed that you created 3 similar issues for this feature so check on which you can close

Copy link
Contributor

@911samuel 911samuel left a comment

Choose a reason for hiding this comment

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

Screenshot from 2024-07-17 11-35-35

Great work!!!

@Emmyfrank
Copy link
Contributor Author

@Emmyfrank try to rename the pr and add description on how we can test the feature. I tried to go through the feature and I noticed that messages are not differentiated. Both the doctor and patient messages looks the same and you can't tell who sent which message. I saw that messages are being auto updated that is nice. Also i don't think you noticed that you created 3 similar issues for this feature so check on which you can close

thanks let me work on it

@Emmyfrank
Copy link
Contributor Author

@Emmyfrank try to rename the pr and add description on how we can test the feature. I tried to go through the feature and I noticed that messages are not differentiated. Both the doctor and patient messages looks the same and you can't tell who sent which message. I saw that messages are being auto updated that is nice. Also i don't think you noticed that you created 3 similar issues for this feature so check on which you can close

thanks let me work on it

check again

1 similar comment
@Emmyfrank
Copy link
Contributor Author

@Emmyfrank try to rename the pr and add description on how we can test the feature. I tried to go through the feature and I noticed that messages are not differentiated. Both the doctor and patient messages looks the same and you can't tell who sent which message. I saw that messages are being auto updated that is nice. Also i don't think you noticed that you created 3 similar issues for this feature so check on which you can close

thanks let me work on it

check again

@justinmihigo justinmihigo self-requested a review July 17, 2024 10:52
Copy link
Contributor

@justinmihigo justinmihigo left a comment

Choose a reason for hiding this comment

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

This looks great

@Emmyfrank Emmyfrank added need aprove Improvements or additions to documentation and removed Need Review labels Jul 18, 2024
@Emmyfrank Emmyfrank force-pushed the ft-sending-media-via-messaging branch from 54b60be to 5dea845 Compare July 22, 2024 09:44
@Adadie
Copy link
Collaborator

Adadie commented Jul 22, 2024

@Emmanuel262 please add loading states for when the image is being loaded

@Adadie Adadie merged commit c09728f into dev Jul 22, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Chore Define what a chore is Feature This is a project feature need aprove Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants