Skip to content

Commit

Permalink
Merge pull request #11831 from Expensify/amechler-properly-generate-p…
Browse files Browse the repository at this point in the history
…olicy-name

Fix policy name when creating a workspace for another user
  • Loading branch information
Luke9389 authored Oct 13, 2022
2 parents d60f8a6 + c3c0034 commit 7d7bc4c
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions src/libs/actions/Policy.js
Original file line number Diff line number Diff line change
Expand Up @@ -715,10 +715,11 @@ function clearDeleteWorkspaceError(policyID) {

/**
* Generate a policy name based on an email and policy list.
* @param {String} [email] the email to base the workspace name on. If not passed, will use the logged in user's email instead
* @returns {String}
*/
function generateDefaultWorkspaceName() {
const emailParts = sessionEmail.split('@');
function generateDefaultWorkspaceName(email = '') {
const emailParts = email ? email.split('@') : sessionEmail.split('@');
let defaultWorkspaceName = '';
if (!emailParts || emailParts.length !== 2) {
return defaultWorkspaceName;
Expand Down Expand Up @@ -769,7 +770,7 @@ function generatePolicyID() {
*/
function createWorkspace(ownerEmail = '', makeMeAdmin = false) {
const policyID = generatePolicyID();
const workspaceName = generateDefaultWorkspaceName();
const workspaceName = generateDefaultWorkspaceName(ownerEmail);

const {
announceChatReportID,
Expand Down

0 comments on commit 7d7bc4c

Please sign in to comment.