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

@kpuriIpac kpuriIpac commented Apr 15, 2026

Copy link
Copy Markdown
Collaborator

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

@robyww robyww left a comment

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.

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