Skip to content

Config for SkyPortal URL and group#29

Merged
karpov-sv merged 3 commits into
karpov-sv:masterfrom
vlad2305m:feature/skyportal-url-config
Jul 15, 2026
Merged

Config for SkyPortal URL and group#29
karpov-sv merged 3 commits into
karpov-sv:masterfrom
vlad2305m:feature/skyportal-url-config

Conversation

@vlad2305m

Copy link
Copy Markdown
Contributor

This is needed to test SkyPortal integrations with a local instance of SkyPortal.

Comment thread stdweb/templates/skyportal.html Outdated
Error: <span class="text-danger">{{ task.error }}
{% else %}
SkyPortal source: <a href="https://skyportal-icare.ijclab.in2p3.fr/source/{{ task.sid }}" target="_blank">{{ task.sid }}</a>
SkyPortal source: <a href="{{ skyportal_url }}/source/{{ task.sid }}" target="_blank">{{ task.sid }}</a>

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You actually do not need to use a separate context variable for that, the settings is directly accessible from the template. See e.g.

{% if perms.stdweb.skyportal_upload and settings.SKYPORTAL_TOKEN %}

Comment thread stdweb/views_skyportal.py Outdated
tasks.append(ctask)

context['tasks'] = tasks
context['skyportal_url'] = settings.SKYPORTAL_BASE_URL

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same as above

@karpov-sv
karpov-sv merged commit de24398 into karpov-sv:master Jul 15, 2026
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