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.

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

Medications incorrectly labeled when the patient is not taking them #156

Closed
1 task done
kylieo123 opened this issue Jan 6, 2025 · 1 comment · Fixed by #176
Closed
1 task done

Medications incorrectly labeled when the patient is not taking them #156

kylieo123 opened this issue Jan 6, 2025 · 1 comment · Fixed by #176
Assignees
Labels
bug An unexpected problem ENGAGE HF ENGAGE-HF-specific issues

Comments

@kylieo123
Copy link

Description

When a patient is first enrolled or if they have no medications entered, it automatically says they are taking 3 medications (Carvedilol, Empagliflozin, Entresto). They have the current dose listed as "N/A" and the color is Yellow and in the "Medications you are taking" section.

Reproduction

For any patient that was either just enrolled or has been enrolled. Delete all of their saved current medications or any one medication in the beta blockers, SGLT2i, or ACE/ARB/ARNI sections.

Expected behavior

Change this so that there are no medications that are automatically assumed to be taking and they are all in the "Medications that may help" section with the appropriate color code.

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct and Contributing Guidelines
@kylieo123 kylieo123 added the bug An unexpected problem label Jan 6, 2025
@PSchmiedmayer
Copy link
Member

@eldcn @pauljohanneskraft this looks like a syncing issue of the medications with the mobile app? It seems like these are mock values that are persisting?

@PSchmiedmayer PSchmiedmayer added the ENGAGE HF ENGAGE-HF-specific issues label Jan 7, 2025
@eldcn eldcn self-assigned this Jan 21, 2025
eldcn added a commit that referenced this issue Jan 28, 2025
# Currently taking and medications that might help sections

## ♻️ Current situation & Problem
- Adapted according to iOS implementation:
https://github.com/StanfordBDHG/ENGAGE-HF-iOS/blob/main/ENGAGEHF/Medications/MedicationsList.swift#L38
- Fixes #156 


## ⚙️ Release Notes 
*Add a bullet point list summary of the feature and possible migration
guides if this is a breaking change so this section can be added to the
release notes.*
*Include code snippets that provide examples of the feature implemented
or links to the documentation if it appends or changes the public
interface.*


## 📚 Documentation
*Please ensure that you properly document any additions in conformance
to [Spezi Documentation
Guide](https://github.com/StanfordSpezi/.github/blob/main/DOCUMENTATIONGUIDE.md).*
*You can use this section to describe your solution, but we encourage
contributors to document your reasoning and changes using in-line
documentation.*


## ✅ Testing
*Please ensure that the PR meets the testing requirements set by CodeCov
and that new functionality is appropriately tested.*
*This section describes important information about the tests and why
some elements might not be testable.*


## 📝 Code of Conduct & Contributing Guidelines 

By submitting creating this pull request, you agree to follow our [Code
of
Conduct](https://github.com/StanfordSpezi/.github/blob/main/CODE_OF_CONDUCT.md)
and [Contributing
Guidelines](https://github.com/StanfordSpezi/.github/blob/main/CONTRIBUTING.md):
- [ ] I agree to follow the [Code of
Conduct](https://github.com/StanfordSpezi/.github/blob/main/CODE_OF_CONDUCT.md)
and [Contributing
Guidelines](https://github.com/StanfordSpezi/.github/blob/main/CONTRIBUTING.md).
@github-project-automation github-project-automation bot moved this from Backlog to Done in Project Planning Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An unexpected problem ENGAGE HF ENGAGE-HF-specific issues
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants