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

State of the common/preview directory #373

Open
arnesetzer opened this issue Mar 12, 2025 · 6 comments
Open

State of the common/preview directory #373

arnesetzer opened this issue Mar 12, 2025 · 6 comments
Labels
bug Something isn't working

Comments

@arnesetzer
Copy link

Describe the bug
Not really a bug. What is the status of the common/preview directory? It's only used in the south africa project and the last change was 7 years ago on docker@eb1077 and it doesn't work anymore due to url changes.

If there are any ambitions to get it running again, I would volunteer to do so.

@arnesetzer arnesetzer added the bug Something isn't working label Mar 12, 2025
@missinglink
Copy link
Member

Interesting, I didn't even know we had this 😆
@orangejulius maybe you remember the history of this, seems like a nice thing to have.

The libs are very out-of-date, I wonder if we can use https://github.com/geocodeearth/autocomplete-element instead, it should be configurable enough to point to a different host than our default api cluster.

@missinglink
Copy link
Member

@orangejulius
Copy link
Member

Haha, yes I was going to say, I had completely forgotten about this experiment, and was going to bet Peter didn't even know about it. 😆

I do think it's a good thing to have, @arnesetzer if you want to brush it up that would be super awesome. I agree the autocomplete element is probably the best autocomplete tooling that exists, though it's not strictly speaking Pelias specific

@missinglink
Copy link
Member

Oh yeah hmm it might enforce an API key 🤔

@arnesetzer
Copy link
Author

Autocomplete-element is currently not an option as it relies on this line which has the protocol hardcoded to https and my motivation to create self-signed certificates is not that great 😁.

Honestly, I was just thinking of updating the leaflet dependency and changing the tile URL.

@orangejulius
Copy link
Member

See also #10 where this idea was originally discussed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants