Skip to content

Conversation

@brainded
Copy link
Contributor

Prefix currently is only a single Boolean but can support the setting per Query By. This change seeks to limit disturbing existing clients but expanding the capability to allow the full functionality.

I did not mark the existing Prefix as obsolete but I can if the community thinks its best.

@runeanielsen runeanielsen self-assigned this Aug 22, 2025
@runeanielsen
Copy link
Member

runeanielsen commented Aug 25, 2025

Hi @brainded,

I'll take a look at it, this week.

// Rune

@runeanielsen
Copy link
Member

Hi @brainded

I looked at the change, normally I would not like breaking changes, but I rather we just change it to a list of bool instead of just bool, so we do not have to maintain it in the future. I think back when it was implemented only a single bool value could be specified.

// Rune

@brainded
Copy link
Contributor Author

@runeanielsen I will refactor to use that suggestion soon! Thanks for the feedback!

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