-
-
Notifications
You must be signed in to change notification settings - Fork 73
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
Display patient age instead of hard coded '58' Fixes #104 #138
Conversation
Thanks for the PR 🥳 A large chunk of the current dashboard will actually be removed, and replaced with user configurable widgets - see #21 However that work is still in early stages, so there's no downside in making this fix for now |
frontend/src/app/pages/source-detail/source-detail.component.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some error handling required.
Apologies. Fixed and did some testing in an updated jsfiddle. Also handles strings that look like dates. |
Sorry about the delay merging this PR, I was a bit busy with the |
No description provided.