Skip to content

Commit

Permalink
Implement VAR for C23_WMDE_Desktop_DE_04
Browse files Browse the repository at this point in the history
- text change in slides and bannertext for VAR

- remove unused custom amount code
- add more feature tests to happy path tests

https://phabricator.wikimedia.org/T344423
  • Loading branch information
moiikana committed Sep 5, 2023
1 parent 78b7ce0 commit 90f45e3
Show file tree
Hide file tree
Showing 6 changed files with 12 additions and 18 deletions.
7 changes: 0 additions & 7 deletions banners/desktop/components/BannerCtrl.vue
Original file line number Diff line number Diff line change
Expand Up @@ -30,11 +30,6 @@
<template #[FormStepNames.UpgradeToYearlyFormStep]="{ pageIndex, submit, isCurrent, previous }: any">
<UpgradeToYearlyForm :page-index="pageIndex" @submit="submit" :is-current="isCurrent" @previous="previous"/>
</template>

<template #[FormStepNames.CustomAmountFormStep]="{ pageIndex, submit, isCurrent, previous }: any">
<CustomAmountForm :page-index="pageIndex" @submit="submit" :is-current="isCurrent" @previous="previous"/>
</template>

</MultiStepDonation>
</template>

Expand Down Expand Up @@ -71,7 +66,6 @@ import MultiStepDonation from '@src/components/DonationForm/MultiStepDonation.vu
import BannerFooter from '@src/components/Footer/BannerFooter.vue';
import MainDonationForm from '@src/components/DonationForm/Forms/MainDonationForm.vue';
import UpgradeToYearlyForm from '@src/components/DonationForm/Forms/UpgradeToYearlyForm.vue';
import CustomAmountForm from '@src/components/DonationForm/Forms/CustomAmountForm.vue';
import KeenSlider from '@src/components/Slider/KeenSlider.vue';
import { useFormModel } from '@src/components/composables/useFormModel';
import {
Expand All @@ -90,7 +84,6 @@ enum ContentStates {
}
enum FormStepNames {
CustomAmountFormStep = 'CustomAmountForm',
MainDonationFormStep = 'MainDonationForm',
UpgradeToYearlyFormStep = 'UpgradeToYearlyForm'
}
Expand Down
7 changes: 0 additions & 7 deletions banners/desktop/components/BannerVar.vue
Original file line number Diff line number Diff line change
Expand Up @@ -30,11 +30,6 @@
<template #[FormStepNames.UpgradeToYearlyFormStep]="{ pageIndex, submit, isCurrent, previous }: any">
<UpgradeToYearlyForm :page-index="pageIndex" @submit="submit" :is-current="isCurrent" @previous="previous"/>
</template>

<template #[FormStepNames.CustomAmountFormStep]="{ pageIndex, submit, isCurrent, previous }: any">
<CustomAmountForm :page-index="pageIndex" @submit="submit" :is-current="isCurrent" @previous="previous"/>
</template>

</MultiStepDonation>
</template>

Expand Down Expand Up @@ -71,7 +66,6 @@ import MultiStepDonation from '@src/components/DonationForm/MultiStepDonation.vu
import BannerFooter from '@src/components/Footer/BannerFooter.vue';
import MainDonationForm from '@src/components/DonationForm/Forms/MainDonationForm.vue';
import UpgradeToYearlyForm from '@src/components/DonationForm/Forms/UpgradeToYearlyForm.vue';
import CustomAmountForm from '@src/components/DonationForm/Forms/CustomAmountForm.vue';
import KeenSlider from '@src/components/Slider/KeenSlider.vue';
import { useFormModel } from '@src/components/composables/useFormModel';
import {
Expand All @@ -90,7 +84,6 @@ enum ContentStates {
}
enum FormStepNames {
CustomAmountFormStep = 'CustomAmountForm',
MainDonationFormStep = 'MainDonationForm',
UpgradeToYearlyFormStep = 'UpgradeToYearlyForm'
}
Expand Down
2 changes: 1 addition & 1 deletion banners/desktop/content/BannerSlides_var.vue
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
</KeenSliderSlide>
<KeenSliderSlide :is-current="currentSlide === 3">
<p>
Dann entscheiden Sie sich, eine der seltenen Ausnahmen zu sein, und geben Sie etwas zurück. Vielen Dank!
Dann entscheiden Sie sich, eine der seltenen Ausnahmen zu sein, und geben Sie etwas zurück. Jede Spende zählt. Vielen Dank!
</p>
</KeenSliderSlide>
</template>
Expand Down
2 changes: 1 addition & 1 deletion banners/desktop/content/BannerText_var.vue
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
aber 99&nbsp;% spenden nicht – sie übergehen diesen Aufruf.</span> Die meisten Menschen spenden, weil sie
Wikipedia nützlich finden. Die durchschnittliche Spende beträgt 22,25&nbsp;€, doch bereits 5&nbsp;€ helfen uns
weiter. Hat Wikipedia Ihnen in diesem Jahr Wissen im Wert einer Tasse Kaffee geschenkt? Dann entscheiden
Sie sich, eine der seltenen Ausnahmen zu sein, und geben Sie etwas zurück. Vielen Dank!
Sie sich, eine der seltenen Ausnahmen zu sein, und geben Sie etwas zurück. Jede Spende zählt. Vielen Dank!
</p>
</div>
</div>
Expand Down
4 changes: 3 additions & 1 deletion test/banners/desktop/components/BannerCtrl.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,9 @@ describe( 'BannerCtrl.vue', () => {
[ 'expectMainDonationFormGoesToUpgrade' ],
[ 'expectUpgradeToYearlyFormSubmitsUpgrade' ],
[ 'expectUpgradeToYearlyFormSubmitsDontUpgrade' ],
[ 'expectUpgradeToYearlyFormGoesToMainDonation' ]
[ 'expectUpgradeToYearlyFormGoesToMainDonation' ],
[ 'expectUpgradeToYearlyFormSubmitsUpgrade' ],
[ 'expectUpgradeToYearlyFormSubmitsDontUpgrade' ]
] )( '%s', async ( testName: string ) => {
await donationFormFeatures[ testName ]( getWrapper() );
} );
Expand Down
8 changes: 7 additions & 1 deletion test/banners/desktop/components/BannerVar.spec.ts
Original file line number Diff line number Diff line change
Expand Up @@ -86,8 +86,14 @@ describe( 'BannerVar.vue', () => {

describe( 'Donation Form Happy Paths', () => {
test.each( [
[ 'expectMainDonationFormSubmitsWhenSofortIsSelected' ],
[ 'expectMainDonationFormSubmitsWhenYearlyIsSelected' ],
[ 'expectMainDonationFormGoesToUpgrade' ],
[ 'expectUpgradeToYearlyFormGoesToMainDonation' ]
[ 'expectUpgradeToYearlyFormSubmitsUpgrade' ],
[ 'expectUpgradeToYearlyFormSubmitsDontUpgrade' ],
[ 'expectUpgradeToYearlyFormGoesToMainDonation' ],
[ 'expectUpgradeToYearlyFormSubmitsUpgrade' ],
[ 'expectUpgradeToYearlyFormSubmitsDontUpgrade' ]
] )( '%s', async ( testName: string ) => {
await donationFormFeatures[ testName ]( getWrapper() );
} );
Expand Down

0 comments on commit 90f45e3

Please sign in to comment.