Skip to content

Advanced option to render fences#72

Draft
ComplementaryPogo wants to merge 16 commits intoccev:mainfrom
ComplementaryPogo:fences
Draft

Advanced option to render fences#72
ComplementaryPogo wants to merge 16 commits intoccev:mainfrom
ComplementaryPogo:fences

Conversation

@ComplementaryPogo
Copy link
Contributor

Adds an advanced option to render fences, as well as an option to filter the koji API response to only contain areas that the user has permissions in.
Screenshot 2025-12-31 at 5 34 46 AM

@ccev
Copy link
Owner

ccev commented Dec 31, 2025

thanks! my long-term idea would be to have a separate route with a map showing your areas, and a separate "Tools" tab that could display them on the main map. I think having a toggle for this in your settings is the wrong place. Probably fine to merge it for the meantime tho!

also please remove the non-english translations (unless you speak those languages), I'm personally pretty opposed to having machine translated strings in this project, I think it's slop.

Also, I'd prefer just calling geofences "Areas" or "Scan areas" if you will, calling them koji fences or even fences is too confusing for a normal person.

@ccev ccev marked this pull request as draft February 1, 2026 00:04
@ccev
Copy link
Owner

ccev commented Feb 1, 2026

decided against merging this, i want to implement areas the "right way", i will either base my implementation on this pr or close it when i'm finished.

converted to a draft and merged main, so people can checkout this pr in the meantime. thanks!

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