Skip to content
This repository has been archived by the owner on Jul 15, 2019. It is now read-only.

Commit

Permalink
Merge pull request #105 from WeTransfer/gh-xxx-maintenance
Browse files Browse the repository at this point in the history
Added new option in variant type plusPage and disabled unused variants
  • Loading branch information
manpreetssethi authored Apr 19, 2017
2 parents f4a0ef5 + 4e166e8 commit cda305f
Showing 1 changed file with 33 additions and 2 deletions.
35 changes: 33 additions & 2 deletions web/static/js/variantpayloadoptions.es6
Original file line number Diff line number Diff line change
Expand Up @@ -233,6 +233,7 @@ const options = {
}
},
signupFormPlanSelectorType: {
disabled: true,
name: 'Singup Form - Plan Selector Type',
key: 'signupFormPlanSelectorType',
defaults: {
Expand Down Expand Up @@ -261,6 +262,7 @@ const options = {
}
},
helpPageType: {
disabled: true,
name: 'Help Page Type',
key: 'helpPageType',
defaults: {
Expand Down Expand Up @@ -319,6 +321,20 @@ const options = {
field: 'type',
value: 'socialProof'
}
},
ctaText: {
type: 'string',
requiredWhen: {
field: 'type',
value: 'bannerWithCTA'
}
},
onPage: {
type: 'string',
requiredWhen: {
field: 'type',
value: 'bannerWithCTA'
}
}
},
properties: [
Expand All @@ -328,7 +344,8 @@ const options = {
type: 'string',
enum: [
{label: 'none', value: 'none'},
{label: 'Social Proof', value: 'socialProof'}
{label: 'Social Proof', value: 'socialProof'},
{label: 'Banner with CTA', value: 'bannerWithCTA'}
]
},
{
Expand All @@ -342,7 +359,21 @@ const options = {
{
key: 'by',
title: 'By'
}
},
{
key: 'ctaText',
title: 'CTA Text'
},
{
key: 'onPage',
title: 'Page',
type: 'string',
enum: [
{label: 'none', value: 'none'},
{label: 'Plus', value: '/plus'},
{label: 'Sign-in', value: '/sign-in'}
]
},
]
}
},
Expand Down

0 comments on commit cda305f

Please sign in to comment.