Skip to content

Feature/filter enhancements (Dates, Attributes, Lines, Points, Polygons - #866

Open
nagisml wants to merge 12 commits into
OpenSAK-Org:betafrom
nagisml:feature/filter-enhancements
Open

nagisml wants to merge 12 commits into
OpenSAK-Org:betafrom
nagisml:feature/filter-enhancements

Conversation

@nagisml

@nagisml nagisml commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Next part of the filter enhancements

  • Date filters with conditions similar to Strings conditions
  • Line/Point/Polygon filtering
  • OR condition for attribute filtering
    Next update from Filter enhancement #557

@AgreeDK

AgreeDK commented Sep 16, 2026

Copy link
Copy Markdown
Member

@nagisml
Thanks for raising this, there is a conflict now, as Im trying to fix a problem with migration on Mac, so until it have been fixed,you dont have to fixed this PR.

@AgreeDK

AgreeDK commented Sep 16, 2026

Copy link
Copy Markdown
Member

Hi @nagisml, thanks for this — great to see the next part of #557 coming together, and Lines/Point/Polygon filtering will be a nice addition alongside #680/#184.

We're in the middle of tracking down a data-loss bug affecting some macOS users (three patch releases out today), so I won't be able to review/merge this for a couple of days. No need to rebase against beta yet — it'll likely move a bit more before I'm free, so hold off until I give you the go-ahead, then a rebase right before review will save you doing it twice.

I've assigned you to #558 too, as requested — feel free to start poking at that in the meantime if you'd like something to work on while this one waits.

Will follow up in ~2 days.

@nagisml

nagisml commented Sep 16, 2026

Copy link
Copy Markdown
Contributor Author

@AgreeDK no hurry as bug fixing has higher prio then features
Will be off starting next week till October will se if I find some minutes to push again

FYI There are 2 more branches ready for PR but as they need this one first I put them on hold:

Both work here without issues.

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