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

Starred obs #1224

Open
wants to merge 22 commits into
base: v0.98
Choose a base branch
from
Open

Starred obs #1224

wants to merge 22 commits into from

Conversation

fredkingham
Copy link
Contributor

No description provided.

fredkingham and others added 19 commits September 28, 2021 09:22
Adds the starred observation model
6493948
Adds the star observation api
5a55ae7
Adds in the front end to star observations in the result view
621a129
Adds in the star in the results view observation serialization
3df995f
Fixes an issue in the starred observation where the star would not va…
f97f09e 
…nish immediately after the mouse over
Changes the api lab test api to return the id of the Observation star…
150b1f9 
… if its starred, totherwise it returns None. If a starred observation is not already returned then its added to the ticker
Adds in the front end for the panel so that the user can star and uns…
dbc92d2 
…tar observations
Disable the caching of the infection service page and the lab test lo…
03f2a08 
…aders
Adds in the show hide logic on the stars on the infection summary page
ec496a2
Fixes up the css around the display and hiding of starred results
7ac9978
Removes the stars from the summary panel but keep the starred observa…
a401a67 
…tion in it
Removes the star from the tabular view
8ca7d2b
Moves the star to the left of the long form result in the results view
5a45c40
If we have an extra long observation in the lab test summary panel th…
012e315 
…en put the value on the row below
Remove the ng-if around the results page, but make sure we only load …
a97a3a5 
…the results once
@davidmiller
Fixup: Colors
45f3662
Remove fields that we no longer need to translate in the infection su…
06b426f 
…mmary api as we have disabled the ability to unstar from this panel
Merge branch 'starred-obs' of github.com:openhealthcare/elcid-rfh int…
8914d57 
…o starred-obs
Remove fields that we no longer need to translate in the covid lab as…
58f941f 
… we have disabled the ability to unstar from this panel
@fredkingham fredkingham changed the base branch from v0.95 to v0.96 October 15, 2021 09:48
fredkingham added 2 commits November 1, 2021 16:24
We had changes implemented that allowed you to 'unstar' results in th…
5a89f7f 
…e episode category page. The consequence of this is that we had to uncache the results view which is costly in performance. It also meant we had to uncache the episode detail page. This behaviour would break on any other page that wanted to reuse the results panel. This change means that we cache everything apart from the panel that we reload when the view changes
Changes the covid lab ticker to return the test name and the lab number
599bc82
@fredkingham fredkingham changed the base branch from v0.96 to v0.98 November 3, 2021 21:37
also load data if the view changes as going to test results and back …
5b5a091 
…again does not necessarily change the episode.category_name
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@fredkingham @davidmiller