Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
| successMessage: 'The provider has been added to your account', | ||
| title: 'Add connected account', | ||
| }, | ||
| enterpriseAccountPage: { |
There was a problem hiding this comment.
Can we generate the translations on all localizations file via LLM prompt?
| isLoading={card.loadingMetadata === loadingKey} | ||
| focusRing={false} | ||
| closeAfterClick={false} | ||
| localizationKey={localizationKeys('userProfile.connectedAccountPage.socialButtonsBlockButton', { |
There was a problem hiding this comment.
| localizationKey={localizationKeys('userProfile.connectedAccountPage.socialButtonsBlockButton', { | |
| localizationKey={localizationKeys('userProfile.enterpriseAccountPage.socialButtonsBlockButton', { |
Let's use an specific translation key for the enterprise accounts section
| ); | ||
| }; | ||
|
|
||
| const AddEnterpriseAccount = ({ |
There was a problem hiding this comment.
| const AddEnterpriseAccount = ({ | |
| const ConnectEnterpriseAccounts = ({ |
Since this component receives a list of enterprise accounts + it has a inner menu to connect to it
| user?.getEnterpriseConnections?.().then(connections => { | ||
| setEnterpriseConnections(connections.filter(c => c.allowAccountLinking)); | ||
| }); |
There was a problem hiding this comment.
We should use our data fetching hooks for it
| ); | ||
|
|
||
| if (!activeEnterpriseAccounts?.length) { | ||
| const hasActiveAccounts = Boolean(activeEnterpriseAccounts?.length); |
There was a problem hiding this comment.
We can make this simpler by using some above on activeEnterpriseAccounts
|
|
||
| if (!activeEnterpriseAccounts?.length) { | ||
| const hasActiveAccounts = Boolean(activeEnterpriseAccounts?.length); | ||
| const hasAvailableConnections = enterpriseConnections.length > 0; |
There was a problem hiding this comment.
| const hasAvailableConnections = enterpriseConnections.length > 0; | |
| const hasAvailableConnectionsForAccountLinking = enterpriseConnectionsForAccountLinking.length > 0; |
2 participants
Description
Checklist
pnpm testruns as expected.pnpm buildruns as expected.Type of change