Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

feat: now search callback response will emit SearchRequest after every search request is sent #730

Merged
merged 4 commits into from
May 27, 2021

Conversation

zyf0330
Copy link
Contributor

@zyf0330 zyf0330 commented May 24, 2021

No description provided.

@zyf0330
Copy link
Contributor Author

zyf0330 commented May 24, 2021

#728

Copy link
Member

@UziTech UziTech left a comment

Choose a reason for hiding this comment

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

You can run npm run lint to fix the linting issues.

docs/client.md Outdated Show resolved Hide resolved
zyf0330 and others added 2 commits May 24, 2021 13:33
Co-authored-by: Tony Brix <tony@brix.ninja>
@zyf0330
Copy link
Contributor Author

zyf0330 commented May 24, 2021

You can run npm run lint to fix the linting issues.

Changes have been applied.

@zyf0330 zyf0330 requested a review from UziTech May 24, 2021 07:31
Copy link
Member

@UziTech UziTech left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@jsumners jsumners left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@UziTech UziTech merged commit 505f1f8 into ldapjs:master May 27, 2021
@UziTech UziTech mentioned this pull request May 27, 2021
@jsumners
Copy link
Member

⚠️ This issue has been locked due to age. If you have encountered a recent
problem that seems to be covered by this issue, please open a new issue.

Please include a minimal reproducible example
when opening a new issue.

@ldapjs ldapjs locked as resolved and limited conversation to collaborators Mar 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants