We now expect clients to get RoleInfos with the specialRole field included. We make the field required (it's still ?SpecialRole, so can be set to null) and make the necessary flow changes.
Depends on D11248
Paths
| Differential D11252 Authored by atul on Mar 5 2024, 9:20 PM.
Details Summary We now expect clients to get RoleInfos with the specialRole field included. We make the field required (it's still ?SpecialRole, so can be set to null) and make the necessary flow changes. Depends on D11248 Test Plan flow + verifying that clients recieve specialRole field in RoleInfo from keyserver responses.
Diff Detail
Event TimelineHerald added a subscriber: ashoat. · View Herald TranscriptMar 5 2024, 9:20 PM2024-03-05 21:20:15 (UTC-8) atul added a child revision: D11253: [lib] Remove `isDefault` field from `RoleInfo`.Mar 5 2024, 9:28 PM2024-03-05 21:28:30 (UTC-8) Harbormaster completed remote builds in B27349: Diff 37871.Mar 5 2024, 9:35 PM2024-03-05 21:35:50 (UTC-8) Harbormaster completed remote builds in B27351: Diff 37873.Mar 5 2024, 9:51 PM2024-03-05 21:51:21 (UTC-8) This revision is now accepted and ready to land.Mar 6 2024, 4:21 AM2024-03-06 04:21:37 (UTC-8) Comment Actions We ended up going with a very different approach from this stack. Abandoning to tidy things up.
Revision Contents
Diff 37873 lib/permissions/minimally-encoded-thread-permissions-test-data.js
lib/permissions/special-roles.js
lib/permissions/special-roles.test.js
lib/reducers/calendar-filters-reducer.test.js
|