Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

CNDIT 1597+1618: Investigation stored procedure update for PB multi-select #39

Merged
merged 4 commits into from
Sep 24, 2024
Merged

CNDIT 1597+1618: Investigation stored procedure update for PB multi-select #39

merged 4 commits into from
Sep 24, 2024

Conversation

upasanapattnaik-eq
Copy link
Collaborator

@upasanapattnaik-eq upasanapattnaik-eq commented Sep 23, 2024 •
edited
Loading

  • Fix for page builder multi-select with OTH specification: Join with SRTE code_value_general to query on the correct investigation form code.
  • Removal of session table for runtime improvement: Notes
  • Fix for NBS_Act_Entity grouping: Min for add time, max for last_chg_time to avoid pulling all entity records.

CNDIT-1597
CNDIT-1618

@upasanapattnaik-eq upasanapattnaik-eq marked this pull request as ready for review September 23, 2024 15:03
sveselev
sveselev approved these changes Sep 24, 2024
View reviewed changes
Copy link
Collaborator

@sveselev sveselev left a comment

Choose a reason for hiding this comment

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

LGTM!

@upasanapattnaik-eq upasanapattnaik-eq merged commit e4a4701 into main Sep 24, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@sveselev sveselev sveselev approved these changes

@rshanmugam-enquizit rshanmugam-enquizit Awaiting requested review from rshanmugam-enquizit

@johneipe johneipe Awaiting requested review from johneipe

@joshua-mills-cdw joshua-mills-cdw Awaiting requested review from joshua-mills-cdw

@alog-enquizit alog-enquizit Awaiting requested review from alog-enquizit

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@upasanapattnaik-eq @sveselev