Tuesday, 2019-10-22

*** jamesmcarthur has quit IRC00:14
*** jamesmcarthur has joined #storyboard00:15
*** jamesmcarthur has quit IRC00:16
*** jamesmcarthur has joined #storyboard00:16
*** jamesmcarthur has quit IRC00:17
*** jamesmcarthur has joined #storyboard00:19
*** jamesmcarthur has quit IRC00:19
*** jamesmcarthur has joined #storyboard00:35
*** jamesmcarthur has quit IRC00:52
*** diablo_rojo has quit IRC01:02
*** ianychoi has quit IRC01:16
*** ianychoi has joined #storyboard01:16
*** jamesmcarthur has joined #storyboard01:32
*** jamesmcarthur has quit IRC01:55
*** jamesmcarthur has joined #storyboard02:31
*** ianychoi has quit IRC02:31
*** ianychoi has joined #storyboard02:33
*** jamesmcarthur has quit IRC02:38
*** ianychoi has quit IRC02:51
*** ianychoi has joined #storyboard02:52
*** diablo_rojo has joined #storyboard03:02
*** diablo_rojo has quit IRC03:39
*** udesale has joined #storyboard04:04
*** ianychoi has quit IRC05:13
*** ianychoi has joined #storyboard05:14
*** ianychoi has quit IRC05:26
*** ianychoi has joined #storyboard05:28
*** jtomasek has joined #storyboard06:23
*** udesale has quit IRC07:06
*** udesale has joined #storyboard07:06
*** tosky has joined #storyboard07:12
*** udesale has quit IRC07:41
*** udesale has joined #storyboard07:42
*** udesale has quit IRC11:18
*** tonyb has joined #storyboard11:31
*** ianychoi has quit IRC11:37
*** ianychoi has joined #storyboard11:40
*** ianychoi has quit IRC12:00
*** ianychoi has joined #storyboard12:03
*** jamesmcarthur has joined #storyboard12:18
*** jamesmcarthur has quit IRC12:30
*** ianychoi has quit IRC12:39
*** ianychoi has joined #storyboard12:41
*** ianychoi has quit IRC12:54
*** ianychoi has joined #storyboard12:57
*** udesale has joined #storyboard13:31
*** diablo_rojo has joined #storyboard13:49
*** ianychoi has quit IRC13:58
*** ianychoi has joined #storyboard14:00
*** ianychoi has quit IRC14:38
*** ianychoi has joined #storyboard14:40
*** jamesmcarthur has joined #storyboard14:53
*** udesale has quit IRC15:34
*** diablo_rojo_phon has joined #storyboard15:42
*** efried has joined #storyboard15:44
efriedo/ storyboard!15:44
diablo_rojo_phonHello efried!15:44
Zara\o15:45
efriedcouple of things I noticed yesterday & today15:45
efried1) I can't assign a task. I tried putting variants of my name/handle in, using keys/mouse, punching the magnifying glass or hitting Return, etc; the tasks stay unassigned.15:46
efriedspecifically for this story: https://storyboard.openstack.org/#!/story/173501615:46
efried1b) I kinda would have expected the same bot that changes Todo=>Review would assign to owner of review?15:47
diablo_rojo_phon1) Which task? I can see if I can try and maybe reproduce the issue.15:47
diablo_rojo_phon1b) that's actually a change that needs to be done by a bot that lives in Gerritland not in Storyboard and it is on our list of things to look into because we agree that would be ideal.15:49
efried2) the bot comment when a related change is uploaded reads e.g.15:49
efriedChange 690071 had a related patch set uploaded by Eric Fried:15:49
efriedLink to (some) plugin doc pages15:49
efriedhttps://review.opendev.org/69007115:49
efriedThat is, it says "Change $gerritnum had... ... $link-to-same-$gerritnum"15:49
efriedThe duplication of $gerritnum isn't as useful as, for instance, if it said "Task $storyboard_task_num had a related patch set uploaded..."15:49
efrieddiablo_rojo_phon: 1) any of the tasks in that story.15:49
efried1b) ack, so known issue ++15:50
*** ianychoi has quit IRC15:52
diablo_rojo2) We have discussed taking the review numbers/urls and associating them with the tasks instead of just posting on the comments section15:53
diablo_rojoI think we are in agreement as a team its cleaner and we want to do it this way.15:54
diablo_rojoWe just havent gotten around to the UI changes yet.15:54
*** ianychoi has joined #storyboard15:54
diablo_rojoAnd that url would get updated each patch instead of a new comment each time15:54
diablo_rojoefried, ^^15:55
efriedoo, that would be neat. Having them in the comment stream is better than nothing, but I agree they would be much more useful tied to the task somehow15:56
diablo_rojoYeah we could include some data I suppose about when it was last updated, but I think the closer association rather than just dumping into the timeline would be more helpful?15:56
efriedo look, there's a twistie on a task where I can add notes.15:56
efrieddiablo_rojo: not sure what's possible here, but ideally a "note" on the task logging the action taken, plus a first-order field on the summary row indicating patch(es) *currently* associated15:57
efriedthe main improvement being associating either/both with the task instead of the story at large15:58
efriedI guess many/most stories probably have only one task so it's not a huge deal...15:58
efriedI need to run to a meeting. Thanks for the help diablo_rojo!15:58
diablo_rojoefried, thanks for the feedback!15:59
diablo_rojoI will keep looking in issue #115:59
diablo_rojo:)15:59
efriedlmk if you want me to open stories for any of this or whatever15:59
diablo_rojoIt would just be 1 if I can't figure it out.15:59
diablo_rojoI think we already have them for the other two points15:59
diablo_rojoBut if you want to comment on them further I can dig them up15:59
SotKoops, part 1 looks like we added a bug in the webclient16:03
* SotK can reproduce it trivially on any task16:04
SotKI'll send a patch for that in a couple of hours16:04
efriedcool, thanks SotK16:04
fungialso note that the story commit message footer is no longer really needed in addition to the task footer. you can just use the task footer and everything will still work16:15
fungithe story footer tends to be more relevant when you want a comment added to that story but you don't have a specific task in that story to update (so sort of like doing related-bug footers for launchpad bugs)16:16
fungiso i would use one or the other in a commit, not both16:16
*** ianychoi has quit IRC17:39
*** ianychoi has joined #storyboard17:42
*** ianychoi has quit IRC17:55
*** ianychoi has joined #storyboard17:58
*** jamesmcarthur has quit IRC18:01
*** ianychoi has quit IRC18:12
*** ianychoi has joined #storyboard18:16
*** ianychoi has quit IRC18:26
*** ianychoi has joined #storyboard18:30
*** ianychoi has quit IRC18:38
*** ianychoi has joined #storyboard18:42
*** jamesmcarthur has joined #storyboard18:47
*** jamesmcarthur_ has joined #storyboard18:50
*** jamesmcarthur has quit IRC18:54
*** ianychoi has quit IRC19:04
*** jamesmcarthur has joined #storyboard19:05
*** jamesmca_ has joined #storyboard19:06
*** ianychoi has joined #storyboard19:08
*** jamesmcarthur_ has quit IRC19:08
*** jamesmcarthur has quit IRC19:09
*** jamesmcarthur has joined #storyboard19:10
*** jamesmcarthur has quit IRC19:12
*** jamesmca_ has quit IRC19:13
*** jamesmcarthur has joined #storyboard19:15
*** ianychoi has quit IRC19:15
*** ianychoi has joined #storyboard19:24
*** jamesmcarthur has quit IRC19:27
*** jamesmcarthur has joined #storyboard19:32
*** ianychoi has quit IRC19:39
*** jamesmcarthur has quit IRC19:46
*** jtomasek has quit IRC19:53
* SotK finally gets to a computer20:05
efriedSotK: I added more tasks to that story for you to play with :P20:06
fungiwe've also got https://storyboard-dev.openstack.org/ if folks need a playground20:10
*** openstackgerrit has joined #storyboard20:12
openstackgerritAdam Coldrick proposed opendev/storyboard-webclient master: Fix user_typeahead result rendering  https://review.opendev.org/69040020:12
SotKdiablo_rojo, fungi: that fixes the task assignment issue if you have time for a quick review20:14
diablo_rojo_phonLooks like it was the show emails change that broke things.20:15
fungifor such a regression, i will make time20:15
diablo_rojo_phon(already looking at the patch)20:15
fungidiablo_rojo_phon: yep, i assumed that to be the case20:15
diablo_rojo_phon+2 ed20:16
fungithis won't show the e-mail address though, right?20:16
SotKit will do, that function renders the email too20:17
fungioh! right20:17
fungigood deal20:17
fungithanks!20:17
efriedthat will hit the real storyboard once the publish-openstack-javascript-content job makes it through post?20:20
* efried guesses...20:20
SotKit'll take a little longer, that job publishes a tarball which gets deployed whenever puppet next runs20:22
efriedight.20:23
efriedThanks SotK fungi diablo_rojo_phon!20:23
SotKyou're welcome!20:23
fungiefried: yeah, hopefully no more than an hour after merge20:25
fungithere are ideas on how to speed that up too20:25
efriedI'm not like slavering to assign my tasks to myself or anything.20:25
efriedjust to pop it off the stack20:25
fungibut it's also preventing you from reassigning your tasks to other unsuspecting victims!20:26
efriedhaha, I should do that :)20:26
efriedfortunately for the world, I've been completing all my own tasks20:26
efriedpatches in seven new-to-me projects since yesterday20:27
fungiSotK: i noticed that the story about not being able to save something if you type the full string instead of a substring is expected to be fixed by upgrading the typeahead lib. i assume we need to bump the version of it somewhere and then get that into an updated webclient tarball?20:28
SotKyeah we'll need to bump the version in package.json20:29
fungicool, i'll give that a shot once i get the webclient drafts sorted20:30
SotKhowever recent enough versions have a slightly different naming convention for the various attribute names it uses so there'll likely be some amount of find/replace needed20:30
SotK(iirc its just a simple prepending of uib- to most of them, but I'd need to check to be sure)20:31
fungicool, will keep that in mind. thanks!20:32
openstackgerritAdam Coldrick proposed opendev/storyboard-webclient master: Render user-submitted links with nofollow attribute  https://review.opendev.org/64832220:41
openstackgerritAdam Coldrick proposed opendev/storyboard-webclient master: Render user-submitted links with nofollow attribute  https://review.opendev.org/64832220:42
openstackgerritMerged opendev/storyboard-webclient master: Fix user_typeahead result rendering  https://review.opendev.org/69040020:44
*** cgoncalves has quit IRC20:49
*** cgoncalves has joined #storyboard20:55
openstackgerritAdam Coldrick proposed opendev/storyboard-webclient master: Allowing the user to choose what Columns are seen  https://review.opendev.org/65127021:02
openstackgerritAdam Coldrick proposed opendev/storyboard-webclient master: Render user-submitted links with nofollow attribute  https://review.opendev.org/64832221:12
efriedSotK: assignment is now working, thank you!  Though the search drop-down doesn't render quiiite right: https://imgur.com/a/fnPM4Vr21:32
*** tosky has quit IRC22:56
*** diablo_rojo has quit IRC23:47

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