Refactor: remove ReviewPhotoStep

falling-fruit · Dec 26, 2024 · 899ee2f · 899ee2f
1 parent 4ee1d24
commit 899ee2f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 7 deletions.
diff --git a/src/components/form/LocationForm.js b/src/components/form/LocationForm.js
@@ -27,7 +27,7 @@ import Label from '../ui/Label'
 import LoadingIndicator from '../ui/LoadingIndicator'
 import { Checkbox, Select, Textarea } from './FormikWrappers'
 import { ProgressButtons, StyledForm } from './FormLayout'
-import { ReviewPhotoStep, ReviewStep } from './ReviewForm'
+import { ReviewStep } from './ReviewForm'
 import TypesSelect from './TypesSelect'
 import { useInvisibleRecaptcha } from './useInvisibleRecaptcha'
 
@@ -247,7 +247,6 @@ export const LocationForm = ({ editingId, initialValues }) => {
               {!editingId && (
                 <>
                   <ReviewStep />
-                  <ReviewPhotoStep />
                 </>
               )}
               <ProgressButtons>

diff --git a/src/components/form/ReviewForm.js b/src/components/form/ReviewForm.js
@@ -111,14 +111,12 @@ export const ReviewStep = ({ standalone, hasHeading = true }) => {
           }
         />
       </FormRatingWrapper>
+
+      <PhotoUploader name="review.photos" label="Upload images" />
     </>
   )
 }
 
-export const ReviewPhotoStep = () => (
-  <PhotoUploader name="review.photos" label="Upload images" />
-)
-
 const RatingLabeledRow = styled(LabeledRow)`
   > div > label {
     color: ${({ theme }) => theme.text};
@@ -208,7 +206,6 @@ export const ReviewForm = ({
           return (
             <Form>
               <ReviewStep standalone hasHeading={editingId == null} />
-              <ReviewPhotoStep />
               <ProgressButtons>
                 <div style={{ textAlign: editingId ? 'center' : 'left' }}>
                   <Button