Skip to content

(feat) O3-5578: Validate Queue Location tags in QueueValidator#107

Open
LxndrKrchvtsv wants to merge 2 commits intoopenmrs:mainfrom
LxndrKrchvtsv:improvement/O3-5578-queue-location-validation
Open

(feat) O3-5578: Validate Queue Location tags in QueueValidator#107
LxndrKrchvtsv wants to merge 2 commits intoopenmrs:mainfrom
LxndrKrchvtsv:improvement/O3-5578-queue-location-validation

Conversation

@LxndrKrchvtsv
Copy link
Copy Markdown

Requirements

  • This PR has a title that briefly describes the work done including
    the ticket number (feat) O3-5578 with conventional commit label.
  • My work is based on the existing validation patterns used throughout the queue module.
  • My work includes tests or is validated by existing tests.

Summary

Found a TODO: Check if the location is tagged as a Queue Location? - Then created new task and implemented it.

  • Preserved the fundamental null check for the location object.
  • Leveraged Context.getLocationService() to retrieve the "Queue Location" tag.
  • Added logic to verify that the location's assigned tags specifically include the "Queue Location" tag.
  • Covered the task with tests

Screenshots

N/A

Related Issue

https://openmrs.atlassian.net/browse/O3-5578

Other

N/A

@LxndrKrchvtsv LxndrKrchvtsv changed the title O3-5578: Validated Queue Location tags in QueueValidator (feat) O3-5578: Validated Queue Location tags in QueueValidator Mar 31, 2026
@LxndrKrchvtsv LxndrKrchvtsv changed the title (feat) O3-5578: Validated Queue Location tags in QueueValidator (feat) O3-5578: Validate Queue Location tags in QueueValidator Mar 31, 2026
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.

1 participant