Page MenuHomePhabricator
Differential D9974

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

Authored by rohan on Nov 24 2023, 9:47 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 12, 3:03 PM
Unknown Object (File)
Fri, Apr 11, 12:45 AM
Unknown Object (File)
Wed, Apr 9, 12:56 AM
Unknown Object (File)
Tue, Apr 8, 11:27 PM
Unknown Object (File)
Tue, Apr 8, 11:14 PM
Unknown Object (File)
Tue, Apr 8, 10:46 PM
Unknown Object (File)
Mon, Mar 31, 11:53 AM
Unknown Object (File)
Mon, Mar 31, 10:24 AM
View All Files
Subscribers
ashoat

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
No Lint Coverage
Unit
No Test Coverage

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
rohan updated this revision to Diff 33726.Nov 27 2023, 6:17 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

PathSize
keyserver/
src/
creators/
7 lines
lib/
permissions/
4 lines

Diff 33726

View Options

keyserver/src/creators/role-creator.js

Loading...
View Options

lib/permissions/special-roles.js

Loading...