Skip to content

Move search string parsing into common helper class#3075

Merged
gunndabad merged 4 commits intomainfrom
searchtexthelper
Apr 17, 2026
Merged

Move search string parsing into common helper class#3075
gunndabad merged 4 commits intomainfrom
searchtexthelper

Conversation

@gunndabad
Copy link
Copy Markdown
Member

No description provided.

@gunndabad
Copy link
Copy Markdown
Member Author

@copilot add unit tests for SearchTextHelper

Copy link
Copy Markdown
Contributor

Copilot AI commented Feb 9, 2026

@gunndabad I've opened a new pull request, #3076, to work on those changes. Once the pull request is ready, I'll request review from you.

@gunndabad gunndabad marked this pull request as ready for review February 9, 2026 10:36
Comment thread TeachingRecordSystem/src/TeachingRecordSystem.Core/SearchTextHelper.cs Outdated
gunndabad and others added 3 commits April 17, 2026 09:32
- [x] Create SearchTextHelperTests.cs file
- [x] Add tests for IsDate method
- [x] Add tests for IsEmailAddress method  
- [x] Add tests for IsSupportTaskReference method
- [x] Add tests for IsTrn method
- [x] Run tests to verify implementation
- [x] Format and build code

<!-- START COPILOT CODING AGENT TIPS -->
---

💡 You can make Copilot smarter by setting up custom instructions,
customizing its development environment and configuring Model Context
Protocol (MCP) servers. Learn more [Copilot coding agent
tips](https://gh.io/copilot-coding-agent-tips) in the docs.

---------

Co-authored-by: James Gunn <james@gunn.io>
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: gunndabad <2041280+gunndabad@users.noreply.github.com>
@gunndabad gunndabad added this pull request to the merge queue Apr 17, 2026
Merged via the queue into main with commit 9852a3f Apr 17, 2026
24 checks passed
@gunndabad gunndabad deleted the searchtexthelper branch April 17, 2026 09:15
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.

4 participants