Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Issue 627: Fix visit types list not refreshing. #645

Merged
merged 1 commit into from
Sep 12, 2016

Conversation

akong
Copy link
Contributor

@akong akong commented Sep 11, 2016

Fixes #627

Changes proposed in this pull request:

  • Fixes the typo that was causing the visitTypesList to not be computed properly.
  • Add volatile to the dependent property of visitTypesList (see app/mixins/visit-types.js). It looks like being dependent on a computer property doesn't quite work as any subsequent updates to the underlying data is not firing the observers (i.e. the mixin functions).

cc @HospitalRun/core-maintainers

@jkleinsc
Copy link
Member

@akong thanks for the PR! Looks good to me. I'll merge it in.

@jkleinsc jkleinsc merged commit caea7a1 into HospitalRun:master Sep 12, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants