Skip to content

Commit

Permalink
document OptionListUtils helper function and rename
Browse files Browse the repository at this point in the history
  • Loading branch information
Julesssss committed May 3, 2023
1 parent dd89377 commit 8042ffb
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 3 deletions.
11 changes: 9 additions & 2 deletions src/libs/OptionsListUtils.js
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,14 @@ function addSMSDomainIfPhoneNumber(login) {
return login;
}

function getAvatarsForParticipants(logins, personalDetails) {
/**
* Collect avatar data for a list of user logins
*
* @param {Array<String>} logins
* @param {Object} personalDetails
* @returns {Object}
*/
function getAvatarsForLogins(logins, personalDetails) {
return _.map(logins, (login) => {
const userPersonalDetail = lodashGet(personalDetails, login, {login: login, avatar: ''});
return {
Expand Down Expand Up @@ -901,7 +908,7 @@ function getHeaderMessage(hasSelectableOptions, hasUserToInvite, searchValue, ma

export {
addSMSDomainIfPhoneNumber,
getAvatarsForParticipants,
getAvatarsForLogins,
isCurrentUser,
getSearchOptions,
getNewChatOptions,
Expand Down
2 changes: 1 addition & 1 deletion src/pages/workspace/WorkspaceInviteMessagePage.js
Original file line number Diff line number Diff line change
Expand Up @@ -166,7 +166,7 @@ class WorkspaceInviteMessagePage extends React.Component {
<View style={[styles.mv4, styles.justifyContentCenter, styles.alignItemsCenter]}>
<MultipleAvatars
size={CONST.AVATAR_SIZE.LARGE}
icons={OptionsListUtils.getAvatarsForParticipants(this.props.invitedMembersDraft)}
icons={OptionsListUtils.getAvatarsForLogins(this.props.invitedMembersDraft)}
shouldStackHorizontally
secondAvatarStyle={[
styles.secondAvatarInline,
Expand Down

0 comments on commit 8042ffb

Please sign in to comment.