| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
| |
Also a few other clean ups, including making the user of the manager responsible
for sorting the list.
Testing Done:
Ran the unit tests and verified the notification list in pidgin was updating correctly.
Reviewed at https://reviews.imfreedom.org/r/2085/
|
|
|
|
|
|
|
|
|
|
| |
Testing Done:
* Ran the unit tests with the new test
* Caused a resource conflict on XMPP to verify that notification was displayed
* Attempted to connect an IRC account that had a bad hostname and verified that notification was displayed.
* Not integration testing was done for Finch.
Reviewed at https://reviews.imfreedom.org/r/1944/
|
|
|
|
|
|
|
|
|
|
| |
This is used when an account is disabled or removed to purge its notifications
from the manager.
Testing Done:
Ran the unit tests and forced them to fail with bad values to verify they were correct.
Reviewed at https://reviews.imfreedom.org/r/1612/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This currently only shows connection error notifications which have to be
removed manually since an account with a connection error can not currently
reconnect successfully.
Testing Done:
I used a IRC account with an server name that was unresolvable as well as an XMPP account with a wrong password to cause a fatal connection errors.
I also used a IRC account connecting to a local instance of ZNC where I terminated ZNC to cause a server side connection failure.
Bugs closed: PIDGIN-17634
Reviewed at https://reviews.imfreedom.org/r/1512/
|
|
|
|
|
|
|
|
|
|
|
|
| |
Adjust the NotificationManager API a bit to be easier to use and map better to
GListStore which the manager is now using internally.
Testing Done:
Ran the unit tests and forced a connection error and verified no issues (used gdb to verify that item was added).
Bugs closed: PIDGIN-17639
Reviewed at https://reviews.imfreedom.org/r/1509/
|
|
* Created PurpleNotification with unit tests.
* Created PurpleNotificationManager with unit tests.
Testing Done:
Ran the unit tests and ran Pidgin in the devenv.
Bugs closed: PIDGIN-17633
Reviewed at https://reviews.imfreedom.org/r/1502/
|