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:messaging appointment implementation #135

Merged
merged 2 commits into from
Jul 12, 2024

Conversation

Emmyfrank
Copy link
Contributor

@Emmyfrank Emmyfrank commented Jul 5, 2024

Description

Implement the functionality for making appointments using messaging chat. This includes capturing user details, selecting a doctor, scheduling the appointment, and providing confirmation.

Checklist

  • Appointment Booking via Message Chat
    • Implement the ability for users to book appointments using text messages.
      • Capture user details (name, contact info).
      • Allow selection of doctor and appointment time.
      • Confirm booking and send confirmation message.

Additional Notes

  • Test the appointment booking functionality thoroughly text message.
  • Follow best practices for coding standards and documentation.
  • Ensure data security and privacy, especially for sensitive user information.

screenshoots of what i will be working on.

Screenshot 2024-06-26 at 17 00 11 Screenshot 2024-06-26 at 17 00 32

Assignees

@Emmyfrank Emmyfrank self-assigned this Jul 5, 2024
@Emmyfrank Emmyfrank added the Feature This is a project feature label Jul 5, 2024
@Latiah
Copy link
Contributor

Latiah commented Jul 7, 2024

Screenshot_1720390412
hey @Emmyfrank I reviewed your work I was able to send a message that was great.
what you can modify is this first screen of details so that it matches what is in that appointment like the amount to be paid and the time when the appointment will start on these last buttons also check on the gender and see how you can retrieve the gender of the patient.

@Emmyfrank
Copy link
Contributor Author

Screenshot_1720390412 hey @Emmyfrank I reviewed your work I was able to send a message that was great. what you can modify is this first screen of details so that it matches what is in that appointment like the amount to be paid and the time when the appointment will start on these last buttons also check on the gender and see how you can retrieve the gender of the patient.

thanks for your comment , i have to consider

@atlp-rwanda atlp-rwanda deleted a comment from gitguardian bot Jul 7, 2024
@Emmyfrank Emmyfrank force-pushed the ft-messaging-appointment-implementation-update branch from 7239e38 to 6cb5865 Compare July 8, 2024 08:25
@atlp-rwanda atlp-rwanda deleted a comment from gitguardian bot Jul 8, 2024
@Emmyfrank
Copy link
Contributor Author

Screenshot_1720390412 hey @Emmyfrank I reviewed your work I was able to send a message that was great. what you can modify is this first screen of details so that it matches what is in that appointment like the amount to be paid and the time when the appointment will start on these last buttons also check on the gender and see how you can retrieve the gender of the patient.

worked on gender and other and some details, the messages are working you can approove might be better to have them in today's demo

@Latiah
Copy link
Contributor

Latiah commented Jul 8, 2024

Screenshot_1720390412 hey @Emmyfrank I reviewed your work I was able to send a message that was great. what you can modify is this first screen of details so that it matches what is in that appointment like the amount to be paid and the time when the appointment will start on these last buttons also check on the gender and see how you can retrieve the gender of the patient.

worked on gender and other and some details, the messages are working you can approove might be better to have them in today's demo

yes let me check again real quick

@Latiah
Copy link
Contributor

Latiah commented Jul 8, 2024

Screenshot_1720390412 hey @Emmyfrank I reviewed your work I was able to send a message that was great. what you can modify is this first screen of details so that it matches what is in that appointment like the amount to be paid and the time when the appointment will start on these last buttons also check on the gender and see how you can retrieve the gender of the patient.

worked on gender and other and some details, the messages are working you can approove might be better to have them in today's demo

yeap let me do that

@Latiah
Copy link
Contributor

Latiah commented Jul 8, 2024

.

@Emmyfrank Emmyfrank force-pushed the ft-messaging-appointment-implementation-update branch from 6cb5865 to cf07aa0 Compare July 8, 2024 11:06
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.

You did amazing work!

@atlp-rwanda atlp-rwanda deleted a comment from gitguardian bot Jul 9, 2024
@Emmyfrank Emmyfrank force-pushed the ft-messaging-appointment-implementation-update branch from cf07aa0 to 9cb665e Compare July 10, 2024 17:44
@atlp-rwanda atlp-rwanda deleted a comment from gitguardian bot Jul 10, 2024
@Adadie
Copy link
Collaborator

Adadie commented Jul 12, 2024

Hey @Emmyfrank, great work on the implementation but images and audio seem not to be getting uploaded!

@Emmyfrank
Copy link
Contributor Author

Thanks and the approach that I used still harder to implement that , because I didn't use get stream, for messaging my approach is better that get stream, on audio part and image still harder , that's why I focused on messaging

@Adadie
Copy link
Collaborator

Adadie commented Jul 12, 2024

@Emmyfrank I'm merging your PR but create another one for sending media via messaging

@Adadie Adadie merged commit 3b40a05 into dev Jul 12, 2024
2 checks passed
@Emmyfrank
Copy link
Contributor Author

Thanks I have to work on it .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature This is a project feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants