Propagate the PurpleAccount::notify signal via PurpleAccountManager::account-changed

Review Request #2032 — Created Nov. 5, 2022 and submitted

Information

pidgin/pidgin
default

Reviewers

This new signal supports details and works just like notify does on the account
instances, but it will be emitted for any account that the manager instance
knows about which means we can remove the old purple signals.

Enabled and disabled accounts in Pidgin via the menus to make sure things were working right, and verified via the console that the accounts were being enabled and disabled.
Ran Finch, but I ran into the infinite loop bug in the buddy list clean when disabling an account which is a known issue.

Summary ID
Propagate the PurpleAccount::notify signal via PurpleAccountManager::account-changed
This new signal supports details and works just like notify does on the account instances, but it will be emitted for any account that the manager instance knows about which means we can remove the old purple signals.
fba96a4d56d362f552ac09c6591ca1facfb4ac9b
Description From Last Updated

data is unused now.

QuLogicQuLogic

when an account's

QuLogicQuLogic

enabled

QuLogicQuLogic
QuLogic
  1. 
      
  2. finch/gntaccount.c (Diff revision 1)
     
     

    data is unused now.

  3. libpurple/purpleaccountmanager.c (Diff revision 1)
     
     

    when an account's

  4. pidgin/pidginaccountsenabledmenu.c (Diff revision 1)
     
     

    enabled

  5. 
      
grim
QuLogic
  1. Ship It!
  2. 
      
grim
Review request changed

Status: Closed (submitted)

Loading...