Skip to content

Commit c554844

Browse files
authored
Merge pull request #760 from RedHatInsights/master
Remove beta flag on user preferences (#757) master-stable
2 parents e02b7aa + 957799e commit c554844

2 files changed

Lines changed: 56 additions & 8 deletions

File tree

src/js/App/Header/HeaderTests/__snapshots__/UserToggle.test.js.snap

Lines changed: 50 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,11 @@ exports[`ConnectedUserToggle -- not org admin should render correctly 1`] = `
5858
>
5959
My profile
6060
</DropdownItem>,
61+
<DropdownItem
62+
href="./user-preferences/email"
63+
>
64+
User preferences
65+
</DropdownItem>,
6166
<DropdownItem
6267
component="button"
6368
onClick={[Function]}
@@ -114,6 +119,11 @@ exports[`ConnectedUserToggle -- not org admin should render correctly 1`] = `
114119
>
115120
My profile
116121
</DropdownItem>,
122+
<DropdownItem
123+
href="./user-preferences/email"
124+
>
125+
User preferences
126+
</DropdownItem>,
117127
<DropdownItem
118128
component="button"
119129
onClick={[Function]}
@@ -171,6 +181,11 @@ exports[`ConnectedUserToggle -- not org admin should render correctly 1`] = `
171181
>
172182
My profile
173183
</DropdownItem>,
184+
<DropdownItem
185+
href="./user-preferences/email"
186+
>
187+
User preferences
188+
</DropdownItem>,
174189
<DropdownItem
175190
component="button"
176191
onClick={[Function]}
@@ -229,6 +244,11 @@ exports[`ConnectedUserToggle -- not org admin should render correctly 1`] = `
229244
>
230245
My profile
231246
</DropdownItem>,
247+
<DropdownItem
248+
href="./user-preferences/email"
249+
>
250+
User preferences
251+
</DropdownItem>,
232252
<DropdownItem
233253
component="button"
234254
onClick={[Function]}
@@ -494,6 +514,11 @@ exports[`ConnectedUserToggle -- org admin should render correctly 1`] = `
494514
>
495515
My profile
496516
</DropdownItem>,
517+
<DropdownItem
518+
href="./user-preferences/email"
519+
>
520+
User preferences
521+
</DropdownItem>,
497522
<DropdownItem
498523
component="button"
499524
onClick={[Function]}
@@ -557,6 +582,11 @@ exports[`ConnectedUserToggle -- org admin should render correctly 1`] = `
557582
>
558583
My profile
559584
</DropdownItem>,
585+
<DropdownItem
586+
href="./user-preferences/email"
587+
>
588+
User preferences
589+
</DropdownItem>,
560590
<DropdownItem
561591
component="button"
562592
onClick={[Function]}
@@ -621,6 +651,11 @@ exports[`ConnectedUserToggle -- org admin should render correctly 1`] = `
621651
>
622652
My profile
623653
</DropdownItem>,
654+
<DropdownItem
655+
href="./user-preferences/email"
656+
>
657+
User preferences
658+
</DropdownItem>,
624659
<DropdownItem
625660
component="button"
626661
onClick={[Function]}
@@ -686,6 +721,11 @@ exports[`ConnectedUserToggle -- org admin should render correctly 1`] = `
686721
>
687722
My profile
688723
</DropdownItem>,
724+
<DropdownItem
725+
href="./user-preferences/email"
726+
>
727+
User preferences
728+
</DropdownItem>,
689729
<DropdownItem
690730
component="button"
691731
onClick={[Function]}
@@ -916,6 +956,11 @@ exports[`UserToggle should render correctly with isSmall false 1`] = `
916956
>
917957
My profile
918958
</DropdownItem>,
959+
<DropdownItem
960+
href="./user-preferences/email"
961+
>
962+
User preferences
963+
</DropdownItem>,
919964
<DropdownItem
920965
component="button"
921966
onClick={[Function]}
@@ -973,6 +1018,11 @@ exports[`UserToggle should render correctly with isSmall true 1`] = `
9731018
>
9741019
My profile
9751020
</DropdownItem>,
1021+
<DropdownItem
1022+
href="./user-preferences/email"
1023+
>
1024+
User preferences
1025+
</DropdownItem>,
9761026
<DropdownItem
9771027
component="button"
9781028
onClick={[Function]}

src/js/App/Header/UserToggle.js

Lines changed: 6 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -44,14 +44,12 @@ function buildItems(username, isOrgAdmin, accountNumber = -1, extraItems) {
4444
rel='noopener noreferrer'>
4545
My profile
4646
</DropdownItem>,
47-
...window.insights.chrome.isBeta() ? [
48-
<DropdownItem
49-
key="User preferences"
50-
href="./user-preferences/email"
51-
>
52-
User preferences
53-
</DropdownItem>
54-
] : [],
47+
<DropdownItem
48+
key="User preferences"
49+
href="./user-preferences/email"
50+
>
51+
User preferences
52+
</DropdownItem>,
5553
<DropdownItem
5654
key="logout"
5755
component="button"

0 commit comments

Comments
 (0)