-
Notifications
You must be signed in to change notification settings - Fork 77
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
Top-level configurable #2429
Comments
@fnielsen, @Daniel-Mietchen, please let me know what you think a patch to do this should take into account. |
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): |
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 |
The setup and reading of the configuration is handle by a separate module.
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:
iframe
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).
The text was updated successfully, but these errors were encountered: