FHIR Chat · fhircast-docs / Issue #69 Close events don't contain context · fhircast-github

Stream: fhircast-github

Topic: fhircast-docs / Issue #69 Close events don't contain context


view this post on Zulip Github Notifications (FHIRcast) (Feb 21 2019 at 09:11):

bvdh commented on Issue #69

This also makes sense from another perspective. There is an issue if two applications want the close the chart at the same time. This will result in closing two charts. The current and the next one.
So yes, the context should be included.

view this post on Zulip Github Notifications (FHIRcast) (Apr 26 2019 at 18:44):

isaacvetter commented on Issue #69

Fixed in #76 . Closing.

view this post on Zulip Github Notifications (FHIRcast) (Apr 26 2019 at 18:44):

isaacvetter closed Issue #69(assigned to isaacvetter)

The closing events currently don't have any context specified, but it seems useful to know the context of what specifically is being closed. This would be extremely useful in the case of unidirectional integration syncing. For example a subscribing app receives the notifications for an open study and puts that in focus. They are not implementing bi-directional syncing and the user then opens a different study in the subscribing application. The driving application then closes the original study. The current spec would dictate that the subscribing app close the current study or both studies, either of which I don't think is necessarily what we want. If they only close the in focus study (study B) then the original study is still open in the subscribing application even though it was closed in the driving application which seems clearly wrong. If they close both then that could annoy the user who wanted to keep in focus that manually opened study.

This has caused some issues with current integrations where some systems don't pay attention to the context of the close and will instead just close everything or the current thing which can cause some weird behavior.


Last updated: Apr 12 2022 at 19:14 UTC