Change Details

We need to add the pinned_count column to SQLite threads table, so when we (in the next diff) try to store pinnedCount in ThreadInfos, the ClientDB will have the pinned_count. I'm unsure if we should need to run a migration if the column is included in the schema for two reasons: 1. It looks like the SQLite database gets deleted on account log out / re-install 2. I don't think SQLite supports `ADD COLUMN IF NOT EXISTS` so we end up failing the migration if we just try to `ALTER TABLE threads`. **Maybe the alternative is to not include `pinned_count` in the schema and just run a migration to include it (question for the reviewers, since this is the easiest way to ask while showing my code)?** Part of https://linear.app/comm/issue/ENG-3396/store-the-number-of-pinned-messages-in-threadinfo Depends on D7199

We need to add the pinned_count column to SQLite threads table, so when we (in the next diff) try to store pinnedCount in ThreadInfos, the ClientDB will have the pinned_count. As mentioned in the test plan: > I'm not entirely sure if we really even need to consider a case where a new client will already have the column besides me in development work, so I can also just remove that check entirely and just add the column Part of https://linear.app/comm/issue/ENG-3396/store-the-number-of-pinned-messages-in-threadinfo Depends on D7199

We need to add the pinned_count column to SQLite threads table, so when we (in the next diff) try to store pinnedCount in ThreadInfos, the ClientDB will have the pinned_count. I'm unsure if we should need to run a migration if the column is includAs mentioned in the schema for two reasons:test plan: 1. It looks like the SQLite database gets deleted on account log out / re-install 2. I don't think SQLite supports `ADD COLUMN IF NOT EXISTS` so we end up failing the migration if we just try to `ALTER TABLE threads`. **Maybe the alternative is to not include `pinned_count` in the schema and just run a migration to include it (question for the reviewers> I'm not entirely sure if we really even need to consider a case where a new client will already have the column besides me in development work, since this is the easiest way to ask while showing my code)?**so I can also just remove that check entirely and just add the column Part of https://linear.app/comm/issue/ENG-3396/store-the-number-of-pinned-messages-in-threadinfo Depends on D7199