feat(InviteAgents): new component

[VadymBezpalko]

Description

New component: FIGMA

Storybook

https://feature-invite-agents--613a8e945a5665003a05113b.chromatic.com

Checklist

Obligatory:

• Self review (use this as your final check for proposed changes before requesting the review)
• Add correct label
• Assign pull request with the correct issue

[mreszke]

IMO it would be better to remove onSetUpChatbotClick from the props and allow to pass an array of custom elements

  <InviteAgents
    agents={agents}
    onAddTeammateClick={onAddTeammateClick}
    customElements={[
       onClick: onSetUpChatbotClick,
       icon: ChatbotIcon,
       text: 'Set up ChatBot'
    ]}
    animatedInviteButton={true}
    tooltipArrowOffset={13}
    className="invite-agents"
  />

[marcinsawicki]

Maybe this should be a part of ComponentCoreProps? 🤔

[marcinsawicki]

I think we shouldn't put the console.log in the docs. Simple () => {} is enough imo 🤔

[marcinsawicki]

IMO a request to change copy here or elsewhere will appear sooner rather than later. 😅 Shouldn't we leave this open for custom string? 🤔

[VadymBezpalko]

There are at least a few different places with this copy, so if we want to move it to props then other places should be covered too, so I'd leave it like that for now

[marcinsawicki]

extend ComponentCoreProps? 🤔

[marcinsawicki]

left some comments

[VadymBezpalko]

IMO it would be better to remove onSetUpChatbotClick from the props and allow to pass an array of custom elements

  <InviteAgents
    agents={agents}
    onAddTeammateClick={onAddTeammateClick}
    customElements={[
       onClick: onSetUpChatbotClick,
       icon: ChatbotIcon,
       text: 'Set up ChatBot'
    ]}
    animatedInviteButton={true}
    tooltipArrowOffset={13}
    className="invite-agents"
  />

@mreszke The problem is that we have a 'Set up ChatBot' CTA in a dedicated tooltip, so it needs to be passed either way.