gui: Changes the connect for the beaconChanged signal on main screen

[jamescowens]

beaconChanged now connects with updateResearcherStatus which should fix the non-update of the researcher status when the beacon is activated.

[Copilot]

Pull Request Overview

This PR updates the connection for the beaconChanged signal so that it invokes updateResearcherStatus instead of updateResearcherAlert, aiming to fix the researcher status not refreshing when the beacon is activated.

• Changed the slot called on beaconChanged from updateResearcherAlert to updateResearcherStatus.

Comments suppressed due to low confidence (2)

src/qt/overviewpage.cpp:376

• Consider adding a unit test to verify that updateResearcherStatus is invoked when beaconChanged is emitted to prevent future regressions.

connect(researcherModel, &ResearcherModel::beaconChanged, this, &OverviewPage::updateResearcherStatus);

src/qt/overviewpage.cpp:376

• Replacing the beaconChanged connection from updateResearcherAlert to updateResearcherStatus removes the alert update; consider connecting beaconChanged to updateResearcherAlert as well or ensuring alert state is updated elsewhere.

connect(researcherModel, &ResearcherModel::beaconChanged, this, &OverviewPage::updateResearcherStatus);

[jamescowens]

Stupid Copilot. updateResearcherStatus calls

    updateMagnitude();
    updatePendingAccrual();
    updateResearcherAlert();