Skip to content

Commit 5a05edb

Browse files
authored
fix(navigation): keep contributor with board-member role (#613)
1 parent 1870bea commit 5a05edb

2 files changed

Lines changed: 5 additions & 4 deletions

File tree

apps/lfx-one/src/server/services/navigation.service.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -101,8 +101,7 @@ export class NavigationService {
101101
// other lenses (e.g., Me → Open) doesn't get overridden by the default picker.
102102
if (selectedUid && !pageToken) {
103103
const alreadyIncluded = accumulated.some((item) => item.uid === selectedUid);
104-
const allowedByPersona = !eligibleUids || eligibleUids.has(selectedUid);
105-
if (!alreadyIncluded && allowedByPersona) {
104+
if (!alreadyIncluded) {
106105
const selectedItem = await this.fetchSelectedItem(req, lens, selectedUid);
107106
if (selectedItem) {
108107
accumulated.unshift(selectedItem);

apps/lfx-one/src/server/services/persona-detection.service.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -372,8 +372,10 @@ export class PersonaDetectionService {
372372
personas.add('contributor');
373373
}
374374

375-
// Drop contributor when a more specific role is present.
376-
if (personas.size > 1 && personas.has('contributor')) {
375+
// Drop contributor only when maintainer is present — the true upgrade
376+
// of the same domain. Board-member / executive-director are orthogonal
377+
// foundation roles and must not cause contributor to be dropped.
378+
if (personas.has('contributor') && personas.has('maintainer')) {
377379
personas.delete('contributor');
378380
}
379381

0 commit comments

Comments
 (0)