Wednesday, 2016-09-07

*** openstackgerrit has quit IRC05:18
*** openstackgerrit has joined #storyboard05:19
*** fay has joined #storyboard07:22
*** fay is now known as Guest4063207:22
*** Guest40632 is now known as faybrocklebank07:23
*** matthewbodkin has joined #storyboard07:54
*** openstackgerrit has quit IRC08:34
*** openstackgerrit has joined #storyboard08:34
SotKmorning!08:46
Zaramrrrrrrrrrrgh09:17
ZaraI did not sleep last night09:17
Zaraafk for a little bit10:12
Zarabk10:46
Zarareminder: storyboard meeting at 15:00 UTC!11:15
Zaraagenda at https://wiki.openstack.org/wiki/Meetings/StoryBoard#Agenda_for_next_meeting ; will update a bit11:16
*** alexismonville has joined #storyboard11:28
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient: Rework the task list layout  https://review.openstack.org/35730611:32
Zaraooh11:32
SotKfinally!11:33
Zara:)11:33
Zaraah, so now it's split out from 'add to worklist' things11:37
ZaraI was going to say things about the commit message and gerrit.11:37
pedroalvarezI looked through the previous version, and looked good11:38
SotKyep, it was getting very large11:39
Zarayeah, I prefer it split out.11:39
Zarareusable tags seem fine on my test instance; could you edit the commit message so that the gerrit plugin can update the task?11:43
Zara^matthewbodkin11:43
*** alexismonville has quit IRC11:45
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient: Rework the task list layout  https://review.openstack.org/35730611:48
persiaThat revision is more readable, but I still wonder if it ought depend on API changes to separate notes from review URL and wonder why "Priority=medium" seems hardcoded.11:50
SotK"Priority=medium" is the starting default, and will disappear when the complex priority patch is put on top of it12:08
SotKand we can probably update the link/notes mess at a later date12:08
openstackgerritMatthew Bodkin proposed openstack-infra/storyboard-webclient: Make tags reusable  https://review.openstack.org/36618812:33
Zarabtw, saw this last night, just remembered-- a lot of handy tips in this thread: https://news.ycombinator.com/item?id=1243752712:33
*** alexismonville has joined #storyboard12:49
Zaramy feet are different colours again.13:14
Zarahm, cancel button is red...13:41
Zara(for 'add new task'; guessing that means the 'save' and 'cancel' buttons will look the same on s.o.o)13:42
Zarayeah, they do (different shades of red, wah)13:43
ZaraI don't think we should have two 'add task' buttons.13:44
Zaraand 'Tasks' looks a bit like it's part of the story description so we should reinstate the horizontal rule13:47
Zarawe've also lost the remove button13:48
SotKthe remove button is still there, just not when there is only one task left in a story13:49
persiaI think that is now called "delete"13:49
persiaI think I like the new add task more than the old one.13:49
SotKre: two add task buttons, one is a shorthand to add a task in one of the existing project+branch bits, the other allows you to set the project too13:50
SotKI guess we can have just the second one, but I suspect that will lead to a frustrating game of "find the task I just made" for stories with many projects/tasks13:51
Zaraokay, it's not obvious that that's the distinction atm; I just thought one task button was missing the option to map the task to a project.13:52
Zaraand similar for the behaviour of the delete button not being obvious, though I don't think it matters if that one confuses folk13:53
SotKis it any more obvious on stories with tasks from multiple projects?13:54
persiaHm.  Maybe I need a better example story.  I was looking at 32.13:55
persiaAny suggestions?13:55
Zarapersia: 35 has one task13:57
ZaraSotK: kind of, but it gets confusing once both dialogues are open (the 'add new task to this project') stays open by default, then there's also an 'add new  task' button13:57
Zaraand if that one is opened too, it is very crowded13:57
persiaLooking at 31, I think it is more obvious, but also takes way too much space.13:57
krotscheckChange the color to mauve14:00
persiaheh14:01
Zarayou've given persia ideas!14:01
persiaI wonder if it would seem less duplicative if the bottom one were renamed "affects other project" or similar?14:02
SotKthat would make sense to me14:05
SotKI think that was the meaning which made me write "Add new task" on that button, which doesn't really convey that at all14:06
persiaMy first thought was "add project", but I realised that was also confusing (because one cannot add an entirely new project there)14:07
ZaraI'd rather just have the one 'add task' button with a field to let a user specify a project, and automatically fill it with the project of the previous task. I don' tthink that's going to be more confusing than having two task-adding dialogues that can be open simultaneously.14:09
Zarabut that's just theoretical so you might have a different idea to what I'm picturing14:10
persiaNot allowing them to be open simultaneously is a great idea.14:16
persiaI don't like defaulting to the previous one, mostly because that means whichever is last becomes default, which can be annoying for certain classes of projects14:17
persia(for example, if there are any tasks in openstack-infra/*, those always sort after openstack/*, which means that anything that needs a change to testing mechanisms automatically has the wrong default)14:18
*** alexismonville has quit IRC14:20
*** SotK has quit IRC14:20
*** anteaya has quit IRC14:20
*** pedroalvarez has quit IRC14:20
Zara(ohhhhh actually the big thing that's going to confuse people is that they're going to try to add a new project name instead of selecting an existing one. oh well. one thing at a time.)14:31
*** alexismonville has joined #storyboard14:42
*** SotK has joined #storyboard14:42
*** anteaya has joined #storyboard14:42
*** pedroalvarez has joined #storyboard14:42
Zarawhile you were gone, I swapped all the storyboard code for a game of snake14:42
persiaYeah, that's why I thought using words other than "add" would be useful.14:43
* SotK wonders if his message was lost in the netsplit14:44
persiaThe last I saw from you started "I think that was the meaning..."14:45
ZaraI don't have anything after 15:0614:45
* SotK wonders if folk would prefer the patch to add an "add to worklist" button on its own or as part of the complex priorities patch?14:45
persiaCan we do three patches?14:45
SotKI'm not sure what you mean14:46
Zaraif you send more than two patches a day the patchmonster will eat you unless you send this message to 6 of your friends14:46
Zarameeting in 10.14:49
persiaI don't like really big patches, because it's hard for me to understand everything all at once.  So, given a question of the form "should feature X be in patch A or patch B?", I tend to prefer that there are three patches.14:51
persiaThat said, maybe this isn't feasible with the current set: depending on what needs doing to make it work.14:51
SotKoh, then I miscommunicated my original question and you want the first option14:52
persiaRather, I misread: I read "the patch" as the subject, and "to add" as the verb, whereas re-reading, clearly "the patch to add an "add to worklist" button" is the object of "prefer".14:54
persiaApologies for failing to grammar today.14:54
persiaYes I prefer it on it's own :)14:54
krotscheckOh my god that timeline UI is lovely14:55
krotscheckWho did that?14:55
ZaraSotK!14:56
Zarahe is amazing14:56
SotKthat was me14:56
Zaraand yes14:56
Zarait is the loveliest thing ever isn't it14:56
krotscheckSotK: That's brilliant.14:56
Zarait's the reason I'm so fussy with his patches; I HAVE BEEN SPOILED14:57
SotKkrotscheck: thanks :)14:59
openstackgerritMerged openstack-infra/python-storyboardclient: Give user option to bypass verification  https://review.openstack.org/36609915:16
*** matthewbodkin has quit IRC15:43
*** faybrocklebank has quit IRC15:56
*** alexismonville has quit IRC16:04
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient: Rework the task list layout  https://review.openstack.org/35730618:23
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient: Add a way to add tasks to worklists from the story view  https://review.openstack.org/36689918:23
*** alexismonville has joined #storyboard18:25
openstackgerritAdam Coldrick proposed openstack-infra/storyboard-webclient: Complex priorities UI in stories  https://review.openstack.org/31266618:32
*** openstackgerrit has quit IRC20:04
*** openstackgerrit has joined #storyboard20:04
*** alexismonville has quit IRC20:10
*** jmccrory has quit IRC21:53
*** jmccrory has joined #storyboard21:53
*** jmccrory has quit IRC21:55
*** jmccrory has joined #storyboard21:55
*** jmccrory has quit IRC21:57
*** jmccrory has joined #storyboard21:57

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!