Skip to content

Set up Alertviewer on suit #84

Merged
kpuriIpac merged 1 commit into
mainfrom
Enable-Alert-Viewer
Apr 24, 2026
Merged

Set up Alertviewer on suit #84
kpuriIpac merged 1 commit into
mainfrom
Enable-Alert-Viewer

Conversation

@kpuriIpac
Copy link
Copy Markdown
Collaborator

@kpuriIpac kpuriIpac commented Apr 15, 2026

Firefly PR: Caltech-IPAC/firefly#1939

  • Added the relevant code to make the /alertviewer endpoint work on suit

Test:
Suit's Alertviewer: https://enable-alert-viewer.irsakubedev.ipac.caltech.edu/suit/alertviewer

  • Test that the endpoint works
  • if you try and actual alert ID (like 170059278837088375), you may expectedly get the missing/invalid token error.
    • to test the UI, check out and run this branch locally
    • I tested the URL API locally as well and it works for me, feel free to test that as well

@kpuriIpac kpuriIpac requested a review from robyww April 15, 2026 19:07
@kpuriIpac kpuriIpac self-assigned this Apr 15, 2026
@kpuriIpac kpuriIpac force-pushed the Enable-Alert-Viewer branch from dfbb7f9 to f80d843 Compare April 15, 2026 23:36
@kpuriIpac kpuriIpac force-pushed the Enable-Alert-Viewer branch 2 times, most recently from 070520f to 7c062eb Compare April 23, 2026 18:09
Copy link
Copy Markdown
Contributor

@robyww robyww left a comment

Choose a reason for hiding this comment

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

looks good. Go ahead and merge.

Comment thread src/suit/js/SUIT.js Outdated
{label: 'IRSA Catalogs', action: 'IrsaCatalog', category:OTHER_CAT},
{label: 'NED Objects', action: 'ClassicNedSearchCmd', primary: false, category:OTHER_CAT},
{label: 'VO Cone Search', action: 'ClassicVOCatalogPanelCmd', primary: false, category: OTHER_CAT},
{label: 'Alert Viewer', action: 'AlertViewerSearch', primary: false, category: OTHER_CAT},
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

make primary: true

@kpuriIpac kpuriIpac force-pushed the Enable-Alert-Viewer branch from 39ff15d to a1c250c Compare April 24, 2026 19:08
@kpuriIpac kpuriIpac merged commit 7e5dc9a into main Apr 24, 2026
1 check passed
@kpuriIpac kpuriIpac deleted the Enable-Alert-Viewer branch April 24, 2026 19:31
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