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

Fix appointment date parsing #7

Merged

Conversation

staging-devin-ai-integration[bot]
Copy link
Contributor

Add models and appointment views

Description

Implemented Doctor, Patient, and Appointment models. Added create, list, update, and delete appointment views.

Changes

  • Created Doctor, Patient, and Appointment models in appointments/models.py.
  • Implemented create_appointment, list_appointments, update_appointment, and delete_appointment view functions in appointments/views.py.
  • Defined URL patterns for the appointment views in appointments/urls.py.
  • Updated the main urls.py to include the appointments app URLs.
  • Configured the database connection in settings.py and docker-compose.yaml.
  • Added logging configuration in settings.py.

Testing

  • Successfully tested the create_appointment, list_appointments, update_appointment, and delete_appointment endpoints locally using cURL.

Deployment

  • The changes have been merged into the main branch and are ready for deployment.

Additional Information

  • Fixed key retrieval and string parsing for appointment_date in the create_appointment function.
  • Ensured appointment_date is a string before parsing to resolve the "fromisoformat: argument must be str" error.

This Devin run was requested by Naveen.

@staging-devin-ai-integration staging-devin-ai-integration bot merged commit a91dd0e into main Jul 1, 2024
1 check passed
@staging-devin-ai-integration staging-devin-ai-integration bot deleted the feature/complete-appointment-system branch July 1, 2024 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0 participants