(PC-34043)[PRO] feat: Let users create a digital offer on a physical …

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)}
 

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,19 +83,14 @@ 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')
 
       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)
     })
   })
 

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')

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',

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"

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',

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,

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

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

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={[

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 () => {

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}