fix: campaign application (#1976)

- fix order of fields (make the goal the last field on the 3-rd step)
- remove "Membership" from the list of campaign types (it's a system type)

Author: gparlakov

src/components/client/campaign-application/steps/CampaignApplicationBasic.tsx

@@ -58,7 +58,7 @@ export default function CampaignApplicationBasic() {
           />
         </Grid>
         <Grid item xs={12}>
-          <CampaignTypeSelect name="applicationBasic.campaignType" />
+          <CampaignTypeSelect name="applicationBasic.campaignType" hideSystemTypes={true} />
         </Grid>
         <Grid item xs={12}>
           <StyledFormTextField

src/components/client/campaign-application/steps/CampaignApplicationDetails.tsx

@@ -22,15 +22,6 @@ export default function CampaignApplicationDetails({ files, setFiles }: Props) {
         <StyledStepHeading variant="h4">{t('steps.details.title')}</StyledStepHeading>
       </Grid>
       <Grid item container spacing={6} justifyContent="space-between" direction="row">
-        <Grid item xs={12}>
-          <FormTextField
-            type="text"
-            name="applicationDetails.cause"
-            label={t('steps.details.cause')}
-            multiline
-            rows={3}
-          />
-        </Grid>
         <Grid item xs={12}>
           <FormTextField
             type="text"
@@ -50,6 +41,15 @@ export default function CampaignApplicationDetails({ files, setFiles }: Props) {
             rows={5}
           />
         </Grid>
+        <Grid item xs={12}>
+          <FormTextField
+            type="text"
+            name="applicationDetails.cause"
+            label={t('steps.details.cause')}
+            multiline
+            rows={3}
+          />
+        </Grid>
         <Grid item xs={12}>
           <FileUpload
             buttonLabel={t('steps.details.documents')}

src/components/client/campaigns/CampaignTypeSelect.tsx

@@ -4,7 +4,9 @@ import { useCampaignTypesList } from 'common/hooks/campaigns'
 import { useField } from 'formik'
 import { useTranslation } from 'next-i18next'
 
-export default function CampaignTypeSelect({ name = 'campaignTypeId' }) {
+const systemTypes = ['Membership']
+
+export default function CampaignTypeSelect({ name = 'campaignTypeId', hideSystemTypes = false }) {
   const { t } = useTranslation()
   const { data } = useCampaignTypesList()
   const [field, meta] = useField(name)
@@ -21,11 +23,13 @@ export default function CampaignTypeSelect({ name = 'campaignTypeId' }) {
         <MenuItem value="" disabled>
           {t('campaigns:campaign.type')}
         </MenuItem>
-        {data?.map((campaignType, index) => (
-          <MenuItem key={index} value={campaignType.id}>
-            {campaignType.name}
-          </MenuItem>
-        ))}
+        {data
+          ?.filter((campaignType) => !hideSystemTypes || !systemTypes.includes(campaignType?.name))
+          ?.map((campaignType, index) => (
+            <MenuItem key={index} value={campaignType.id}>
+              {campaignType.name}
+            </MenuItem>
+          ))}
       </Select>
       {helperText && <FormHelperText error>{helperText}</FormHelperText>}
     </FormControl>