flutter / uxr

UXR work for Flutter
BSD 3-Clause "New" or "Revised" License
227 stars 28 forks source link

Update contextual-survey-metadata.json for pilot test #1 #113

Closed jayoung-lee closed 10 months ago

jayoung-lee commented 10 months ago

The URL for "More Info" may be update later to /user-surveys once https://github.com/flutter/website/pull/8654 is merged

eliasyishak commented 10 months ago

I think you may also want to make the conditions array empty since you want to test with everyone right? As it stands right now, each user testing this feature would need to have already sent 1000 events previously, which may not be the case for everyone.

https://github.com/flutter/uxr/pull/113/files#diff-bdf7173921cada97180d013ce22fd3e8c7560bb877eddbb417724eeb29a3ed8cR13-R19

eliasyishak commented 10 months ago

Also if you want to test the snooze functionality, we may want to try a snooze period less than 7200 minutes? Perhaps something like 2-5 minutes to test the snooze functionality?

https://github.com/flutter/uxr/pull/113/files#diff-bdf7173921cada97180d013ce22fd3e8c7560bb877eddbb417724eeb29a3ed8cR8

jayoung-lee commented 10 months ago

Hi Elias, thanks for reviewing! 5 minutes makes more sense, actually. Updated!