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

SMART on FHIR pt 3 - frontend changes #458

Merged
merged 23 commits into from
Mar 31, 2025
Merged

Conversation

fzhao99
Copy link
Collaborator

@fzhao99 fzhao99 commented Mar 24, 2025

PULL REQUEST

Summary

Part three of SMART on FHIR implementation that completes the SMART on FHIR work by implementing the frontend and adding an e2e test.

Related Issue

Follow up to #452 and fixes QUE-211

Authentication Enhancements:

This PR propagates the code in the previous backend / infra PR's into the frontend FHIR server modal. Users now have the ability to select / integrate SMART into their FHIR server setup experience

Copy link

linear bot commented Mar 24, 2025

Copy link

github-actions bot commented Mar 24, 2025

Coverage report for ./query-connector

St.
Category Percentage Covered / Total
🔴 Statements
59.41% (-0.05% 🔻)
1720/2895
🔴 Branches
49.25% (-1.47% 🔻)
493/1001
🟡 Functions 62.85% 357/568
🔴 Lines
59.34% (-0.05% 🔻)
1623/2735
Show files with reduced coverage 🔻
St.
File Statements Branches Functions Lines
🔴
... / fhir-servers.ts
31.58% (-0.85% 🔻)
13.21% (-11.79% 🔻)
42.86%
29.58% (-0.86% 🔻)

Test suite run success

159 tests passing in 20 suites.

Report generated by 🧪jest coverage report action from 162ce15

@fzhao99 fzhao99 force-pushed the nickclyde/smart-backend branch 2 times, most recently from 9268950 to 5ec83a0 Compare March 24, 2025 14:48
@fzhao99 fzhao99 force-pushed the nickclyde/smart-workflow-frontend branch from 52ad2e4 to cb1b979 Compare March 24, 2025 14:51
@fzhao99 fzhao99 force-pushed the nickclyde/smart-backend branch 2 times, most recently from 9fce88a to c4ff5ed Compare March 25, 2025 14:42
Base automatically changed from nickclyde/smart-backend to main March 25, 2025 15:08
@fzhao99 fzhao99 force-pushed the nickclyde/smart-workflow-frontend branch from 3356d77 to cd089d6 Compare March 25, 2025 15:08
@fzhao99 fzhao99 marked this pull request as ready for review March 27, 2025 20:53
Copy link
Member

@nickclyde nickclyde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks for shepherding this through Bob!

@fzhao99 fzhao99 merged commit 3737a2f into main Mar 31, 2025
12 checks passed
@fzhao99 fzhao99 deleted the nickclyde/smart-workflow-frontend branch March 31, 2025 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants