Skip to content

Feat/get all documents - #137

Merged
ChienNQuang merged 7 commits into
devfrom
feat/get-all-documents
Jun 2, 2023
Merged

ChienNQuang merged 7 commits into
devfrom
feat/get-all-documents

Conversation

@ChienNQuang

Copy link
Copy Markdown
Member

No description provided.

@ChienNQuang ChienNQuang added enhancement New feature or request test Introduce Unit or Integration tests labels May 31, 2023
@ChienNQuang ChienNQuang added this to the Iteration 2 milestone May 31, 2023
@ChienNQuang
ChienNQuang requested a review from vzart May 31, 2023 15:57
@ChienNQuang ChienNQuang self-assigned this May 31, 2023
if (!(request.SearchTerm is null || request.SearchTerm.Trim().Equals(string.Empty)))
{
documents = documents.Where(x =>
x.Title.ToLower().Contains(request.SearchTerm.ToLower()));

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Trym()

@ChienNQuang
ChienNQuang merged commit 207b954 into dev Jun 2, 2023
@ChienNQuang ChienNQuang linked an issue Jun 2, 2023 that may be closed by this pull request
@ChienNQuang
ChienNQuang deleted the feat/get-all-documents branch June 2, 2023 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request test Introduce Unit or Integration tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: get all documents

2 participants