Skip to content

Prefix update to allow specifying if Prefix should be used on the different Query By parameters#293

Open
brainded wants to merge 2 commits intoDAXGRID:masterfrom
brainded:prefix-update
Open

Prefix update to allow specifying if Prefix should be used on the different Query By parameters#293
brainded wants to merge 2 commits intoDAXGRID:masterfrom
brainded:prefix-update

Conversation

@brainded
Copy link
Copy Markdown
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
Copy Markdown
Member

runeanielsen commented Aug 25, 2025

Hi @brainded,

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

// Rune

@runeanielsen
Copy link
Copy Markdown
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
Copy Markdown
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