Skip to content

Commit

Permalink
Merge pull request #18008 from dukenv0307/fix/16925
Browse files Browse the repository at this point in the history
Fix: Convert workspace name to lower case before sort
  • Loading branch information
madmax330 authored Apr 27, 2023
2 parents 28ce89a + 4661e01 commit 0262812
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/pages/settings/InitialSettingsPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -159,7 +159,7 @@ class InitialSettingsPage extends React.Component {
getDefaultMenuItems() {
const policiesAvatars = _.chain(this.props.policies)
.filter(policy => PolicyUtils.shouldShowPolicy(policy, this.props.network.isOffline))
.sortBy(policy => policy.name)
.sortBy(policy => policy.name.toLowerCase())
.map(policy => ({
source: policy.avatar || ReportUtils.getDefaultWorkspaceAvatar(policy.name),
name: policy.name,
Expand Down
2 changes: 1 addition & 1 deletion src/pages/workspace/WorkspacesListPage.js
Original file line number Diff line number Diff line change
Expand Up @@ -137,7 +137,7 @@ class WorkspacesListPage extends Component {
dismissError: () => dismissWorkspaceError(policy.id, policy.pendingAction),
disabled: policy.pendingAction === CONST.RED_BRICK_ROAD_PENDING_ACTION.DELETE,
}))
.sortBy(policy => policy.title)
.sortBy(policy => policy.title.toLowerCase())
.value();
}

Expand Down

0 comments on commit 0262812

Please sign in to comment.