B2022-031-Add-filtering-for-Proc-and-Section-name-from-Global-Search #537

Merged
jjenko merged 5 commits from B2022-031-Add-filtering-for-Proc-and-Section-name-from-Global-Search into Development 2025-04-07 10:59:47 -04:00
Owner

B2022-031 Add filtering for Proc and Section name from Global Search

B2022-031 Add filtering for Proc and Section name from Global Search
plarsen self-assigned this 2025-04-04 13:16:53 -04:00
djankowski was assigned by plarsen 2025-04-04 13:16:53 -04:00
jjenko was assigned by plarsen 2025-04-04 13:16:53 -04:00
mvickers was assigned by plarsen 2025-04-04 13:16:53 -04:00
plarsen added 4 commits 2025-04-04 13:16:53 -04:00
plarsen requested review from jjenko 2025-04-04 13:17:05 -04:00
jjenko requested changes 2025-04-04 14:18:40 -04:00
Dismissed
jjenko left a comment
Owner

Please add a couple comments with the CSM ticket number.
(ex. B2022-031 - added a {parameter name} to filter out finds in the procedure and section titles)
Add comments to the code in:
PROMSFixes.sql
ItemExt.cs
DisplaySearch.cs

Please add a couple comments with the CSM ticket number. (ex. B2022-031 - added a {parameter name} to filter out finds in the procedure and section titles) Add comments to the code in: PROMSFixes.sql ItemExt.cs DisplaySearch.cs
plarsen added 1 commit 2025-04-07 10:48:26 -04:00
Author
Owner

B2022-031 Add filtering for Proc and Section name from Global Search.
I added the requested comments.

B2022-031 Add filtering for Proc and Section name from Global Search. I added the requested comments.
jjenko approved these changes 2025-04-07 10:59:32 -04:00
jjenko left a comment
Owner

changes look good

changes look good
jjenko merged commit 042b80b9f4 into Development 2025-04-07 10:59:47 -04:00
Member

Tested with version 2.2.2504.711 and 2504.0318 SQL fixes. Works as expected, but would suggest also capitalizing the word "title" in the description of the checkbox so that it's consistent with the way the other options are done.

Tested with version 2.2.2504.711 and 2504.0318 SQL fixes. Works as expected, but would suggest also capitalizing the word "title" in the description of the checkbox so that it's consistent with the way the other options are done.
Member

I had submitted C2025-029 for correcting the word "titles", but it was updated to "Title" as an update to B2022-031. Closed C2025-029 and updated information on both CSM items to reflect this information.
The tested version was 2.2.2504.714.

I had submitted C2025-029 for correcting the word "titles", but it was updated to "Title" as an update to B2022-031. Closed C2025-029 and updated information on both CSM items to reflect this information. The tested version was 2.2.2504.714.
Sign in to join this conversation.
No Reviewers
No Label
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Volian/SourceCode#537
No description provided.