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

Feature/labs column - Adds lab_unit column in routine csv #1381

Closed
wants to merge 20 commits into from

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.
  • The PR includes a video showing the changes for the work done.
  • [ x] The PR title follows conventional commit label standards.
  • The changes confirm to the OpenElis Global x3 Styleguide and design
    documentation.
  • The changes include tests or are validated by existing tests.
  • I have read and agree to the Contributing Guidelines of this project.

Summary

  1. Added the lab_unit column definition
  2. Created the proper SQL subquery to get the test section name

Screenshots

image

[Add relevant screenshots here if applicable]

Related Issue

Missing Lab Unit in Routine CSV File:

[Add a link to the related issue or mention it here if applicable]

Other

[Add any additional information or notes here]

@github-actions github-actions bot added the merge conflict Merge Conflicts label Jan 9, 2025
Copy link

github-actions bot commented Jan 9, 2025

👋 Hi, @pascagihozo,
Conflicts have been detected against the base branch. Please rebase your branch against the base branch.


This message is automatically generated by prince-chrismc/label-merge-conflicts-action so don't hesitate to report issues/improvements there.

@mozzy11
Copy link
Collaborator

mozzy11 commented Jan 9, 2025

Changes implemented here #1383

@mozzy11 mozzy11 closed this Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge conflict Merge Conflicts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants