Skip to content

Remove legacy export without files option #424

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.

Sign up for GitHub

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

Open
Abh1ramManugur1 wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Remove legacy export without files option #424

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
4262296
Remove legacy export endpoint
Abh1ramManugur1 Jun 22, 2026
a843279
remove legacy export button from transient download dropdown
Abh1ramManugur1 Jun 22, 2026
39ae084
remove helper modules
Abh1ramManugur1 Jun 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
146 changes: 0 additions & 146 deletions app/api/components.py
View file
Open in desktop

This file was deleted.

72 changes: 0 additions & 72 deletions app/api/datamodel.py
View file
Open in desktop

This file was deleted.

122 changes: 0 additions & 122 deletions app/api/tests/test_datamodel.py
View file
Open in desktop

This file was deleted.

4 changes: 0 additions & 4 deletions app/api/urls.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,10 +9,6 @@
base_path = f"""{base_path}/"""

urlpatterns = [
path(
f"""{base_path}transient/get/<str:transient_name>""",
views.get_transient_science_payload,
),
re_path(
base_path + r"^transient/delete/(?P<transient_name>[a-zA-Z0-9_-]+)/(?P<all>all/|)$",
views.delete_transient_view,
Expand Down
21 changes: 0 additions & 21 deletions app/api/views.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,9 +41,6 @@
from api.serializers import TaskSerializer
from api.serializers import HostSerializer
from api.serializers import AliasSerializer
from api.datamodel import unpack_component_groups
from api.datamodel import serialize_blast_science_data
from api.components import data_model_components

from host.log import get_logger
logger = get_logger(__name__)
Expand Down Expand Up @@ -273,24 +270,6 @@ def ra_dec_valid(ra: str, dec: str) -> bool:
# return exists


@api_view(["GET"])
@log_usage_metric()
def get_transient_science_payload(request, transient_name):
if not transient_exists(transient_name):
return Response(
{"message": f"{transient_name} not in database"},
status=status.HTTP_404_NOT_FOUND,
)

component_groups = [
component_group(transient_name) for component_group in data_model_components
]
components = unpack_component_groups(component_groups)
data = serialize_blast_science_data(components)

return Response(data, status=status.HTTP_200_OK)


@api_view(["POST"])
def post_transient(request, transient_name, transient_ra, transient_dec):
if transient_exists(transient_name):
Expand Down
2 changes: 1 addition & 1 deletion app/host/templates/host/transient_actions.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
<a class="dropdown-item" title="Export with all files" href="/api/transient/export/{{transient.name}}/all"><i class="bi bi-file-earmark-zip"></i> Export with all files (.tar.gz)</a>
<div class="dropdown-divider"></div>
<a class="dropdown-item" title="Export without files" href="/api/transient/export/{{transient.name}}"><i class="bi bi-filetype-json"></i> Export without files (.json)</a>
<a class="dropdown-item" title="Export without files (legacy)" href="/api/transient/get/{{transient.name}}?format=json"><i class="bi bi-filetype-json"></i> Export without files (legacy .json)</a>

{% if local_sed_results %}
<div class="dropdown-divider"></div>
<a class="dropdown-item" title="Local parameter confidence intervals" href="{% url 'download_modelfit' transient.name 'local' %}"><i class="bi bi-graph-up"></i> Local SED best-fit model (.npz)</a>
Expand Down