On 15/2/17 10:05, Shane Tomlinson wrote:
> Yep, I have to send two summaries. That's my reward for not sending last
> Thursday's on time.
>
> *Theme: Mid train-81
> *
> *Action items from last time
>
> * NOT DONE - stomlinson to move Send SMS feature doc to github.
Without wanting to re-open a can of worms, I wonder...is this "not done"
because the "take everything from the google doc and put it into github"
step does not deliver sufficient value to us in practice?
Cheers,
Ryan
> * DONE vlad collect data for () => {} #democracy
>
> * DONE - stomlinson to re-enable fxa-auth-mailer in fxa-ci
>
> * DONE (eoger on it before I pinged) - stomlinson to ping eoger about 4709
>
> *UX/PM*
>
> * In some last minute testing of the new firstrun page A/B test, an
> integration issue was discovered that made the UX less than awesome.
> The new firstrun flows being tested [1][2] contain a "Skip this
> step" button that allows users to easily skip signing up/in to FxA.
> The problem is the button never went away, even if the user submit
> the FxA form. The firstrun team asked us for help, by sending them
> notifications whenever the FxA form's validation state changes and
> after a user submits the form. This was a high priority fix because
> the funnelcake test started on Sunday. The FxA fix [3] was released
> as train-79.4 on Monday. Thanks for a 2nd quick release in a week jrgm.
>
> * Ryan Feeley has started on the "Get the Sync party started" feature
> doc. The idea is once a user has connected their 2nd device, we
> should educate them of the most important ways their life can be
> made simpler with Sync. I need to get a link for this.
>
> * Integration with PageShot is on the horizon. PageShot ships with Fx
> 54 as an addon, and within a couple of revs would like to integrate
> with FxA so that a user can view their PageShots on multiple
> devices. This may be the driver to prototype the web extension auth API.
>
> * Ryan Feeley asked if train-79 released "Connect Another Device" to
> more people since the train was supposed to bump the feature release
> to 25% of people. I did some digging in DataDog, and yes, we have
> released to more people, though DataDog does not show an order of
> magnitude increase. We want from a total release of 1.25% of people,
> all of whom reported metrics, to 25% of people, only 10% of which
> report metrics. So the feature is in front of far more people, but
> only roughly double show up in DataDog. Interestingly (or perhaps
> sadly), Re:dash does not show an expected uptick in the number of
> users that are connecting a 2nd device even though our initial
> release showed a non-trivial percentage of users clicking on the app
> store install links. More investigation is needed.
>
> * Thanks to Vijay, the content server now has the one true
> "signin.success" metric. This metric is emit whenever a user
> successfully signs in, or completes a signin verification. Vijay
> updated a bunch of DataDog graphs that had to sum two distinct
> values that now only require one.
>
> *Ops/General Dev*
>
> * The firstrun patch [3] was against train-79. When that merges, we'll
> need to reverse merge train-79 to train-80, then train-80 to master.
>
> * The fat arrow discussion continues. There was a lot of pushback
> against choosing one of the choices, jrgm found a third option -
> require parenthesis except for one-liners. There may be consensus
> forming.
>
> * A couple of meetings ago jbuck reported that node-sass occasionally
> blew up Circle-CI tests with a "stack level too deep" error.
> Unfortunately, this still occurs when using cached resources.
>
> * We have a couple of new flaky functional tests that we need to track
> the cause of:
> o
> confirm - sign up with a restmail address, get the open restmail
> button
> o
> sign in with incorrect email case before normalization fix, on
> second attempt canonical form is used
>
> * And finally, e10s is enabled in the functional tests, which shaves ~
> 1 minute off of a test run. Thanks Vlad!
>
> *Action items
>
> *
>
> * stomlinson to ask jrgm & jbuck what content-experiments is being used.
>
> * rfeeley to add a feature card to "fxa-features"
>
> * vlad to do merge from 79-80 w/ firstrun fix.
>
> Shane
>
> [1] - https://www.mozilla.org/en-US/firefox/51.0.1/firstrun/?f=99
> [2] - https://www.mozilla.org/en-US/firefox/51.0.1/firstrun/?f=100
> [3] - https://github.com/mozilla/fxa-content-server/pull/4717
> [4] -
>
> **
>
>
> _______________________________________________
> Dev-fxacct mailing list
> [email protected]
> https://mail.mozilla.org/listinfo/dev-fxacct
>
_______________________________________________
Dev-fxacct mailing list
[email protected]
https://mail.mozilla.org/listinfo/dev-fxacct
