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

Add labs and tests to VAHB #10405

Draft
wants to merge 52 commits into
base: develop
Choose a base branch
from
Draft

Add labs and tests to VAHB #10405

wants to merge 52 commits into from

Conversation

stephenBarrs
Copy link
Contributor

Draft - Do Not Merge

Description of Change

Screenshots/Video

Testing

  • Tested on iOS
  • Tested on Android

Reviewer Validations

PR Checklist

Reviewer: Confirm the items below as you review

  • PR is connected to issue(s)
  • Tests are included to cover this change (when possible)
  • No magic strings (All string unions follow the Union -> Constant type pattern)
  • No secrets or API keys are checked in
  • All imports are absolute (no relative imports)
  • New functions and Redux work have proper TSDoc annotations

For QA

Run a build for this branch

stephenBarrs and others added 30 commits November 22, 2024 16:43
@stephenBarrs
Add allergy management features including data fetching and UI compon…
e4832b5 
…ents
@stephenBarrs
Add allergy management features including data fetching and UI compon…
e2a96bc 
…ents
@stephenBarrs
Refactor allergy-related components and types, remove unused allergy …
9949ba5 
…location API, and update error handling for allergy screens
@stephenBarrs
Merge branch 'develop' into feature/addAllergiesList
179d962
@stephenBarrs
wip - add medical record structure and stub out labs and tests
610091d
@stephenBarrs
fix linter warnings and create link to va.gov MR in script
dfb8e46
@stephenBarrs
initial mock data and type def for labs and tests
7d9cc60
@stephenBarrs
Implement basic infrastructure for Labs and Tests. List page implemen…
dfb7a27 
…ted but not Detail page. Test files exist but need to be implemented
@mdewey
(feature) added navigation to the health screen details
2907e21
@acrollet
Clean up warnings
3322f5f
@acrollet
Merge branch 'feature/addAllergiesList' of https://github.com/departm…
5057e11 
…ent-of-veterans-affairs/va-mobile-app into feature/addAllergiesList
@acrollet
Merge branch 'develop' into feature/addAllergiesList
da334c6
@acrollet
Remove unused button and commit lock changes
680cfef
@acrollet
Update VAMobile/src/screens/HealthScreen/Allergies/AllergyList/Allerg…
6468a91 
…yListScreen.tsx
@acrollet
Update VAMobile/src/screens/HealthScreen/HealthScreen.tsx
f19b09b
@acrollet
Update VAMobile/src/translations/en/common.json
3069204
@acrollet
Update VAMobile/src/translations/en/common.json
c039832
@acrollet
Update VAMobile/src/translations/en/common.json
0e4dd7e
@acrollet
Update VAMobile/src/translations/en/common.json
2026785
@mdewey
(feature) added analytics for details and list screen
9b7c6fe
@acrollet
Fix linting error
369ce92
@acrollet
Fix Vaccines test
9d065b9
@mdewey
(feature) add support for allergy-related app level errors
f561e1f
@acrollet
Add allergies e2e test
a4eb16f
@acrollet
Add unit tests for MedicalRecordsScreen functionality
2ecfb2f
@acrollet
Add error handling test for AllergyListScreen
599fe41
@acrollet
Merge branch 'feature/addAllergiesList' of https://github.com/departm…
95f9321 
…ent-of-veterans-affairs/va-mobile-app into feature/addAllergiesList
@acrollet
Update HealthScreen tests to reflect changes in medical records termi…
0ca3e39 
…nology
@acrollet
Fix extraneous import
2952465
@acrollet
Update detox mapping
5e27815
acrollet and others added 22 commits February 5, 2025 16:45
@acrollet
Merge branch 'develop' into feature/addAllergiesList
bfa79b8
@acrollet
Update HomeScreen test to check for Medical Records button instead of…
1a933a9 
… Vaccines button
@acrollet
Update navigation test
525d563
@acrollet
Merge branch 'feature/addAllergiesList' of https://github.com/departm…
484f70f 
…ent-of-veterans-affairs/va-mobile-app into feature/addAllergiesList
@mdewey
(feature) implement Labs and Tests details screen and add mock data
26576af
@mdewey
Add mock data for Labs and Tests diagnostic reports
3a2fcf0
@mdewey
(feature) added skeleton for datepicker
a9d2751
@mdewey
(feature) add date range filtering for Labs and Tests API and update …
1220732 
…UI options
@mdewey
(feature) add TimeFrameType and constants for Labs and Tests filtering
abcd151
@stephenBarrs
merge MR section and allergies work
f7e5706
@mdewey
(test) add end-to-end tests for Labs and Tests date range selection
b41cb31
@stephenBarrs
update Labs and Tests data structure and filtering logic, begin secti…
b403db9 
…oning test data and mock infrastructure to support date ranges
@stephenBarrs
update testID for Labs and Tests navigation and confirm button
553a217
@stephenBarrs
Merge branch 'develop' into feature/labsAndTests
1cb7941
@stephenBarrs
Merge remote-tracking branch 'origin/feature/labsAndTests' into featu…
1cf9995 
…re/labsAndTests
@stephenBarrs
refactor: simplify Labs and Tests data structure and update date rang…
5eef75b 
…e logic
@stephenBarrs
enhance Labs and Tests test data to reflect actual time ranges expect…
bc2fcd9 
…ed from API response
@stephenBarrs
add labTypes parameter to Labs and Tests API and query hook
72791a1
@stephenBarrs
Merge branch 'develop' into feature/labsAndTests
572787a
@mdewey
Refactor LabsAndTestsDetailsScreen tests and add base64 decoding for …
8f34c3a 
…report data
@stephenBarrs
Remove labTypes parameter from getLabsAndTests function and update us…
a104131 
…eLabsAndTests queryKey
@stephenBarrs
add tests for Labs and Tests, update component for no records found
a38eb47
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.
3 participants
@stephenBarrs @mdewey @acrollet