Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Search characters by level #212

Closed
wants to merge 917 commits into from
Closed

Conversation

jkisor
Copy link
Contributor

@jkisor jkisor commented Oct 30, 2021

Search characters by level #211

Scope is limited to Character search by name and level.

The supported permutations:

  • Search for characters by level
    • Example: "level:275"
  • Search by name and level.
    • Example: "Asheron level:275"
  • Search only name (existing behavior)
    • Example: "Asheron"

Notes:

  • filter is identified as a word containing :; anything other words are assumed to be a name filter.
  • Incomplete filters "level:" are discarded
  • Fields must be whitelisted/allowed otherwise discarded; (current only allow "name" and "level")

Future considerations:

Special handling per field

As we allow more fields as filters, we need an approach to special handling different fields. For example, name needs regex case insensitive. A solution for this would lead to allowing different filters to have their own format/parsing (for example, supporting level ranges "level:15-25")

@jkisor
Copy link
Contributor Author

jkisor commented Oct 30, 2021

Oops, we may have done the same work. No worries, Feel free to close.

@amoeba
Copy link
Owner

amoeba commented Nov 15, 2021

Hey @jkisor, apologies for the radio silence here. We did do similar work here, though I think there's some good stuff in yours. And mine could be improved for sure.

I'm going to close this PR mainly because of all the git-related noise. I've refactored how I do client message validation so this repo is no longer a mirror of a private repo and is instead the primary repo. I was using git-filter-branch for years which rewrites commits in order to filter out sensitive info and causes pain even when I was just working alone but even more pain when trying to take contributions.

Future contributions will likely be much easier to merge. Thanks for taking the time to contribute!

@amoeba amoeba closed this Nov 15, 2021
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.

2 participants