diff --git a/pro/src/pages/Home/Offerers/OffererDetails.tsx b/pro/src/pages/Home/Offerers/OffererDetails.tsx index 3eece90d48c..914e6b347fa 100644 --- a/pro/src/pages/Home/Offerers/OffererDetails.tsx +++ b/pro/src/pages/Home/Offerers/OffererDetails.tsx @@ -72,19 +72,20 @@ export const OffererDetails = ({
- { - logEvent(OffererLinkEvents.CLICKED_INVITE_COLLABORATOR, { - offererId: selectedOffererId ?? undefined, - }) - }} - > - Inviter - + {isUserOffererValidated && ( + { + logEvent(OffererLinkEvents.CLICKED_INVITE_COLLABORATOR, { + offererId: selectedOffererId ?? undefined, + }) + }} + > + Inviter + + )}
diff --git a/pro/src/pages/Home/VenueOfferSteps/VenueOfferSteps.tsx b/pro/src/pages/Home/VenueOfferSteps/VenueOfferSteps.tsx index dbb75bdad50..4cf9c7e10e9 100644 --- a/pro/src/pages/Home/VenueOfferSteps/VenueOfferSteps.tsx +++ b/pro/src/pages/Home/VenueOfferSteps/VenueOfferSteps.tsx @@ -131,7 +131,6 @@ export const VenueOfferSteps = ({ {venue && !venue.hasCreatedOffer && ( { ).toBeInTheDocument() }) - it('should display disabled link to eac informations if venue doesnt have adage id', () => { - vi.spyOn( - venueUtils, - 'shouldDisplayEACInformationSectionForVenue' - ).mockReturnValue(true) - props.venue = { ...defaultGetOffererVenueResponseModel } - props.venue = { - ...defaultGetOffererVenueResponseModel, - hasAdageId: false, - } - - renderVenueOfferSteps(props) - expect( - screen.getByRole('link', { - name: 'Renseigner mes informations à destination des enseignants Action non disponible', - }) - ).toBeInTheDocument() - }) - it('should not display dms link if condition to display it is false', () => { props.hasVenue = false diff --git a/pro/src/ui-kit/Button/ButtonLink.tsx b/pro/src/ui-kit/Button/ButtonLink.tsx index 07ce9a3ecb3..afa6fe3fd8a 100644 --- a/pro/src/ui-kit/Button/ButtonLink.tsx +++ b/pro/src/ui-kit/Button/ButtonLink.tsx @@ -9,7 +9,6 @@ import styles from './Button.module.scss' import { ButtonVariant, IconPositionEnum, SharedButtonProps } from './types' export type LinkProps = { - isDisabled?: boolean svgAlt?: string isExternal?: boolean to: string @@ -29,7 +28,6 @@ export const ButtonLink = forwardRef( className, children, icon, - isDisabled = false, onClick, variant = ButtonVariant.TERNARY, iconPosition = IconPositionEnum.LEFT, @@ -46,7 +44,6 @@ export const ButtonLink = forwardRef( styles['button'], styles[`button-${variant}`], styles[`button-${iconPosition}`], - { [styles[`button-disabled`]]: isDisabled }, styles['button-link'], className ) @@ -85,14 +82,7 @@ export const ButtonLink = forwardRef( // for internal links so that developers can't make mistakes/forget to add the slash const absoluteUrl = isExternal || to.startsWith('/') ? to : `/${to}` - const callback: MouseEventHandler = (e) => - isDisabled ? e.preventDefault() : onClick?.(e) - - const disabled = isDisabled ? ( - Action non disponible - ) : ( - <> - ) + const callback: MouseEventHandler = (e) => onClick?.(e) body = isExternal ? ( {body} - {disabled} ) : ( {body} - {disabled} ) diff --git a/pro/src/ui-kit/Button/__specs__/ButtonLink.spec.tsx b/pro/src/ui-kit/Button/__specs__/ButtonLink.spec.tsx index fe4e51b2335..78f5ed89609 100644 --- a/pro/src/ui-kit/Button/__specs__/ButtonLink.spec.tsx +++ b/pro/src/ui-kit/Button/__specs__/ButtonLink.spec.tsx @@ -72,21 +72,6 @@ describe('ButtonLink', () => { expect(button).toHaveAttribute('href', '/offers') }) - it('should not call callback action when button disabled', async () => { - const onClick = vi.fn() - render( - - test - - ) - - const button = screen.getByRole('link', { name: /test/ }) - await userEvent.click(button) - - expect(onClick).not.toHaveBeenCalled() - expect(screen.getByText('Action non disponible')).toBeInTheDocument() - }) - it('should have right attributes for open in new tab', () => { render(