Summary

Add serialization support to the coversation manager

Review Request #3698 — Created Dec. 10, 2024 and updated Dec. 23, 2024, 8:37 p.m.

Information

Owner

grim

Repository

pidgin/pidgin

Branch

default

Bugs

PIDGIN-17989

Depends On

Reviewers

Groups

pidgin

People

Description

Members were skipped for the time being as we have some signals to fix up to
notify of changes and we do need to store they for all conversation types to
keep try of tags, aliases, etc.

Testing Done

Connected to my local IRC servers and verified everything was saved properly. Also kicked a pidgin3 instance out of a room and verified the error was serialized as well.

Commits

Summary	ID
Add serialization support to the coversation manager Members were skipped for the time being as we have some signals to fix up to notify of changes and we do need to store they for all conversation types to keep try of tags, aliases, etc.	7483adcb7d9a05a553c808798a641b530cd88e31

Issues

Description	From	Last Updated
It feels like a couple of words in the description could need replacing. s/they/them s/try/track	ivanhoe	Dec. 12, 2024, 2:01 p.m.
We're punting on this for now as the volume of code related to sqlite is extremely high and we want …	grim	Dec. 23, 2024, 8:37 p.m.
This reads oddly. I think the alias is superfluous.	ivanhoe	Dec. 12, 2024, 2:01 p.m.
s/of time/of the time	ivanhoe	Dec. 12, 2024, 2:01 p.m.
s/distribute/distributed s/conversation/conversations	ivanhoe	Dec. 12, 2024, 2:01 p.m.
s/the the/the	ivanhoe	Dec. 12, 2024, 2:01 p.m.
s/of the/the	ivanhoe	Dec. 12, 2024, 2:01 p.m.

Ship it! Fix it!

```
Ship It!
```

An issue was opened. Show all issues

It feels like a couple of words in the description could need replacing.
s/they/them
s/try/track

libpurple/data/im.pidgin.Purple.ConversationManager.gschema.xml (Diff revision 1)
An issue was opened. Show all issues
```
This reads oddly. I think the alias is superfluous.
```
libpurple/data/im.pidgin.Purple.ConversationManager.gschema.xml (Diff revision 1)
An issue was opened. Show all issues
```
s/of time/of the time
```
libpurple/data/im.pidgin.Purple.ConversationManager.gschema.xml (Diff revision 1)
An issue was opened. Show all issues
```
s/distribute/distributed

s/conversation/conversations
```
libpurple/data/im.pidgin.Purple.ConversationManager.gschema.xml (Diff revision 1)
An issue was opened. Show all issues
```
s/the the/the
```
libpurple/data/im.pidgin.Purple.ConversationManager.gschema.xml (Diff revision 1)
An issue was opened. Show all issues
```
s/of the/the
```

Fix it!

An issue was opened. Show all issues

We're punting on this for now as the volume of code related to sqlite is extremely high and we want to build something to minimize that. However that is not going to be ready in time for exp1.