[16:26:55] matt_flaschen: mooeypoo, yeah, that's what I meant re findLatestFlowArchive. Sorry, not sure why I didn't see your question earlier.
[16:29:21] grrrit-wm: (PS1) Mooeypoo: [wip] Set the guided tour cookie in onUserSaveOptions [extensions/Flow] - https://gerrit.wikimedia.org/r/236993
[16:29:36] mooeypoo: matt_flaschen, are you sure we're using 'flow-link-to-archive' ?
[16:29:45] mooeypoo: I don't have it, and that makes the second tour step not show up
[16:29:46] matt_flaschen: mooeypoo, yes, will give you the command in a second.
[16:30:34] matt_flaschen: mooeypoo, delete Template:LQT_page_converted_to_Flow and Template:Wikitext_talk_page_converted_to_Flow, then run:
[16:30:35] matt_flaschen: mwscript extensions/Flow/maintenance/FlowCreateTemplates.php --force
[16:31:01] matt_flaschen: We'll have to do this in production too (and reinstate local modifications, if any, afterwards).
[16:31:03] matt_flaschen: Will add a blocker.
[16:31:05] mooeypoo: gotcha
[16:34:51] wikibugs: Collaboration-Team-Current, Echo, Patch-For-Review, WMF-deploy-2015-09-08_(1.26wmf22): Split notifications: minor display issues - https://phabricator.wikimedia.org/T111432#1618707 (Etonkovidova) Checked in betalabs in Chrome and FF.
[16:36:22] grrrit-wm: (PS2) Mooeypoo: Set the guided tour cookie in onUserSaveOptions [extensions/Flow] - https://gerrit.wikimedia.org/r/236993
[16:36:26] matt_flaschen: mooeypoo, is https://he.wikipedia.org/wiki/%D7%AA%D7%91%D7%A0%D7%99%D7%AA:%D7%93%D7%A3_%D7%A7%D7%95%D7%93_%D7%95%D7%99%D7%A7%D7%99_%D7%9E%D7%95%D7%9E%D7%A8_%D7%9C%D7%96%D7%A8%D7%99%D7%9E%D7%94 the right title for flow-importer-wt-converted-template?
[16:36:26] mooeypoo: matt_flaschen, ^^
[16:36:34] matt_flaschen: I'm a little confused since I don't see a 'page does not exist' message.
[16:36:51] mooeypoo: matt_flaschen, yes it looks like it
[16:37:12] mooeypoo: it literally says 'wiki page imported to flow'
[16:37:14] mooeypoo: or converted
[16:37:20] matt_flaschen: Thanks
[16:38:24] matt_flaschen: mooeypoo, oh, it does exist, but it's in includeonly. Man, that diff is confusing. Hopefully it's not as bad when you use Hebrew UI language.
[16:38:40] mooeypoo: ... it's a little as bad
[16:38:42] mooeypoo: :D
[16:39:16] mooeypoo: matt_flaschen, okay, that commit makes things work for me now. I'm just hoping i did things right, I added a public function to OptInController and took advantage of it in the hook
[16:39:29] matt_flaschen: mooeypoo, that's what I meant.
[16:42:10] matt_flaschen: mooeypoo, I guess in prod we should just manually add the class. That seems easier than my previous suggestion. Can you do Hebrew? Just that one, we don't need to worry about flow-importer-lqt-converted-template on any of these wikis.
[16:43:38] matt_flaschen: dannyh, remind me to check this when we enable opt-in on new wikis (beyond cawiki, frwiki, hewiki, wikidatawiki). It should only be an issue on wikis that have been using Flow for a while, but I can check it easily.