Paths

Table of Contentst

[keyserver/lib] Populate special_role field when initial roles are created
ClosedPublic
Actions

Authored by rohan on Nov 24 2023, 9:47 AM.

Details

Reviewers

atul
ginsu
tomek

Commits

rCOMM4948dcde64ba: [keyserver/lib] Populate special_role field when initial roles are created

Summary

Now that we've createde a roles.special_role column in MariaDB and wrote a migration that populates it for existing threads, we should make sure new threads going forward populate this upon creation.

Depends on D9961

Part of ENG-5833

Test Plan

Create some new communities and make sure the special_role is populated for Members (no special role for Admins yet)

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

rohan created this revision.Nov 24 2023, 9:47 AM

Herald added a subscriber: ashoat. · View Herald TranscriptNov 24 2023, 9:47 AM

rohan added a child revision: D9975: [keyserver] Update usage of threads.default_role in role-deleters.js.Nov 24 2023, 9:49 AM

rohan edited the test plan for this revision. (Show Details)Nov 24 2023, 10:02 AM

Harbormaster completed remote builds in B24401: Diff 33600.Nov 24 2023, 10:21 AM

rohan requested review of this revision.Nov 24 2023, 10:21 AM

ashoat added inline comments.Nov 26 2023, 3:34 PM

keyserver/src/creators/role-creator.js
54 ↗	(On Diff #33600)	Best to opt for `??` over `\|\|` when possible

tomek accepted this revision.Nov 27 2023, 12:51 AM

This revision is now accepted and ready to land.Nov 27 2023, 12:51 AM

Use ?? over ||

Harbormaster completed remote builds in B24520: Diff 33726.Nov 27 2023, 9:25 AM

Closed by commit rCOMM4948dcde64ba: [keyserver/lib] Populate special_role field when initial roles are created (authored by rohan). · Explain WhyNov 27 2023, 10:36 AM

This revision was automatically updated to reflect the committed changes.

rohan added a commit: rCOMM4948dcde64ba: [keyserver/lib] Populate special_role field when initial roles are created.

rohan mentioned this in D10076: [keyserver] Run migration to populate special_role column for Admins.Nov 28 2023, 11:32 AM

rohan mentioned this in rCOMM11ae2a8c39ac: [keyserver] Run migration to populate special_role column for Admins.Nov 29 2023, 6:06 PM

Revision Contents
Changeset List

Path

Size

keyserver/

src/

creators/

role-creator.js

7 lines

lib/

permissions/

special-roles.js

4 lines

Diff 33752

View Options

keyserver/src/creators/role-creator.js