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

Update filter documentation #909

Closed
jsumners opened this issue Jun 27, 2023 · 3 comments
Closed

Update filter documentation #909

jsumners opened this issue Jun 27, 2023 · 3 comments
Labels
documentation good first issue

Comments

@jsumners
Copy link
Member

jsumners commented Jun 27, 2023 •
edited
Loading

The substring filter documentation is inconsistent with the current implentation. It should be updated.

The latter are also not mentioned in the example.

The documentation should be updated.

Source: #907
@jsumners jsumners mentioned this issue Jun 27, 2023
Closed
@devbysn devbysn mentioned this issue Jun 28, 2023
Closed
@devbysn
Copy link

devbysn commented Jun 28, 2023

Hello @jsumners
I've updated the parameters and created a pull request.
Can you verify?

@devbysn
Copy link

devbysn commented Jun 28, 2023

I am still very beginner,
i hope i didn't mess anything :)

@jsumners
Copy link
Member Author

This is being resolved by ldapjs/filter#5.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
documentation good first issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jsumners @devbysn