WDscholia / scholia

Wikidata-based scholarly profiles
https://scholia.toolforge.org
Other
219 stars 78 forks source link

Top-level configurable #2429

Closed egonw closed 4 weeks ago

egonw commented 7 months ago

Is your feature request related to a problem? Please describe. This is related to the ongoing developments around splits and integration with the UIs for queries, like WDQS.

See also:

Describe the solution you'd like We need in both Python and JavaScript global URLs for the following things:

I think these three templates need to have a paramater (like $sparql) where the encoded SPARQL is embedded.

Describe alternatives you've considered The current solution is to have these URL patterns hardcoded all over the place. This makes it hard to maintain and update Scholia for other endpoints (like Qlever).

egonw commented 7 months ago

@fnielsen, @Daniel-Mietchen, please let me know what you think a patch to do this should take into account.

fnielsen commented 7 months ago

Yes. Long time ago we an issue or PR.

egonw commented 7 months ago

Yes. Long time ago we an issue or PR.

@fnielsen, can you help me out? I can find the following related queries, but none capture this issue (fully):

fnielsen commented 6 months ago

Yes. I think it would be fine with a configuration. I have previously used the config module: https://github.com/fnielsen/brede/blob/master/brede/config.py