Refactored add to favourites button to avoid refresh - Githubissues

askap-vast / vast-pipeline

This repository holds the code of the Radio Transient detection pipeline for the VAST project.

https://vast-survey.org/vast-pipeline/

MIT License

7 stars 3 forks source link

Refactored add to favourites button to avoid refresh #614

Closed ajstewart closed 2 years ago

ajstewart commented 2 years ago

Used ajax to post request to avoid page load.
Split off favourite button and modal components to avoid duplication.
Added functionality to toasts to show pop-up ajax alerts.
Fixes bug I introduced to favouriting a source from the detail page in #586.

I did the refactor so if someone wanted to in the future they could take it further and make the whole thing a toggle.

I had a brief look at doing the same to the comments and tags, and adding them to the analysis page, but it needs quite a bit of a refactor.