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.

Sign up for GitHub

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

Jump to bottom

Dashboard fixes #140

Merged
merged 2 commits into from
Oct 30, 2024
Merged

Dashboard fixes #140

merged 2 commits into from
Oct 30, 2024

Conversation

rubenpeters91
Copy link
Contributor

Fixed

  • Removed TTL in patient list in calling dashboard and also show patient that were called today to prevent patients from disappearing during calls
  • Corrected database column types so databases are identical over different environments

Added

  • Added Alembic for database migrations
  • Added a timestamp for call response, so we can track when patients were called

Closes: #139
Closes: #138

See acc dashboard: https://rsc.ds.umcutrecht.nl/content/a4f403a9-8d43-4757-8caf-c78fb0669dfc

@rubenpeters91 rubenpeters91 added bug Something isn't working enhancement New feature or request labels Oct 30, 2024
@rubenpeters91 rubenpeters91 self-assigned this Oct 30, 2024
WoltersEric
WoltersEric approved these changes Oct 30, 2024
View reviewed changes
Copy link
Contributor

@WoltersEric WoltersEric left a comment

Choose a reason for hiding this comment

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

LGTM

@rubenpeters91 rubenpeters91 merged commit 6adea35 into main Oct 30, 2024
3 checks passed
@rubenpeters91 rubenpeters91 deleted the dashboard_fixes branch October 30, 2024 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@WoltersEric WoltersEric WoltersEric approved these changes

Assignees

@rubenpeters91 rubenpeters91

Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Fix dashboard changing patient list while calling Add timestamp of call_response to database
2 participants
@rubenpeters91 @WoltersEric