Although the initial bug report mentions showing both ancestors and descendants, I think the feeling at Sitka is that it's not necessary to see the descendant shelving locations, especially since that seems to be how the shelving location selector works elsewhere in the client. Limiting to the selected org unit and its ancestors is the important part. Do we need to show descendants, or can the fix be committed as-is?
Although the initial bug report mentions showing both ancestors and descendants, I think the feeling at Sitka is that it's not necessary to see the descendant shelving locations, especially since that seems to be how the shelving location selector works elsewhere in the client. Limiting to the selected org unit and its ancestors is the important part. Do we need to show descendants, or can the fix be committed as-is?