Skip to content

Commit

Permalink
[keyserver] Add updateRolesAndPermissionsForAllThreads migration
Browse files Browse the repository at this point in the history
Summary:
Now that we're at the end of the stack, this is an `updateRolesAndPermissionsForAllThreads` migration that will help account for changes in both D9599 and D9827

Addresses [[ https://linear.app/comm/issue/ENG-5703/add-updaterolesandpermissionsforallthreads-migration-at-the-end-of-the | ENG-5703 ]]

Depends on D9827

Test Plan: Created some communities and roles on `master`, then switched to my branch and ran the migration. Confirmed that it ran successfully and the roles / permissions were correctly updated

Reviewers: atul, ginsu, ashoat

Reviewed By: ashoat

Subscribers: ashoat, tomek

Differential Revision: https://phab.comm.dev/D9945
  • Loading branch information
RohanK6 committed Nov 22, 2023
1 parent d31d2a0 commit ddaed89
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions keyserver/src/database/migration-config.js
Original file line number Diff line number Diff line change
Expand Up @@ -663,6 +663,7 @@ const migrations: $ReadOnlyMap<number, () => Promise<mixed>> = new Map([
await dbQuery(updateQuery);
},
],
[52, updateRolesAndPermissionsForAllThreads],
]);
const newDatabaseVersion: number = Math.max(...migrations.keys());

Expand Down

0 comments on commit ddaed89

Please sign in to comment.