Fixes #3427 - Remove min char requirement from description field #3428
Conversation
|
Also any feedback on the copy is appreciated :) We can change it to something else |
There was a problem hiding this comment.
So, let's merge and deploy this, but it would be nice to do A/B testing so we can compare apples to apples in the future. I guess we'll be able to compare this weeks data with last weeks data though to decide if we want this or not?
(Also, If you have a good reason for not using our AB testing stuff anymore, we should remove it)
| @@ -230,32 +230,31 @@ <h3 class="page-heading">Filing a web compatibility issue</h3> | |||
| </div> | |||
| </div> | |||
|
|
|||
| <div class="step-container step-description col"> | |||
| <div class="step-container step-description col" data-min-char="false"> | |||
There was a problem hiding this comment.
Cool, so we can toggle it on or off based on this.
I assumed we were going to do some A/B testing based on the existing AB test stuff we have, this approach would be compatible with that -- you set the value in the template based on active experiment.
|
Thanks! Yeah, the plan is to compare previous two weeks with the week when this change was shipped. I was focusing on reducing drop off at this step, since the rate is pretty high. I think we can still use A/B testing for other steps, where the reason for drop off is not as clear. |
I've removed min char requirement, made the description optional and changed the copy: