From 98aab5008a017dd1b45dd0d8db70faae1333caa9 Mon Sep 17 00:00:00 2001
From: gmeigniez-pass <139768952+gmeigniez-pass@users.noreply.github.com>
Date: Fri, 31 Jan 2025 15:02:30 +0100
Subject: [PATCH 1/2] (PC-34043)[PRO] feat: Let users create a digital offer on
 a physical venue.

---
 .../commons/__specs__/utils.spec.tsx          | 100 ++++++----------
 .../IndividualOfferDetails/commons/utils.ts   |  54 ++-------
 .../components/DetailsForm/DetailsForm.tsx    |  32 +++--
 .../IndividualOfferDetailsScreen.spec.tsx     |  37 +++++-
 .../IndividualOfferDetailsScreen.tsx          |  25 ++--
 .../UsefulInformationForm.tsx                 |   2 +-
 .../__specs__/getFilteredVenueList.spec.ts    | 112 ------------------
 .../commons/getFilteredVenueList.ts           |  39 ------
 8 files changed, 112 insertions(+), 289 deletions(-)
 delete mode 100644 pro/src/pages/IndividualOffer/commons/__specs__/getFilteredVenueList.spec.ts
 delete mode 100644 pro/src/pages/IndividualOffer/commons/getFilteredVenueList.ts

diff --git a/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/__specs__/utils.spec.tsx b/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/__specs__/utils.spec.tsx
index b6e6791c7d7..3f0586ef258 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/__specs__/utils.spec.tsx
+++ b/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/__specs__/utils.spec.tsx
@@ -12,10 +12,9 @@ import {
   buildShowSubTypeOptions,
   completeSubcategoryConditionalFields,
   buildSubcategoryOptions,
-  buildVenueOptions,
+  formatVenuesOptions,
   deSerializeDurationMinutes,
   serializeExtraData,
-  setDefaultInitialValues,
   setDefaultInitialValuesFromOffer,
   setFormReadOnlyFields,
   serializeDetailsPostData,
@@ -150,69 +149,46 @@ describe('buildSubcategoryFields', () => {
   })
 })
 
-describe('buildVenueOptions', () => {
-  it('should build venues options', () => {
-    expect(
-      buildVenueOptions(
-        [venueListItemFactory({}), venueListItemFactory({})],
-        false,
-        false
-      )
-    ).toStrictEqual([
-      {
-        label: 'Sélectionner le partenaire',
-        value: '',
-      },
-      {
-        label: 'Le nom du lieu 1',
-        value: '1',
-      },
-      {
-        label: 'Le nom du lieu 2',
-        value: '2',
-      },
-    ])
+describe('formatVenuesOptions', () => {
+  it('should format venues as options', () => {
+    const formattedVenuesOptions = formatVenuesOptions(
+      [
+        venueListItemFactory({ isVirtual: false, id: 10 }),
+        venueListItemFactory({ isVirtual: false, id: 3 }),
+      ],
+      false
+    )
+    expect(formattedVenuesOptions).toEqual(
+      expect.arrayContaining([expect.objectContaining({ value: '10' })])
+    )
+
+    expect(formattedVenuesOptions).toEqual(
+      expect.arrayContaining([expect.objectContaining({ value: '3' })])
+    )
   })
-})
 
-describe('setDefaultInitialValues', () => {
-  it('should set default initial values', () => {
-    expect(
-      setDefaultInitialValues({
-        filteredVenues: [venueListItemFactory({}), venueListItemFactory({})],
-        areSuggestedSubcategoriesUsed: false,
-      })
-    ).toStrictEqual({
-      author: '',
-      categoryId: '',
-      description: '',
-      durationMinutes: '',
-      ean: '',
-      gtl_id: '',
-      name: '',
-      performer: '',
-      showSubType: '',
-      showType: '',
-      speaker: '',
-      stageDirector: '',
-      subcategoryConditionalFields: [],
-      suggestedSubcategory: '',
-      subcategoryId: '',
-      venueId: '',
-      visa: '',
-      productId: '',
-      callId: '',
-    })
+  it('should exclude digital venues if a physcal venue is available', () => {
+    const formattedVenuesOptions = formatVenuesOptions(
+      [
+        venueListItemFactory({ isVirtual: true, id: 10 }),
+        venueListItemFactory({ isVirtual: false, id: 3 }),
+      ],
+      false
+    )
+    expect(formattedVenuesOptions).not.toEqual(
+      expect.arrayContaining([expect.objectContaining({ value: '10' })])
+    )
 
-    expect(
-      setDefaultInitialValues({
-        filteredVenues: [venueListItemFactory({ id: 666 })],
-        areSuggestedSubcategoriesUsed: false,
-      })
-    ).toStrictEqual(
-      expect.objectContaining({
-        venueId: '666',
-      })
+    expect(formattedVenuesOptions).toEqual(
+      expect.arrayContaining([expect.objectContaining({ value: '3' })])
+    )
+
+    const formattedDigitalVenuesOptions = formatVenuesOptions(
+      [venueListItemFactory({ isVirtual: true, id: 10 })],
+      true
+    )
+    expect(formattedDigitalVenuesOptions).toEqual(
+      expect.arrayContaining([expect.objectContaining({ value: '10' })])
     )
   })
 })
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/utils.ts b/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/utils.ts
index 58c551d5c90..dcb1b0365cb 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/utils.ts
+++ b/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/utils.ts
@@ -179,58 +179,22 @@ export const onSubcategoryChange = async ({
   })
 }
 
-export const buildVenueOptions = (
+export const formatVenuesOptions = (
   venues: VenueListItemResponseModel[],
-  areSuggestedCategoriesEnabled: boolean,
-  isOfferAddressEnabled: boolean
+  isOnline: boolean
 ) => {
-  let venueOptions = venues
-    .filter((venue) => !areSuggestedCategoriesEnabled || !venue.isVirtual)
+  //  We want to display the virtual venues only if there are no physical venues available
+  //  We also want to prevent selecting a virtual venue for a physical offer form
+  const hasAtLeastOnePhysicalVenue = venues.some((v) => !v.isVirtual)
+  return venues
+    .filter((venue) =>
+      hasAtLeastOnePhysicalVenue || !isOnline ? !venue.isVirtual : true
+    )
     .map((venue) => ({
       value: venue.id.toString(),
       label: computeVenueDisplayName(venue),
     }))
     .sort((a, b) => a.label.localeCompare(b.label, 'fr'))
-  if (venueOptions.length > 1) {
-    venueOptions = [
-      {
-        value: '',
-        label: isOfferAddressEnabled
-          ? 'Sélectionner la structure'
-          : 'Sélectionner le partenaire',
-      },
-      ...venueOptions,
-    ]
-  }
-
-  return venueOptions
-}
-
-type SetDefaultInitialValuesProps = {
-  filteredVenues: VenueListItemResponseModel[]
-  areSuggestedSubcategoriesUsed: boolean
-}
-
-export function setDefaultInitialValues({
-  filteredVenues,
-  areSuggestedSubcategoriesUsed,
-}: SetDefaultInitialValuesProps): DetailsFormValues {
-  let venueId = ''
-
-  const venues = areSuggestedSubcategoriesUsed
-    ? filteredVenues.filter((v) => !v.isVirtual)
-    : filteredVenues
-
-  if (venues.length === 1) {
-    venueId = String(venues[0].id)
-  } else if (venues.length === 0 && filteredVenues.length > 0) {
-    venueId = String(filteredVenues[0].id)
-  }
-
-  return {
-    ...DEFAULT_DETAILS_FORM_VALUES,
-    venueId,
-  }
 }
 
 type SetDefaultInitialValuesFromOfferProps = {
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/DetailsForm/DetailsForm.tsx b/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/DetailsForm/DetailsForm.tsx
index 681d904c888..3002cbcf0e6 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/DetailsForm/DetailsForm.tsx
+++ b/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/DetailsForm/DetailsForm.tsx
@@ -7,7 +7,6 @@ import {
   CategoryResponseModel,
   SubcategoryResponseModel,
   SuggestedSubcategoriesResponseModel,
-  VenueListItemResponseModel,
 } from 'apiClient/v1'
 import { useIndividualOfferContext } from 'commons/context/IndividualOfferContext/IndividualOfferContext'
 import { CATEGORY_STATUS } from 'commons/core/Offers/constants'
@@ -19,10 +18,7 @@ import { OnImageUploadArgs } from 'components/ImageUploader/components/ButtonIma
 import fullMoreIcon from 'icons/full-more.svg'
 import { DEFAULT_DETAILS_FORM_VALUES } from 'pages/IndividualOffer/IndividualOfferDetails/commons/constants'
 import { DetailsFormValues } from 'pages/IndividualOffer/IndividualOfferDetails/commons/types'
-import {
-  buildVenueOptions,
-  isSubCategoryCD,
-} from 'pages/IndividualOffer/IndividualOfferDetails/commons/utils'
+import { isSubCategoryCD } from 'pages/IndividualOffer/IndividualOfferDetails/commons/utils'
 import { Callout } from 'ui-kit/Callout/Callout'
 import { CalloutVariant } from 'ui-kit/Callout/types'
 import { Select } from 'ui-kit/form/Select/Select'
@@ -38,7 +34,7 @@ const DEBOUNCE_TIME_BEFORE_REQUEST = 400
 type DetailsFormProps = {
   isEanSearchDisplayed: boolean
   isProductBased: boolean
-  filteredVenues: VenueListItemResponseModel[]
+  venuesOptions: { label: string; value: string }[]
   filteredCategories: CategoryResponseModel[]
   filteredSubcategories: SubcategoryResponseModel[]
   readonlyFields: string[]
@@ -50,7 +46,7 @@ type DetailsFormProps = {
 export const DetailsForm = ({
   isEanSearchDisplayed,
   isProductBased,
-  filteredVenues,
+  venuesOptions,
   filteredCategories,
   filteredSubcategories,
   readonlyFields: readOnlyFields,
@@ -71,12 +67,6 @@ export const DetailsForm = ({
   const { offer, subCategories } = useIndividualOfferContext()
   const offerAddressEnabled = useActiveFeature('WIP_ENABLE_OFFER_ADDRESS')
 
-  const venueOptions = buildVenueOptions(
-    filteredVenues,
-    areSuggestedSubcategoriesUsed,
-    offerAddressEnabled
-  )
-
   async function getSuggestedSubcategories() {
     if (!areSuggestedSubcategoriesUsed && !offer) {
       return
@@ -131,14 +121,14 @@ export const DetailsForm = ({
     subcategoryId !== DEFAULT_DETAILS_FORM_VALUES.subcategoryId
 
   const showAddVenueBanner =
-    !areSuggestedSubcategoriesUsed && venueOptions.length === 0
+    !areSuggestedSubcategoriesUsed && venuesOptions.length === 0
 
   const isSuggestedSubcategoryDisplayed =
     areSuggestedSubcategoriesUsed && !offer && !isProductBased
 
   const showOtherAddVenueBanner =
     areSuggestedSubcategoriesUsed &&
-    venueOptions.length === 0 &&
+    venuesOptions.length === 0 &&
     subcategory?.onlineOfflinePlatform === CATEGORY_STATUS.OFFLINE
 
   return (
@@ -166,12 +156,18 @@ export const DetailsForm = ({
         )}
         {!showAddVenueBanner && (
           <>
-            {venueOptions.length > 1 && (
+            {venuesOptions.length > 1 && (
               <FormLayout.Row>
                 <Select
                   label={offerAddressEnabled ? 'Qui propose l’offre ?' : 'Lieu'}
                   name="venueId"
-                  options={venueOptions}
+                  options={venuesOptions}
+                  defaultOption={{
+                    value: '',
+                    label: offerAddressEnabled
+                      ? 'Sélectionner la structure'
+                      : 'Sélectionner le partenaire',
+                  }}
                   onChange={async (ev) => {
                     if (isProductBased) {
                       return
@@ -181,7 +177,7 @@ export const DetailsForm = ({
                   }}
                   disabled={
                     readOnlyFields.includes('venueId') ||
-                    venueOptions.length === 1
+                    venuesOptions.length === 1
                   }
                   {...(isSuggestedSubcategoryDisplayed && {
                     'aria-controls': 'suggested-subcategories',
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/IndividualOfferDetailsScreen.spec.tsx b/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/IndividualOfferDetailsScreen.spec.tsx
index 4c14f3fca04..c74f06a821e 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/IndividualOfferDetailsScreen.spec.tsx
+++ b/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/IndividualOfferDetailsScreen.spec.tsx
@@ -1087,14 +1087,14 @@ describe('IndividualOfferDetails', () => {
     })
   })
 
-  it('should not render venue field when there is just one venue', () => {
+  it('should not render venue field when there is just one virtual venue', () => {
     vi.spyOn(useAnalytics, 'useRemoteConfigParams').mockReturnValue({
       SUGGESTED_CATEGORIES: 'true',
     })
 
     renderDetailsScreen({
       props: {
-        venues: [venueListItemFactory({ id: 189 })],
+        venues: [venueListItemFactory({ id: 189, isVirtual: true })],
       },
       contextValue,
       // There is no world where WIP_SUGGESTED_SUBCATEGORIES is enabled without WIP_SPLIT_OFFER
@@ -1104,6 +1104,39 @@ describe('IndividualOfferDetails', () => {
     expect(screen.queryByText(/Lieu/)).not.toBeInTheDocument()
   })
 
+  it('should not render venue field when there is just one physical venue', () => {
+    vi.spyOn(useAnalytics, 'useRemoteConfigParams').mockReturnValue({
+      SUGGESTED_CATEGORIES: 'true',
+    })
+
+    renderDetailsScreen({
+      props: {
+        venues: [venueListItemFactory({ id: 189, isVirtual: false })],
+      },
+      contextValue,
+    })
+
+    expect(screen.queryByText(/Lieu/)).not.toBeInTheDocument()
+  })
+
+  it('should not render venue field when there is one physical venue and one virtual venue', () => {
+    vi.spyOn(useAnalytics, 'useRemoteConfigParams').mockReturnValue({
+      SUGGESTED_CATEGORIES: 'true',
+    })
+
+    renderDetailsScreen({
+      props: {
+        venues: [
+          venueListItemFactory({ id: 189, isVirtual: false }),
+          venueListItemFactory({ id: 190, isVirtual: true }),
+        ],
+      },
+      contextValue,
+    })
+
+    expect(screen.queryByText(/Lieu/)).not.toBeInTheDocument()
+  })
+
   describe('onboarding', () => {
     beforeEach(() => {
       vi.spyOn(router, 'useNavigate').mockReturnValue(mockNavigate)
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/IndividualOfferDetailsScreen.tsx b/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/IndividualOfferDetailsScreen.tsx
index 80153231c63..da9b23cc321 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/IndividualOfferDetailsScreen.tsx
+++ b/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/IndividualOfferDetailsScreen.tsx
@@ -10,6 +10,7 @@ import { GET_OFFER_QUERY_KEY } from 'commons/config/swrQueryKeys'
 import { useIndividualOfferContext } from 'commons/context/IndividualOfferContext/IndividualOfferContext'
 import { Events } from 'commons/core/FirebaseEvents/constants'
 import {
+  CATEGORY_STATUS,
   INDIVIDUAL_OFFER_SUBTYPE,
   OFFER_WIZARD_MODE,
 } from 'commons/core/Offers/constants'
@@ -35,7 +36,6 @@ import {
   getOfferSubtypeFromParam,
   isOfferSubtypeEvent,
 } from 'pages/IndividualOffer/commons/filterCategories'
-import { getFilteredVenueListByCategoryStatus } from 'pages/IndividualOffer/commons/getFilteredVenueList'
 import { ActionBar } from 'pages/IndividualOffer/components/ActionBar/ActionBar'
 import {
   DetailsFormValues,
@@ -43,10 +43,10 @@ import {
 } from 'pages/IndividualOffer/IndividualOfferDetails/commons/types'
 import { useIndividualOfferImageUpload } from 'pages/IndividualOffer/IndividualOfferDetails/commons/useIndividualOfferImageUpload'
 import {
+  formatVenuesOptions,
   hasMusicType,
   serializeDetailsPatchData,
   serializeDetailsPostData,
-  setDefaultInitialValues,
   setDefaultInitialValuesFromOffer,
   setFormReadOnlyFields,
 } from 'pages/IndividualOffer/IndividualOfferDetails/commons/utils'
@@ -98,18 +98,23 @@ export const IndividualOfferDetailsScreen = ({
     isOfferSubtypeEvent(offerSubtype)
   )
 
-  const filteredVenues = getFilteredVenueListByCategoryStatus(
+  const availableVenuesOptions = formatVenuesOptions(
     venues,
-    categoryStatus
+    categoryStatus === CATEGORY_STATUS.ONLINE
   )
 
   const areSuggestedSubcategoriesUsed = useSuggestedSubcategoriesAbTest()
 
   const initialValues = isDirtyDraftOffer
-    ? setDefaultInitialValues({
-        filteredVenues,
-        areSuggestedSubcategoriesUsed,
-      })
+    ? //  When there is only one venue available the venueId field is not displayed
+      //  Thus we need to set the venueId programmatically
+      {
+        ...DEFAULT_DETAILS_FORM_VALUES,
+        venueId:
+          availableVenuesOptions.length === 1
+            ? availableVenuesOptions[0]?.value
+            : '',
+      }
     : setDefaultInitialValuesFromOffer({
         offer,
         subcategories: subCategories,
@@ -222,7 +227,7 @@ export const IndividualOfferDetailsScreen = ({
     : setFormReadOnlyFields(offer, isProductBased)
   const isEanSearchAvailable =
     isSearchByEanEnabled &&
-    isRecordStore(filteredVenues) &&
+    isRecordStore(venues) &&
     queryOfferType === INDIVIDUAL_OFFER_SUBTYPE.PHYSICAL_GOOD
   const isEanSearchDisplayed =
     isEanSearchAvailable &&
@@ -315,13 +320,13 @@ export const IndividualOfferDetailsScreen = ({
             <DetailsForm
               isEanSearchDisplayed={isEanSearchDisplayed}
               isProductBased={isProductBased}
-              filteredVenues={filteredVenues}
               filteredCategories={filteredCategories}
               filteredSubcategories={filteredSubcategories}
               readonlyFields={readOnlyFields}
               onImageUpload={onImageUpload}
               onImageDelete={onImageDelete}
               imageOffer={imageOffer}
+              venuesOptions={availableVenuesOptions}
             />
           </FormLayout>
           <ActionBar
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferInformations/components/UsefulInformationForm/UsefulInformationForm.tsx b/pro/src/pages/IndividualOffer/IndividualOfferInformations/components/UsefulInformationForm/UsefulInformationForm.tsx
index dfd259167e8..c24a76c372d 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferInformations/components/UsefulInformationForm/UsefulInformationForm.tsx
+++ b/pro/src/pages/IndividualOffer/IndividualOfferInformations/components/UsefulInformationForm/UsefulInformationForm.tsx
@@ -70,7 +70,7 @@ export const UsefulInformationForm = ({
 
   // we use venue is virtual here because we cannot infer it from the offerSubCategory
   // because of CATEGORY_STATUS.ONLINE_OR_OFFLINE who can be both virtual or not
-  const isVenueVirtual = venue.isVirtual || false
+  const isVenueVirtual = venue.isVirtual
 
   const {
     currentUser: { isAdmin, email },
diff --git a/pro/src/pages/IndividualOffer/commons/__specs__/getFilteredVenueList.spec.ts b/pro/src/pages/IndividualOffer/commons/__specs__/getFilteredVenueList.spec.ts
deleted file mode 100644
index 38a1dab7eec..00000000000
--- a/pro/src/pages/IndividualOffer/commons/__specs__/getFilteredVenueList.spec.ts
+++ /dev/null
@@ -1,112 +0,0 @@
-import { SubcategoryResponseModel } from 'apiClient/v1'
-import { CATEGORY_STATUS } from 'commons/core/Offers/constants'
-import {
-  subcategoryFactory,
-  venueListItemFactory,
-} from 'commons/utils/factories/individualApiFactories'
-
-import {
-  getFilteredVenueListByCategoryStatus,
-  getFilteredVenueListBySubcategory,
-} from '../getFilteredVenueList'
-
-describe('getFilteredVenueList', () => {
-  const virtualVenueId = 1
-  const secondVenueId = 2
-  const thirdVenueId = 3
-
-  const subCategories: SubcategoryResponseModel[] = [
-    subcategoryFactory({
-      id: 'ONLINE_ONLY',
-      onlineOfflinePlatform: CATEGORY_STATUS.ONLINE,
-    }),
-    subcategoryFactory({
-      id: 'OFFLINE_ONLY',
-      onlineOfflinePlatform: CATEGORY_STATUS.OFFLINE,
-    }),
-    subcategoryFactory({
-      id: 'ONLINE_OR_OFFLINE',
-      onlineOfflinePlatform: CATEGORY_STATUS.ONLINE_OR_OFFLINE,
-    }),
-  ]
-
-  const virtualVenue = venueListItemFactory({
-    id: virtualVenueId,
-    isVirtual: true,
-  })
-
-  const venueList = [
-    venueListItemFactory({
-      id: secondVenueId,
-      isVirtual: false,
-    }),
-    venueListItemFactory({
-      id: thirdVenueId,
-      isVirtual: false,
-    }),
-  ]
-
-  describe('getFilteredVenueListBySubcategory', () => {
-    it('should return all venues when subCatagory is ONLINE or OFFLINE', () => {
-      const result = getFilteredVenueListBySubcategory(
-        [...venueList, virtualVenue],
-        subCategories.find((s) => s.id === 'ONLINE_OR_OFFLINE')
-      )
-      expect(result.length).toEqual(3)
-      expect(result[0].id).toEqual(secondVenueId)
-      expect(result[1].id).toEqual(thirdVenueId)
-      expect(result[2].id).toEqual(virtualVenueId)
-    })
-
-    it('should return virtual venues when subCatagory is ONLINE only', () => {
-      const result = getFilteredVenueListBySubcategory(
-        [...venueList, virtualVenue],
-        subCategories.find((s) => s.id === 'ONLINE_ONLY')
-      )
-      expect(result.length).toEqual(1)
-      expect(result[0].id).toEqual(virtualVenueId)
-    })
-
-    it('should return not virtual when subCatagory is OFFLINE only', () => {
-      const result = getFilteredVenueListBySubcategory(
-        [...venueList, virtualVenue],
-        subCategories.find((s) => s.id === 'OFFLINE_ONLY')
-      )
-      expect(result.length).toEqual(2)
-      expect(result[0].id).toEqual(secondVenueId)
-      expect(result[1].id).toEqual(thirdVenueId)
-    })
-  })
-
-  describe('getFilteredVenueListBySubcategory', () => {
-    it('should return all venues when categoryStatus is ONLINE or OFFLINE', () => {
-      const result = getFilteredVenueListByCategoryStatus(
-        [...venueList, virtualVenue],
-        CATEGORY_STATUS.ONLINE_OR_OFFLINE
-      )
-      expect(result.length).toEqual(3)
-      expect(result[0].id).toEqual(secondVenueId)
-      expect(result[1].id).toEqual(thirdVenueId)
-      expect(result[2].id).toEqual(virtualVenueId)
-    })
-
-    it('should return vitual venues when categoryStatus is ONLINE only', () => {
-      const result = getFilteredVenueListByCategoryStatus(
-        [...venueList, virtualVenue],
-        CATEGORY_STATUS.ONLINE
-      )
-      expect(result.length).toEqual(1)
-      expect(result[0].id).toEqual(virtualVenueId)
-    })
-
-    it('should return not virtual when categoryStatus is OFFLINE only', () => {
-      const result = getFilteredVenueListByCategoryStatus(
-        [...venueList, virtualVenue],
-        CATEGORY_STATUS.OFFLINE
-      )
-      expect(result.length).toEqual(2)
-      expect(result[0].id).toEqual(secondVenueId)
-      expect(result[1].id).toEqual(thirdVenueId)
-    })
-  })
-})
diff --git a/pro/src/pages/IndividualOffer/commons/getFilteredVenueList.ts b/pro/src/pages/IndividualOffer/commons/getFilteredVenueList.ts
deleted file mode 100644
index 09f1a7fbbf9..00000000000
--- a/pro/src/pages/IndividualOffer/commons/getFilteredVenueList.ts
+++ /dev/null
@@ -1,39 +0,0 @@
-import {
-  SubcategoryResponseModel,
-  VenueListItemResponseModel,
-} from 'apiClient/v1'
-import { CATEGORY_STATUS } from 'commons/core/Offers/constants'
-
-export const getFilteredVenueListBySubcategory = (
-  venueList: VenueListItemResponseModel[],
-  subcategory?: SubcategoryResponseModel
-): VenueListItemResponseModel[] => {
-  if (!subcategory) {
-    return venueList
-  }
-
-  if (subcategory.onlineOfflinePlatform === CATEGORY_STATUS.ONLINE_OR_OFFLINE) {
-    return venueList
-  }
-
-  return venueList.filter((venue) =>
-    subcategory.onlineOfflinePlatform === CATEGORY_STATUS.ONLINE
-      ? venue.isVirtual
-      : !venue.isVirtual
-  )
-}
-
-export const getFilteredVenueListByCategoryStatus = (
-  venueList: VenueListItemResponseModel[],
-  categoryStatus: CATEGORY_STATUS
-): VenueListItemResponseModel[] => {
-  if (categoryStatus === CATEGORY_STATUS.ONLINE_OR_OFFLINE) {
-    return venueList
-  }
-
-  return venueList.filter((venue) =>
-    categoryStatus === CATEGORY_STATUS.ONLINE
-      ? venue.isVirtual
-      : !venue.isVirtual
-  )
-}

From 8f3f305a1e59247f3c37c1fc1649acbf849ae7e5 Mon Sep 17 00:00:00 2001
From: gmeigniez-pass <139768952+gmeigniez-pass@users.noreply.github.com>
Date: Tue, 4 Feb 2025 12:42:02 +0100
Subject: [PATCH 2/2] (PC-34043)[PRO] feat: Move the url field to the first
 step of individual offer creation.

---
 .../pcapi/sandboxes/scripts/getters/pro.py    |  5 +-
 .../e2e/editDigitalIndividualOffer.cy.ts      | 42 ++++++++++------
 .../utils/getOfferConditionalFields.ts        |  6 ---
 .../DetailsSummary/DetailsSummary.tsx         |  9 +++-
 .../OfferSection/OfferSection.tsx             | 19 ++++---
 .../OfferSection/__specs__/serializer.spec.ts |  1 -
 .../SummaryScreen/OfferSection/serializer.ts  |  1 -
 .../SummaryScreen/SummaryScreen.tsx           |  1 -
 .../__specs__/SummaryScreen.spec.tsx          |  8 +--
 .../UsefulInformationsSummary.tsx             |  8 +--
 .../UsefulInformationsSummary.spec.tsx        |  2 -
 .../OfferAppPreview/OfferAppPreview.tsx       |  2 +-
 .../commons/__specs__/utils.spec.tsx          |  2 +
 .../commons/constants.ts                      |  1 +
 .../IndividualOfferDetails/commons/types.ts   |  1 +
 .../IndividualOfferDetails/commons/utils.ts   |  4 ++
 .../commons/validationSchema.ts               | 24 +++++++++
 .../components/DetailsForm/DetailsForm.tsx    | 26 +++++++++-
 .../IndividualOfferDetailsScreen.spec.tsx     |  2 +
 .../IndividualOfferDetailsScreen.tsx          | 12 +++--
 .../commons/__specs__/utils.spec.ts           |  8 ---
 .../__specs__/validationSchema.spec.tsx       | 21 --------
 .../commons/constants.ts                      |  1 -
 .../commons/types.ts                          |  3 --
 .../commons/utils.ts                          |  4 +-
 .../commons/validationSchema.ts               | 29 ++---------
 .../IndividualOfferInformationsScreen.tsx     |  2 +-
 .../OfferLocation/validationSchema.ts         | 50 ++++++-------------
 .../UsefulInformationForm.tsx                 | 31 ++----------
 .../commons/__specs__/serializers.spec.ts     |  1 -
 .../pages/IndividualOffer/commons/types.ts    |  1 -
 31 files changed, 144 insertions(+), 183 deletions(-)

diff --git a/api/src/pcapi/sandboxes/scripts/getters/pro.py b/api/src/pcapi/sandboxes/scripts/getters/pro.py
index 8308803c389..13fc5ceeb68 100644
--- a/api/src/pcapi/sandboxes/scripts/getters/pro.py
+++ b/api/src/pcapi/sandboxes/scripts/getters/pro.py
@@ -90,7 +90,10 @@ def create_regular_pro_user_with_virtual_offer() -> dict:
     offerers_factories.VenueFactory(name="Mon Lieu", managingOfferer=offerer, isPermanent=True)
     virtual_venue = offerers_factories.VirtualVenueFactory(managingOfferer=offerer)
     offers_factories.OfferFactory(
-        name="Mon offre virtuelle", subcategoryId=subcategories.ABO_PLATEFORME_VIDEO.id, venue=virtual_venue
+        name="Mon offre virtuelle",
+        subcategoryId=subcategories.ABO_PLATEFORME_VIDEO.id,
+        venue=virtual_venue,
+        url="http://www.example.com",
     )
     return {"user": get_pro_user_helper(pro_user)}
 
diff --git a/pro/cypress/e2e/editDigitalIndividualOffer.cy.ts b/pro/cypress/e2e/editDigitalIndividualOffer.cy.ts
index 174c76aa8d7..91d1faf4c61 100644
--- a/pro/cypress/e2e/editDigitalIndividualOffer.cy.ts
+++ b/pro/cypress/e2e/editDigitalIndividualOffer.cy.ts
@@ -1,6 +1,9 @@
 import { addDays, format } from 'date-fns'
 
-import { sessionLogInAndGoToPage, logInAndGoToPage } from '../support/helpers.ts'
+import {
+  sessionLogInAndGoToPage,
+  logInAndGoToPage,
+} from '../support/helpers.ts'
 
 describe('Edit digital individual offers', () => {
   describe('Display and url modification', () => {
@@ -17,7 +20,11 @@ describe('Edit digital individual offers', () => {
     })
 
     it('An edited offer is displayed with 4 tabs', function () {
-      sessionLogInAndGoToPage('Session edit digital individual offer', login1, '/offre/individuelle/1/recapitulatif/details')
+      sessionLogInAndGoToPage(
+        'Session edit digital individual offer',
+        login1,
+        '/offre/individuelle/1/recapitulatif/details'
+      )
 
       cy.contains('Récapitulatif')
 
@@ -33,30 +40,40 @@ describe('Edit digital individual offers', () => {
     })
 
     it('I should be able to modify the url of a digital offer', function () {
-      sessionLogInAndGoToPage('Session edit digital individual offer', login1, '/offres')
+      sessionLogInAndGoToPage(
+        'Session edit digital individual offer',
+        login1,
+        '/offres'
+      )
 
       cy.stepLog({ message: 'I open the first offer in the list' })
       cy.findAllByTestId('offer-item-row')
         .first()
         .within(() => {
-          cy.findByRole('button', { name: 'Voir les actions'}).click()
+          cy.findByRole('button', { name: 'Voir les actions' }).click()
         })
       cy.findByRole('menuitem', { name: 'Voir l’offre' }).click()
       cy.url().should('contain', '/recapitulatif')
 
+      cy.findByRole('link', { name: 'Modifier les détails de l’offre' }).click()
+
+      cy.stepLog({ message: 'I update the url link' })
+      const randomUrl = `http://myrandomurl.fr/`
+      cy.get('input#url').type('{selectall}{del}' + randomUrl)
+
       cy.stepLog({ message: 'I display Informations pratiques tab' })
+      cy.findByText('Enregistrer les modifications').click()
+
+      cy.findByText('http://myrandomurl.fr/').should('exist')
+
       cy.findByText('Informations pratiques').click()
       cy.url().should('contain', '/pratiques')
 
       cy.stepLog({ message: 'I edit the offer displayed' })
-      cy.get('a[aria-label^="Modifier les détails de l’offre"]').click()
+      cy.findByRole('link', { name: 'Modifier les détails de l’offre' }).click()
       cy.url().should('contain', '/edition/pratiques')
 
-      cy.stepLog({ message: 'I update the url link' })
-      const randomUrl = `http://myrandomurl.fr/`
-      cy.get('input#url').type('{selectall}{del}' + randomUrl)
       cy.findByText('Enregistrer les modifications').click()
-      cy.findByText('http://myrandomurl.fr/').should('exist')
       cy.findByText('Retour à la liste des offres').click()
       cy.url().should('contain', '/offres')
       cy.findAllByTestId('spinner', { timeout: 30 * 1000 }).should('not.exist')
@@ -66,7 +83,7 @@ describe('Edit digital individual offers', () => {
       cy.findAllByTestId('offer-item-row')
         .first()
         .within(() => {
-          cy.findByRole('button', { name: 'Voir les actions'}).click()
+          cy.findByRole('button', { name: 'Voir les actions' }).click()
         })
       cy.findByRole('menuitem', { name: 'Voir l’offre' }).click()
       cy.url().should('contain', '/recapitulatif')
@@ -74,11 +91,6 @@ describe('Edit digital individual offers', () => {
       cy.stepLog({ message: 'I display Informations pratiques tab' })
       cy.findByText('Informations pratiques').click()
       cy.url().should('contain', '/pratiques')
-
-      cy.stepLog({
-        message: 'the url updated is retrieved in the details of the offer',
-      })
-      cy.contains('URL d’accès à l’offre : ' + randomUrl)
     })
   })
 
diff --git a/pro/src/commons/utils/getOfferConditionalFields.ts b/pro/src/commons/utils/getOfferConditionalFields.ts
index 51bc095cc38..273d73a8d98 100644
--- a/pro/src/commons/utils/getOfferConditionalFields.ts
+++ b/pro/src/commons/utils/getOfferConditionalFields.ts
@@ -4,14 +4,12 @@ interface GetOfferConditionalFieldsProps {
   offerSubCategory?: SubcategoryResponseModel | null
   isUserAdmin?: boolean | null
   receiveNotificationEmails?: boolean | null
-  isVenueVirtual?: boolean | null
 }
 
 export const getOfferConditionalFields = ({
   offerSubCategory = null,
   isUserAdmin = null,
   receiveNotificationEmails = null,
-  isVenueVirtual = null,
 }: GetOfferConditionalFieldsProps): string[] => {
   const offerConditionalFields = []
 
@@ -39,10 +37,6 @@ export const getOfferConditionalFields = ({
     offerConditionalFields.push('bookingEmail')
   }
 
-  if (isVenueVirtual) {
-    offerConditionalFields.push('url')
-  }
-
   if (offerSubCategory?.canBeWithdrawable) {
     offerConditionalFields.push('withdrawalType')
     offerConditionalFields.push('withdrawalDelay')
diff --git a/pro/src/components/IndividualOffer/DetailsSummary/DetailsSummary.tsx b/pro/src/components/IndividualOffer/DetailsSummary/DetailsSummary.tsx
index 760045a82b1..07111e471b1 100644
--- a/pro/src/components/IndividualOffer/DetailsSummary/DetailsSummary.tsx
+++ b/pro/src/components/IndividualOffer/DetailsSummary/DetailsSummary.tsx
@@ -59,7 +59,14 @@ export function DetailsSummaryScreen({ offer }: DetailsSummaryScreenProps) {
   const aboutDescriptions: Description[] = [
     { title: 'Titre de l’offre', text: offerData.name },
     { title: 'Description', text: offerData.description },
-  ]
+  ].concat(
+    offer.isDigital
+      ? {
+          title: 'URL d’accès à l’offre',
+          text: offer.url || ' - ',
+        }
+      : []
+  )
   const venueName = offerData.venuePublicName || offerData.venueName
   aboutDescriptions.unshift({
     title: isOfferAddressEnabled ? 'Structure' : 'Lieu',
diff --git a/pro/src/components/IndividualOffer/SummaryScreen/OfferSection/OfferSection.tsx b/pro/src/components/IndividualOffer/SummaryScreen/OfferSection/OfferSection.tsx
index 1752cd580c8..74884d94830 100644
--- a/pro/src/components/IndividualOffer/SummaryScreen/OfferSection/OfferSection.tsx
+++ b/pro/src/components/IndividualOffer/SummaryScreen/OfferSection/OfferSection.tsx
@@ -71,7 +71,14 @@ export const OfferSection = ({
   const aboutDescriptions: Description[] = [
     { title: 'Titre de l’offre', text: offerData.name },
     { title: 'Description', text: offerData.description || '-' },
-  ]
+  ].concat(
+    offer.isDigital
+      ? {
+          title: 'URL d’accès à l’offre',
+          text: offer.url || ' - ',
+        }
+      : []
+  )
   const venueName = offerData.venuePublicName || offerData.venueName
   aboutDescriptions.unshift({
     title: isOfferAddressEnabled ? 'Structure' : 'Lieu',
@@ -165,14 +172,6 @@ export const OfferSection = ({
         offerData.bookingContact || ' - ',
     })
   }
-  if (conditionalFields.includes('url')) {
-    practicalInfoDescriptions.push({
-      title: 'URL d’accès à l’offre',
-      text:
-        /* istanbul ignore next: DEBT, TO FIX */
-        offerData.url || ' - ',
-    })
-  }
 
   const artisticInformationsFields = [
     'speaker',
@@ -229,7 +228,7 @@ export const OfferSection = ({
         })}
         aria-label="Modifier les informations pratiques de l’offre"
       >
-        {!offerData.isVenueVirtual && isOfferAddressEnabled && (
+        {!offer.isDigital && isOfferAddressEnabled && (
           <SummarySubSection title="Localisation de l’offre">
             <SummaryDescriptionList
               listDataTestId="localisation-offer-details"
diff --git a/pro/src/components/IndividualOffer/SummaryScreen/OfferSection/__specs__/serializer.spec.ts b/pro/src/components/IndividualOffer/SummaryScreen/OfferSection/__specs__/serializer.spec.ts
index f38c57a4115..18427c47945 100644
--- a/pro/src/components/IndividualOffer/SummaryScreen/OfferSection/__specs__/serializer.spec.ts
+++ b/pro/src/components/IndividualOffer/SummaryScreen/OfferSection/__specs__/serializer.spec.ts
@@ -83,7 +83,6 @@ describe('routes::Summary::serializers', () => {
       venueName: 'Le nom du lieu 1',
       venuePublicName: 'Mon Lieu',
       venueDepartmentCode: '78',
-      isVenueVirtual: false,
       offererName: 'Le nom de la structure 1',
       bookingEmail: 'booking@email.com',
       bookingContact: 'alfonsoLeBg@exampple.com',
diff --git a/pro/src/components/IndividualOffer/SummaryScreen/OfferSection/serializer.ts b/pro/src/components/IndividualOffer/SummaryScreen/OfferSection/serializer.ts
index 54e065a6315..7cab23ad244 100644
--- a/pro/src/components/IndividualOffer/SummaryScreen/OfferSection/serializer.ts
+++ b/pro/src/components/IndividualOffer/SummaryScreen/OfferSection/serializer.ts
@@ -116,7 +116,6 @@ export const serializeOfferSectionData = (
     venueName: offer.venue.name,
     venuePublicName: offer.venue.publicName,
     venueDepartmentCode: offer.venue.departementCode,
-    isVenueVirtual: offer.venue.isVirtual,
     offererName: offer.venue.managingOfferer.name,
     bookingEmail: offer.bookingEmail,
     bookingContact: offer.bookingContact,
diff --git a/pro/src/components/IndividualOffer/SummaryScreen/SummaryScreen.tsx b/pro/src/components/IndividualOffer/SummaryScreen/SummaryScreen.tsx
index 30195cf2487..f25e1aec316 100644
--- a/pro/src/components/IndividualOffer/SummaryScreen/SummaryScreen.tsx
+++ b/pro/src/components/IndividualOffer/SummaryScreen/SummaryScreen.tsx
@@ -134,7 +134,6 @@ export const SummaryScreen = () => {
     offerSubCategory,
     isUserAdmin: false,
     receiveNotificationEmails: true,
-    isVenueVirtual: offer.venue.isVirtual,
   })
   const subCategoryConditionalFields = offerSubCategory
     ? offerSubCategory.conditionalFields
diff --git a/pro/src/components/IndividualOffer/SummaryScreen/__specs__/SummaryScreen.spec.tsx b/pro/src/components/IndividualOffer/SummaryScreen/__specs__/SummaryScreen.spec.tsx
index 2641d3d7b62..81a6d20d09e 100644
--- a/pro/src/components/IndividualOffer/SummaryScreen/__specs__/SummaryScreen.spec.tsx
+++ b/pro/src/components/IndividualOffer/SummaryScreen/__specs__/SummaryScreen.spec.tsx
@@ -133,6 +133,7 @@ describe('Summary', () => {
         description: 'ma description',
         subcategoryId: SubcategoryIdEnum.CONCERT,
         url: 'https://offer-url.example.com',
+        isDigital: true,
         withdrawalDetails: 'détails de retrait',
         bookingEmail: 'booking@example.com',
         venue: getOfferVenueFactory({
@@ -165,18 +166,12 @@ describe('Summary', () => {
     expect(
       screen.getByText('Notifications des réservations')
     ).toBeInTheDocument()
-    expect(
-      screen.getByText('URL d’accès à l’offre', { exact: false })
-    ).toBeInTheDocument()
     expect(screen.getByText('Aperçu dans l’app')).toBeInTheDocument()
 
     expect(screen.getByText(categories[0].proLabel)).toBeInTheDocument()
     expect(screen.getByText(subCategories[0].proLabel)).toBeInTheDocument()
     expect(screen.getByText('ma venue (nom public)')).toBeInTheDocument()
     expect(screen.getByText('détails de retrait')).toBeInTheDocument()
-    expect(
-      screen.getByText('https://offer-url.example.com')
-    ).toBeInTheDocument()
     expect(screen.getByText('Non accessible')).toBeInTheDocument()
     expect(screen.getByText('booking@example.com')).toBeInTheDocument()
     expect(screen.getAllByText('mon offre')).toHaveLength(2)
@@ -218,6 +213,7 @@ describe('Summary', () => {
         screen.queryByText('Visualiser dans l’app')
       ).not.toBeInTheDocument()
     })
+
     it('should render component with new sections', async () => {
       vi.spyOn(api, 'getOfferer').mockResolvedValue(
         defaultGetOffererResponseModel
diff --git a/pro/src/components/IndividualOffer/UsefulInformationsSummaryScreen/UsefulInformationsSummary.tsx b/pro/src/components/IndividualOffer/UsefulInformationsSummaryScreen/UsefulInformationsSummary.tsx
index 01b304fe0dd..818ee452ed0 100644
--- a/pro/src/components/IndividualOffer/UsefulInformationsSummaryScreen/UsefulInformationsSummary.tsx
+++ b/pro/src/components/IndividualOffer/UsefulInformationsSummaryScreen/UsefulInformationsSummary.tsx
@@ -52,12 +52,6 @@ export function UsefulInformationsSummaryScreen({
       text: offer.bookingContact || ' - ',
     })
   }
-  if (offer.venue.isVirtual) {
-    practicalInfoDescriptions.push({
-      title: 'URL d’accès à l’offre',
-      text: offer.url || ' - ',
-    })
-  }
   return (
     <SummaryLayout>
       <SummaryContent>
@@ -70,7 +64,7 @@ export function UsefulInformationsSummaryScreen({
           })}
           aria-label="Modifier les détails de l’offre"
         >
-          {!offer.venue.isVirtual && isOfferAddressEnabled && (
+          {!offer.isDigital && isOfferAddressEnabled && (
             <SummarySubSection title="Localisation de l’offre">
               <SummaryDescriptionList
                 descriptions={[
diff --git a/pro/src/components/IndividualOffer/UsefulInformationsSummaryScreen/__specs__/UsefulInformationsSummary.spec.tsx b/pro/src/components/IndividualOffer/UsefulInformationsSummaryScreen/__specs__/UsefulInformationsSummary.spec.tsx
index 4db7890bcb1..76f0c11e450 100644
--- a/pro/src/components/IndividualOffer/UsefulInformationsSummaryScreen/__specs__/UsefulInformationsSummary.spec.tsx
+++ b/pro/src/components/IndividualOffer/UsefulInformationsSummaryScreen/__specs__/UsefulInformationsSummary.spec.tsx
@@ -44,7 +44,6 @@ describe('UsefulInformationsSummaryScreen', () => {
       withdrawalDelay: 120,
       bookingContact: 'robert@exemple.com',
       venue: { ...getOfferVenueFactory(), isVirtual: true },
-      url: 'https://www.example.com',
     })
 
     renderUsefulInformationsSummaryScreen(offer)
@@ -57,7 +56,6 @@ describe('UsefulInformationsSummaryScreen', () => {
       screen.getByText('2 minutes avant le début de l’évènement')
     ).toBeInTheDocument()
     expect(screen.getByText('robert@exemple.com')).toBeInTheDocument()
-    expect(screen.getByText('https://www.example.com')).toBeInTheDocument()
   })
 
   it('should render summary with right field with OA FF', async () => {
diff --git a/pro/src/components/OfferAppPreview/OfferAppPreview.tsx b/pro/src/components/OfferAppPreview/OfferAppPreview.tsx
index c3104269002..a93ee7d88cf 100644
--- a/pro/src/components/OfferAppPreview/OfferAppPreview.tsx
+++ b/pro/src/components/OfferAppPreview/OfferAppPreview.tsx
@@ -58,7 +58,7 @@ export const OfferAppPreview = ({
           </div>
         )}
 
-        {!venue.isVirtual && (
+        {!offer.isDigital && (
           <VenueDetails
             venue={venue}
             address={isOfferAddressEnabled ? offer.address : undefined}
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/__specs__/utils.spec.tsx b/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/__specs__/utils.spec.tsx
index 3f0586ef258..a9201b53bf2 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/__specs__/utils.spec.tsx
+++ b/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/__specs__/utils.spec.tsx
@@ -224,6 +224,7 @@ describe('setDefaultInitialValuesFromOffer', () => {
       visa: 'USA',
       productId: '',
       callId: '',
+      url: undefined,
     })
   })
 })
@@ -412,6 +413,7 @@ describe('serializeDetailsPostData', () => {
         visa: '123456789',
         ean: 'any ean',
       },
+      url: undefined,
       productId: undefined,
       callId: '1',
     })
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/constants.ts b/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/constants.ts
index f0acc95fe08..b2c7e152523 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/constants.ts
+++ b/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/constants.ts
@@ -18,4 +18,5 @@ export const DEFAULT_DETAILS_FORM_VALUES = {
   suggestedSubcategory: '',
   productId: '',
   callId: '',
+  url: null,
 }
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/types.ts b/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/types.ts
index 194b8b1f67d..f29a05ebfcc 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/types.ts
+++ b/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/types.ts
@@ -19,6 +19,7 @@ export type DetailsFormValues = {
   suggestedSubcategory: string
   productId: string
   callId?: string
+  url?: string | null
 }
 
 export type Product = {
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/utils.ts b/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/utils.ts
index dcb1b0365cb..f528303fa6f 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/utils.ts
+++ b/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/utils.ts
@@ -244,6 +244,7 @@ export function setDefaultInitialValuesFromOffer({
       DEFAULT_DETAILS_FORM_VALUES.stageDirector,
     productId:
       offer.productId?.toString() ?? DEFAULT_DETAILS_FORM_VALUES.productId,
+    url: offer.url,
   }
 }
 
@@ -320,6 +321,7 @@ export function serializeDetailsPostData(
     extraData: serializeExtraData(formValues),
     productId: formValues.productId ? Number(formValues.productId) : undefined,
     callId: formValues.callId,
+    url: formValues.url,
   })
 }
 
@@ -329,6 +331,7 @@ type PatchPayload = {
   extraData?: Record<string, unknown>
   name: string
   subcategoryId: string
+  url?: string | null
 }
 
 export function serializeDetailsPatchData(
@@ -340,5 +343,6 @@ export function serializeDetailsPatchData(
     description: formValues.description,
     durationMinutes: serializeDurationMinutes(formValues.durationMinutes ?? ''),
     extraData: serializeExtraData(formValues),
+    url: formValues.url,
   }
 }
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/validationSchema.ts b/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/validationSchema.ts
index 2cfb87ab896..7ba5654a689 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/validationSchema.ts
+++ b/pro/src/pages/IndividualOffer/IndividualOfferDetails/commons/validationSchema.ts
@@ -10,10 +10,20 @@ const eanValidation = yup
     test: (ean) => ean === undefined || ean.length === 13,
   })
 
+// TODO: this regex is subject to backtracking which can lead to "catastrophic backtracking", high memory usage and slow performance
+// we cannot use the yup url validation because we need to allow {} in the url to interpolate some data
+const offerFormUrlRegex = new RegExp(
+  /*eslint-disable-next-line no-useless-escape*/
+  /^(http:\/\/www\.|https:\/\/www\.|http:\/\/|https:\/\/)(([a-z0-9]+([\-\.\.-\.@_a-z0-9]+)*\.[a-z]{2,5})|((25[0-5]|(2[0-4]|1\d|[1-9]|)\d)\.){3}(25[0-5]|(2[0-4]|1\d|[1-9]|)\d))(:[0-9]{1,5})?\S*?$/,
+  'i'
+)
+
 export const getValidationSchema = ({
   isOfferAddressEnabled = false,
+  isDigitalOffer = false,
 }: {
   isOfferAddressEnabled: boolean
+  isDigitalOffer: boolean
 }) => {
   return yup.object().shape({
     name: yup.string().trim().max(90).required('Veuillez renseigner un titre'),
@@ -71,6 +81,20 @@ export const getValidationSchema = ({
           ? 'Veuillez sélectionner une structure'
           : 'Veuillez sélectionner un lieu'
       ),
+    url: isDigitalOffer
+      ? yup
+          .string()
+          .required(
+            'Veuillez renseigner une URL valide. Ex : https://exemple.com'
+          )
+          .test({
+            name: 'url',
+            message:
+              'Veuillez renseigner une URL valide. Ex : https://exemple.com',
+            test: (url?: string) =>
+              url ? url.match(offerFormUrlRegex) !== null : true,
+          })
+      : yup.string().nullable(),
   })
 }
 
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/DetailsForm/DetailsForm.tsx b/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/DetailsForm/DetailsForm.tsx
index 3002cbcf0e6..cb64ef9a20c 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/DetailsForm/DetailsForm.tsx
+++ b/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/DetailsForm/DetailsForm.tsx
@@ -24,6 +24,7 @@ import { CalloutVariant } from 'ui-kit/Callout/types'
 import { Select } from 'ui-kit/form/Select/Select'
 import { TextArea } from 'ui-kit/form/TextArea/TextArea'
 import { TextInput } from 'ui-kit/form/TextInput/TextInput'
+import { InfoBox } from 'ui-kit/InfoBox/InfoBox'
 
 import { DetailsSubForm } from './DetailsSubForm/DetailsSubForm'
 import { Subcategories } from './Subcategories/Subcategories'
@@ -37,10 +38,11 @@ type DetailsFormProps = {
   venuesOptions: { label: string; value: string }[]
   filteredCategories: CategoryResponseModel[]
   filteredSubcategories: SubcategoryResponseModel[]
-  readonlyFields: string[]
+  readOnlyFields: string[]
   onImageUpload: (values: OnImageUploadArgs) => Promise<void>
   onImageDelete: () => Promise<void>
   imageOffer?: IndividualOfferImage
+  categoryStatus: CATEGORY_STATUS
 }
 
 export const DetailsForm = ({
@@ -49,10 +51,11 @@ export const DetailsForm = ({
   venuesOptions,
   filteredCategories,
   filteredSubcategories,
-  readonlyFields: readOnlyFields,
+  readOnlyFields,
   onImageUpload,
   onImageDelete,
   imageOffer,
+  categoryStatus,
 }: DetailsFormProps): JSX.Element => {
   const areSuggestedSubcategoriesUsed = useSuggestedSubcategoriesAbTest()
   const [hasSuggestionsApiBeenCalled, setHasSuggestionsApiBeenCalled] =
@@ -211,6 +214,25 @@ export const DetailsForm = ({
                 })}
               />
             </FormLayout.Row>
+            {(categoryStatus === CATEGORY_STATUS.ONLINE ||
+              offer?.isDigital) && (
+              <FormLayout.Row
+                sideComponent={
+                  <InfoBox>
+                    Lien vers lequel seront renvoyés les bénéficiaires ayant
+                    réservé votre offre sur l’application pass Culture.
+                  </InfoBox>
+                }
+              >
+                <TextInput
+                  label="URL d’accès à l’offre"
+                  name="url"
+                  type="text"
+                  description="Format : https://exemple.com"
+                  disabled={readOnlyFields.includes('url')}
+                />
+              </FormLayout.Row>
+            )}
           </>
         )}
       </FormLayout.Section>
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/IndividualOfferDetailsScreen.spec.tsx b/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/IndividualOfferDetailsScreen.spec.tsx
index c74f06a821e..030a5e2ec64 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/IndividualOfferDetailsScreen.spec.tsx
+++ b/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/IndividualOfferDetailsScreen.spec.tsx
@@ -470,6 +470,7 @@ describe('IndividualOfferDetails', () => {
       subcategoryId: 'physical',
       venueId: 189,
       callId: '',
+      url: null,
     })
     expect(mockNavigate).toHaveBeenCalledWith(
       '/offre/individuelle/12/creation/details',
@@ -668,6 +669,7 @@ describe('IndividualOfferDetails', () => {
         subcategoryId: 'physical',
         venueId: 189,
         callId: MOCK_DATA.suggestedSubcatCallId,
+        url: null,
       })
     })
   })
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/IndividualOfferDetailsScreen.tsx b/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/IndividualOfferDetailsScreen.tsx
index da9b23cc321..ff165f6d98c 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/IndividualOfferDetailsScreen.tsx
+++ b/pro/src/pages/IndividualOffer/IndividualOfferDetails/components/IndividualOfferDetailsScreen.tsx
@@ -100,7 +100,7 @@ export const IndividualOfferDetailsScreen = ({
 
   const availableVenuesOptions = formatVenuesOptions(
     venues,
-    categoryStatus === CATEGORY_STATUS.ONLINE
+    categoryStatus === CATEGORY_STATUS.ONLINE || Boolean(offer?.isDigital)
   )
 
   const areSuggestedSubcategoriesUsed = useSuggestedSubcategoriesAbTest()
@@ -194,10 +194,13 @@ export const IndividualOfferDetailsScreen = ({
       formik.setStatus('apiError')
     }
   }
-
   const formik = useFormik({
     initialValues,
-    validationSchema: getValidationSchema({ isOfferAddressEnabled }),
+    validationSchema: getValidationSchema({
+      isOfferAddressEnabled,
+      isDigitalOffer:
+        categoryStatus === CATEGORY_STATUS.ONLINE || Boolean(offer?.isDigital),
+    }),
     onSubmit,
   })
   const handlePreviousStepOrBackToReadOnly = () => {
@@ -322,11 +325,12 @@ export const IndividualOfferDetailsScreen = ({
               isProductBased={isProductBased}
               filteredCategories={filteredCategories}
               filteredSubcategories={filteredSubcategories}
-              readonlyFields={readOnlyFields}
+              readOnlyFields={readOnlyFields}
               onImageUpload={onImageUpload}
               onImageDelete={onImageDelete}
               imageOffer={imageOffer}
               venuesOptions={availableVenuesOptions}
+              categoryStatus={categoryStatus}
             />
           </FormLayout>
           <ActionBar
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/__specs__/utils.spec.ts b/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/__specs__/utils.spec.ts
index cbbe9bf97b5..579b905f88e 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/__specs__/utils.spec.ts
+++ b/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/__specs__/utils.spec.ts
@@ -29,7 +29,6 @@ describe('setDefaultInitialValuesFromOffer', () => {
     const expectedValues = {
       isEvent: true,
       isNational: false,
-      isVenueVirtual: false,
       withdrawalDetails: 'Detailed info',
       withdrawalDelay: 3,
       withdrawalType: WithdrawalTypeEnum.BY_EMAIL,
@@ -43,7 +42,6 @@ describe('setDefaultInitialValuesFromOffer', () => {
       bookingEmail: 'test@example.com',
       bookingContact: 'Contact Info',
       receiveNotificationEmails: true,
-      url: 'http://example.com',
     }
 
     const result = setDefaultInitialValuesFromOffer({ offer: mockOffer })
@@ -64,7 +62,6 @@ describe('setDefaultInitialValuesFromOffer', () => {
     const expectedValues = {
       isEvent: false,
       isNational: true,
-      isVenueVirtual: false,
       withdrawalDetails:
         DEFAULT_USEFUL_INFORMATION_INITIAL_VALUES['withdrawalDetails'],
       withdrawalDelay: undefined,
@@ -79,7 +76,6 @@ describe('setDefaultInitialValuesFromOffer', () => {
       bookingEmail: '',
       bookingContact: undefined,
       receiveNotificationEmails: false,
-      url: DEFAULT_USEFUL_INFORMATION_INITIAL_VALUES['url'],
     }
 
     const result = setDefaultInitialValuesFromOffer({
@@ -102,7 +98,6 @@ describe('setDefaultInitialValuesFromOffer', () => {
     const expectedValues = {
       isEvent: false,
       isNational: true,
-      isVenueVirtual: false,
       withdrawalDetails:
         DEFAULT_USEFUL_INFORMATION_INITIAL_VALUES['withdrawalDetails'],
       withdrawalDelay: undefined,
@@ -117,7 +112,6 @@ describe('setDefaultInitialValuesFromOffer', () => {
       bookingEmail: '',
       bookingContact: undefined,
       receiveNotificationEmails: false,
-      url: DEFAULT_USEFUL_INFORMATION_INITIAL_VALUES['url'],
     }
 
     const result = setDefaultInitialValuesFromOffer({
@@ -167,7 +161,6 @@ describe('setDefaultInitialValuesFromOffer', () => {
     const expectedValues = {
       isEvent: true,
       isNational: false,
-      isVenueVirtual: false,
       withdrawalDetails: 'Detailed info',
       withdrawalDelay: 3,
       withdrawalType: WithdrawalTypeEnum.BY_EMAIL,
@@ -181,7 +174,6 @@ describe('setDefaultInitialValuesFromOffer', () => {
       bookingEmail: 'test@example.com',
       bookingContact: 'Contact Info',
       receiveNotificationEmails: true,
-      url: 'http://example.com',
 
       offerLocation: OFFER_LOCATION.OTHER_ADDRESS,
       manuallySetAddress: true,
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/__specs__/validationSchema.spec.tsx b/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/__specs__/validationSchema.spec.tsx
index b1125ac1111..ae2051aefe0 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/__specs__/validationSchema.spec.tsx
+++ b/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/__specs__/validationSchema.spec.tsx
@@ -46,27 +46,6 @@ describe('getValidationSchema', () => {
     ).rejects.toThrow('Vous devez choisir l’une des options ci-dessus')
   })
 
-  it('should validate url correctly when isVenueVirtual is true', async () => {
-    const schema = getValidationSchema({ subcategories: [] })
-    await expect(
-      schema.validate({
-        ...defaultAccessibility,
-        isVenueVirtual: true,
-        url: 'https://example.com',
-      })
-    ).resolves.toBeTruthy()
-
-    await expect(
-      schema.validate({
-        ...defaultAccessibility,
-        isVenueVirtual: true,
-        url: 'invalid-url',
-      })
-    ).rejects.toThrow(
-      'Veuillez renseigner une URL valide. Ex : https://exemple.com'
-    )
-  })
-
   it('should validate bookingContact correctly', async () => {
     const schema = getValidationSchema({ subcategories: ['bookingContact'] })
     await expect(
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/constants.ts b/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/constants.ts
index f49e4f0ac50..e0c43711585 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/constants.ts
+++ b/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/constants.ts
@@ -13,7 +13,6 @@ export const DEFAULT_USEFUL_INFORMATION_INITIAL_VALUES: UsefulInformationFormVal
     withdrawalDelay: SUBCATEGORIES_FIELDS_DEFAULT_VALUES['withdrawalDelay'],
     receiveNotificationEmails: false,
     bookingEmail: '',
-    url: '',
     accessibility: {
       [AccessibilityEnum.VISUAL]: false,
       [AccessibilityEnum.MENTAL]: false,
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/types.ts b/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/types.ts
index 4db5765bb23..91151a14ecb 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/types.ts
+++ b/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/types.ts
@@ -10,10 +10,7 @@ export type UsefulInformationFormValues = {
   accessibility: AccessibilityFormValues
   receiveNotificationEmails: boolean
   bookingEmail: string
-  url: string
-  isVenueVirtual?: boolean
   bookingContact?: string
-
   offerLocation?: string | undefined
   manuallySetAddress?: boolean
   'search-addressAutocomplete'?: string
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/utils.ts b/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/utils.ts
index 9f8ab93e3ba..04c5e43abe0 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/utils.ts
+++ b/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/utils.ts
@@ -67,7 +67,7 @@ export function setDefaultInitialValuesFromOffer({
       latitude: String(offer.address.latitude),
       longitude: String(offer.address.longitude),
     }
-  } else if (selectedVenue && selectedVenue.address) {
+  } else if (!offer.isDigital && selectedVenue && selectedVenue.address) {
     addressFields = {
       offerLocation: String(selectedVenue.address.id_oa),
       coords: `${selectedVenue.address.latitude}, ${selectedVenue.address.longitude}`,
@@ -100,8 +100,6 @@ export function setDefaultInitialValuesFromOffer({
     bookingEmail: offer.bookingEmail || '',
     bookingContact: offer.bookingContact || undefined,
     receiveNotificationEmails: !!offer.bookingEmail,
-    url: offer.url || DEFAULT_USEFUL_INFORMATION_INITIAL_VALUES['url'],
-    isVenueVirtual: offer.venue.isVirtual || false,
     ...addressFields,
   }
 }
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/validationSchema.ts b/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/validationSchema.ts
index 4f7eeaf9e3b..6c8927c5954 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/validationSchema.ts
+++ b/pro/src/pages/IndividualOffer/IndividualOfferInformations/commons/validationSchema.ts
@@ -3,15 +3,7 @@ import * as yup from 'yup'
 import { WithdrawalTypeEnum } from 'apiClient/v1'
 import { emailSchema } from 'commons/utils/isValidEmail'
 
-import { validationSchema as offerLocationSchema } from '../components/OfferLocation/validationSchema'
-
-// TODO: this regex is subject to backtracking which can lead to "catastrophic backtracking", high memory usage and slow performance
-// we cannot use the yup url validation because we need to allow {} in the url to interpolate some data
-const offerFormUrlRegex = new RegExp(
-  /*eslint-disable-next-line no-useless-escape*/
-  /^(http:\/\/www\.|https:\/\/www\.|http:\/\/|https:\/\/)(([a-z0-9]+([\-\.\.-\.@_a-z0-9]+)*\.[a-z]{2,5})|((25[0-5]|(2[0-4]|1\d|[1-9]|)\d)\.){3}(25[0-5]|(2[0-4]|1\d|[1-9]|)\d))(:[0-9]{1,5})?\S*?$/,
-  'i'
-)
+import { validationSchema as locationSchema } from '../components/OfferLocation/validationSchema'
 
 const isAnyTrue = (values: Record<string, boolean>): boolean =>
   Object.values(values).includes(true)
@@ -19,10 +11,12 @@ const isAnyTrue = (values: Record<string, boolean>): boolean =>
 type ValidationSchemaProps = {
   subcategories: string[]
   isOfferAddressEnabled?: boolean
+  isDigitalOffer?: boolean
 }
 export const getValidationSchema = ({
   subcategories,
   isOfferAddressEnabled = false,
+  isDigitalOffer = false,
 }: ValidationSchemaProps) => {
   const validationSchema = {
     withdrawalType: yup.string().when([], {
@@ -41,21 +35,6 @@ export const getValidationSchema = ({
       then: (schema) =>
         schema.required('Vous devez choisir l’une des options ci-dessus'),
     }),
-    url: yup.string().when('isVenueVirtual', {
-      is: (isVenueVirtual: boolean) => isVenueVirtual,
-      then: (schema) =>
-        schema
-          .required(
-            'Veuillez renseigner une URL valide. Ex : https://exemple.com'
-          )
-          .test({
-            name: 'url',
-            message:
-              'Veuillez renseigner une URL valide. Ex : https://exemple.com',
-            test: (url?: string) =>
-              url ? url.match(offerFormUrlRegex) !== null : true,
-          }),
-    }),
     bookingContact: yup.string().when([], {
       is: () => subcategories.includes('bookingContact'),
       then: (schema) =>
@@ -96,6 +75,6 @@ export const getValidationSchema = ({
 
   return yup.object().shape({
     ...validationSchema,
-    ...(isOfferAddressEnabled ? offerLocationSchema : {}),
+    ...(isOfferAddressEnabled && !isDigitalOffer ? locationSchema : {}),
   })
 }
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferInformations/components/IndividualOfferInformationsScreen.tsx b/pro/src/pages/IndividualOffer/IndividualOfferInformations/components/IndividualOfferInformationsScreen.tsx
index 5d694a72779..7bee29d5114 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferInformations/components/IndividualOfferInformationsScreen.tsx
+++ b/pro/src/pages/IndividualOffer/IndividualOfferInformations/components/IndividualOfferInformationsScreen.tsx
@@ -198,12 +198,12 @@ export const IndividualOfferInformationsScreen = ({
     offerSubCategory,
     isUserAdmin: false,
     receiveNotificationEmails: true,
-    isVenueVirtual: offer.venue.isVirtual,
   })
 
   const validationSchema = getValidationSchema({
     subcategories: conditionalFields,
     isOfferAddressEnabled,
+    isDigitalOffer: offer.isDigital,
   })
 
   const initialValues = setDefaultInitialValuesFromOffer({
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferInformations/components/OfferLocation/validationSchema.ts b/pro/src/pages/IndividualOffer/IndividualOfferInformations/components/OfferLocation/validationSchema.ts
index b4f36242faf..511a0307d62 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferInformations/components/OfferLocation/validationSchema.ts
+++ b/pro/src/pages/IndividualOffer/IndividualOfferInformations/components/OfferLocation/validationSchema.ts
@@ -4,25 +4,13 @@ import { checkCoords } from 'commons/utils/coords'
 import { OFFER_LOCATION } from 'pages/IndividualOffer/commons/constants'
 
 const locationSchema = {
-  offerLocation: yup
-    .string()
-    .trim()
-    .when('isVenueVirtual', {
-      is: false,
-      then: (schema) => schema.required('Veuillez sélectionner un choix'),
-    }),
+  offerLocation: yup.string().trim().required('Veuillez sélectionner un choix'),
   addressAutocomplete: yup
     .string()
     .trim()
-    .when(['offerLocation', 'manuallySetAddress', 'isVenueVirtual'], {
-      is: (
-        offerLocation: string,
-        manuallySetAddress: boolean,
-        isVenueVirtual: boolean
-      ) =>
-        !isVenueVirtual &&
-        offerLocation === OFFER_LOCATION.OTHER_ADDRESS &&
-        !manuallySetAddress,
+    .when(['offerLocation', 'manuallySetAddress'], {
+      is: (offerLocation: string, manuallySetAddress: boolean) =>
+        offerLocation === OFFER_LOCATION.OTHER_ADDRESS && !manuallySetAddress,
       then: (schema) =>
         schema.required(
           'Veuillez sélectionner une adresse parmi les suggestions'
@@ -31,18 +19,18 @@ const locationSchema = {
   street: yup
     .string()
     .trim()
-    .when(['offerLocation', 'isVenueVirtual'], {
-      is: (offerLocation: string, isVenueVirtual: boolean) =>
-        !isVenueVirtual && offerLocation === OFFER_LOCATION.OTHER_ADDRESS,
+    .when(['offerLocation'], {
+      is: (offerLocation: string) =>
+        offerLocation === OFFER_LOCATION.OTHER_ADDRESS,
       then: (schema) =>
         schema.required('Veuillez renseigner une adresse postale'),
     }),
   postalCode: yup
     .string()
     .trim()
-    .when(['offerLocation', 'isVenueVirtual'], {
-      is: (offerLocation: string, isVenueVirtual: boolean) =>
-        !isVenueVirtual && offerLocation === OFFER_LOCATION.OTHER_ADDRESS,
+    .when(['offerLocation'], {
+      is: (offerLocation: string) =>
+        offerLocation === OFFER_LOCATION.OTHER_ADDRESS,
       then: (schema) => schema.required('Veuillez renseigner un code postal'),
     })
     .min(5, 'Veuillez renseigner un code postal valide')
@@ -50,23 +38,17 @@ const locationSchema = {
   city: yup
     .string()
     .trim()
-    .when(['offerLocation', 'isVenueVirtual'], {
-      is: (offerLocation: string, isVenueVirtual: boolean) =>
-        !isVenueVirtual && offerLocation === OFFER_LOCATION.OTHER_ADDRESS,
+    .when(['offerLocation'], {
+      is: (offerLocation: string) =>
+        offerLocation === OFFER_LOCATION.OTHER_ADDRESS,
       then: (schema) => schema.required('Veuillez renseigner une ville'),
     }),
   coords: yup
     .string()
     .trim()
-    .when(['offerLocation', 'manuallySetAddress', 'isVenueVirtual'], {
-      is: (
-        offerLocation: string,
-        manuallySetAddress: boolean,
-        isVenueVirtual: boolean
-      ) =>
-        !isVenueVirtual &&
-        offerLocation === OFFER_LOCATION.OTHER_ADDRESS &&
-        manuallySetAddress,
+    .when(['offerLocation', 'manuallySetAddress'], {
+      is: (offerLocation: string, manuallySetAddress: boolean) =>
+        offerLocation === OFFER_LOCATION.OTHER_ADDRESS && manuallySetAddress,
       then: (schema) =>
         schema
           .required('Veuillez renseigner les coordonnées GPS')
diff --git a/pro/src/pages/IndividualOffer/IndividualOfferInformations/components/UsefulInformationForm/UsefulInformationForm.tsx b/pro/src/pages/IndividualOffer/IndividualOfferInformations/components/UsefulInformationForm/UsefulInformationForm.tsx
index c24a76c372d..9c2fd9ef49a 100644
--- a/pro/src/pages/IndividualOffer/IndividualOfferInformations/components/UsefulInformationForm/UsefulInformationForm.tsx
+++ b/pro/src/pages/IndividualOffer/IndividualOfferInformations/components/UsefulInformationForm/UsefulInformationForm.tsx
@@ -68,10 +68,6 @@ export const UsefulInformationForm = ({
     ? Object.keys(DEFAULT_USEFUL_INFORMATION_INITIAL_VALUES)
     : setFormReadOnlyFields(offer)
 
-  // we use venue is virtual here because we cannot infer it from the offerSubCategory
-  // because of CATEGORY_STATUS.ONLINE_OR_OFFLINE who can be both virtual or not
-  const isVenueVirtual = venue.isVirtual
-
   const {
     currentUser: { isAdmin, email },
   } = useCurrentUser()
@@ -80,7 +76,7 @@ export const UsefulInformationForm = ({
     offerSubCategory?.reimbursementRule === REIMBURSEMENT_RULES.NOT_REIMBURSED
 
   const displayWithdrawalReminder =
-    !offerSubCategory?.isEvent && !isVenueVirtual
+    !offerSubCategory?.isEvent && !offer.isDigital
 
   const displayBookingContact = offerSubCategory?.canBeWithdrawable
 
@@ -103,7 +99,7 @@ export const UsefulInformationForm = ({
 
   return (
     <>
-      {isOfferAddressEnabled && !isVenueVirtual && (
+      {isOfferAddressEnabled && !offer.isDigital && (
         <OfferLocation venue={selectedVenue} readOnlyFields={readOnlyFields} />
       )}
       <FormLayout.Section title="Retrait de l’offre">
@@ -185,7 +181,7 @@ export const UsefulInformationForm = ({
                 text: 'Quelles modalités de retrait choisir ?',
               }}
             >
-              {isVenueVirtual
+              {offer.isDigital
                 ? 'Indiquez ici tout ce qui peut être utile au bénéficiaire pour le retrait de l’offre.'
                 : 'Indiquez ici tout ce qui peut être utile au bénéficiaire pour le retrait de l’offre. En renseignant ces informations depuis les paramètres généraux de votre page partenaire, elles s’appliqueront par défaut à toutes vos offres.'}
             </InfoBox>
@@ -198,7 +194,7 @@ export const UsefulInformationForm = ({
             maxLength={500}
             disabled={readOnlyFields.includes('withdrawalDetails')}
             description={
-              isVenueVirtual
+              offer.isDigital
                 ? 'Exemples : une création de compte, un code d’accès spécifique, une communication par email...'
                 : 'Exemples : une autre adresse, un horaire d’accès, un délai de retrait, un guichet spécifique, un code d’accès, une communication par email...'
             }
@@ -224,25 +220,6 @@ export const UsefulInformationForm = ({
             />
           </FormLayout.Row>
         )}
-
-        {isVenueVirtual && (
-          <FormLayout.Row
-            sideComponent={
-              <InfoBox>
-                Lien vers lequel seront renvoyés les bénéficiaires ayant réservé
-                votre offre sur l’application pass Culture.
-              </InfoBox>
-            }
-          >
-            <TextInput
-              label="URL d’accès à l’offre"
-              name="url"
-              type="text"
-              description="Format : https://exemple.com"
-              disabled={readOnlyFields.includes('url')}
-            />
-          </FormLayout.Row>
-        )}
         {isAdmin && (
           <FormLayout.Row>
             <Checkbox
diff --git a/pro/src/pages/IndividualOffer/commons/__specs__/serializers.spec.ts b/pro/src/pages/IndividualOffer/commons/__specs__/serializers.spec.ts
index a2cf9ef4f39..228d544ccaf 100644
--- a/pro/src/pages/IndividualOffer/commons/__specs__/serializers.spec.ts
+++ b/pro/src/pages/IndividualOffer/commons/__specs__/serializers.spec.ts
@@ -99,7 +99,6 @@ describe('IndividualOffer:commons:serializers', () => {
         bookingContact: undefined,
         isDuo: false,
         url: 'https://my.url',
-        isVenueVirtual: false,
         city: 'Paris',
         latitude: '48.853320',
         longitude: '2.348979',
diff --git a/pro/src/pages/IndividualOffer/commons/types.ts b/pro/src/pages/IndividualOffer/commons/types.ts
index 507396a836e..1ad1b79f02a 100644
--- a/pro/src/pages/IndividualOffer/commons/types.ts
+++ b/pro/src/pages/IndividualOffer/commons/types.ts
@@ -33,7 +33,6 @@ export interface IndividualOfferFormValues extends Partial<AddressFormValues> {
   bookingEmail: string
   isDuo: boolean
   url: string
-  isVenueVirtual?: boolean
   bookingContact?: string
   offerLocation?: string | undefined
   locationLabel?: string | null