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

Landing Page and Immuno conclusion dropdown #1245

Merged
merged 12 commits into from
Aug 27, 2024

Conversation

pascagihozo
Copy link
Contributor

Pull Requests Requirements

  • [ x] The PR title includes a brief description of the work done, including the
    Issue number if applicable.
  • [ x] The PR includes a video showing the changes for the work done.
  • The PR title follows conventional commit label standards.
  • [ x] The changes confirm to the OpenElis Global x3 Styleguide and design
    documentation.
  • The changes include tests or are validated by existing tests.
  • [ x] I have read and agree to the Contributing Guidelines of this project.

Summary

This pull request includes the landing page that allows the user to select the lab units before they access the common dashboard and it adds a drop down on Immuno-chemistry conclusion

Screenshots

Screenshot from 2024-08-22 18-20-00
Screenshot from 2024-08-22 18-00-16

Screencast.from.2024-08-22.18-21-01.webm

Related Issue

-User expressed a need to select department immediately after they log into the system
-At Immunochemistry lab they require ICD code which should be uniform to pathology conclusion and Immunochemistry conclusion report

Further work

Features are currently implemented to the frontend and they are yet to be implemented with the backend logic as well

@mozzy11 @caseyi

frontend/src/App.js Outdated Show resolved Hide resolved
Copy link
Contributor Author

@pascagihozo pascagihozo left a comment

Choose a reason for hiding this comment

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

Please look into it and let me know what you think can be a resolution!

frontend/src/App.js Outdated Show resolved Hide resolved
@pascagihozo pascagihozo requested a review from mozzy11 August 22, 2024 16:14
@mozzy11 mozzy11 merged commit 037e1bd into I-TECH-UW:develop Aug 27, 2024
4 checks passed
This was referenced Aug 27, 2024
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