fluid-work IRC Logs-2010-10-05

Owned by Fluid IRC Log Bot
Last updated: Oct 05, 2010Version comment
5 min read

[09:37:11 CDT(-0500)] * Topic is 'This channel is logged – for details see: http://wiki.fluidproject.org/display/fluid/IRC+Channel' set by jessm on 07:30:00 CST(-0600)
[10:07:25 CDT(-0500)] <colinclark> mlam, Justin_o: Nice patch for Jorn, thanks!
[10:08:07 CDT(-0500)] <mlam> Thanks! Hope Jorn gets back to us
[10:11:54 CDT(-0500)] <Justin_o> colinclark: I was chatting with bosmon yesterday and we thought of another bug when running multiple infusions. http://issues.fluidproject.org/browse/FLUID-3783 I've updated the multiple-infusions-test example in the scratchpad to show the issue
[10:12:19 CDT(-0500)] <colinclark> What's the issue?
[10:12:36 CDT(-0500)] <Justin_o> basically the guid is only unique within a single instance of infusion, so you could end up having duplicate ids on a page for example
[10:13:02 CDT(-0500)] <Justin_o> anyways... wondering if you think this warrants being on the bug parade for 1.3 or we should wait for 1.4
[10:14:34 CDT(-0500)] <golam> Justin_o:please let me know when you have some time so I can clear few things about the events
[10:14:54 CDT(-0500)] <colinclark> Justin_o: I think it should probably wait until 1.4
[10:14:55 CDT(-0500)] <golam> oops special char got in
[10:15:04 CDT(-0500)] <colinclark> not ideal
[10:15:27 CDT(-0500)] <Justin_o> golam: if you still need me, i should be free in a minute or two
[10:15:32 CDT(-0500)] <Justin_o> colinclark: okay
[10:15:33 CDT(-0500)] <golam> cool
[10:23:38 CDT(-0500)] <Justin_o> jhung, jameswy: do you guys know which of you will be looking after http://issues.fluidproject.org/browse/FLUID-3731
[10:24:59 CDT(-0500)] <jhung> justin_o: if Jameswy is already loaded, I can take care of it.
[10:26:21 CDT(-0500)] <jameswy> jhung: Thanks for volunteering, (smile)
[10:50:54 CDT(-0500)] <jhung> justin_o: are demo updates for reorderer, inline edit, and uploader part of bug parade? (i.e. design and development?)
[10:52:00 CDT(-0500)] <Justin_o> jhung: sorry..missed your previous messages
[10:52:23 CDT(-0500)] <Justin_o> jhung: yep those demo updates should be on bug parade i believe
[10:52:39 CDT(-0500)] <colinclark> jhung: I didn't fully understand your question at standup
[10:52:49 CDT(-0500)] <colinclark> Can we talk through it here for a second with Justin_o?
[10:53:03 CDT(-0500)] <jhung> colinclark: sure.
[10:53:21 CDT(-0500)] <colinclark> Ok, so you've got a new wireframe for an Inline Edit demo, right?
[10:53:35 CDT(-0500)] <jhung> Yes.
[10:53:43 CDT(-0500)] <colinclark> And people seemed to respond well to it, right?
[10:53:46 CDT(-0500)] <jhung> yes.
[10:54:02 CDT(-0500)] <colinclark> I can't remember if there were any changes we were thinking of for it, or is it pretty much ready to go?
[10:54:27 CDT(-0500)] <jhung> There are some minor changes, but the same design.
[10:54:31 CDT(-0500)] <colinclark> Ok, great
[10:54:50 CDT(-0500)] <colinclark> So, the previous design, we really don't want to implement, right?
[10:54:54 CDT(-0500)] <colinclark> The more form-like one
[10:55:07 CDT(-0500)] <jhung> My confusion comes from the updates done to the existing inline edit demo done late last week for Jan's AEGIS presentation.
[10:55:19 CDT(-0500)] <jhung> That was just a stop-gap?
[10:55:26 CDT(-0500)] <colinclark> jhung: No updates were done to the existing Inline Edit demo
[10:55:37 CDT(-0500)] <colinclark> mlam and I made some improvements to the component itself
[10:55:41 CDT(-0500)] <jhung> colinclark: ah. okay.
[10:55:45 CDT(-0500)] <colinclark> Which of course come for free with the demo
[10:55:56 CDT(-0500)] <colinclark> You had made a folder in the incubator for the markup in your previous demo, right?
[10:56:14 CDT(-0500)] <jhung> yes.
[10:56:28 CDT(-0500)] <colinclark> This one? https://source.fluidproject.org/svn/incubator/inlineEdit/FLUID-3752-demo/
[10:56:52 CDT(-0500)] <jhung> yep. That will be replaced with the new design.
[10:56:56 CDT(-0500)] <colinclark> Ok
[10:57:11 CDT(-0500)] <colinclark> So, we should be sure to do a couple of things for the new design:
[10:57:39 CDT(-0500)] <colinclark> 1. Make sure we're actually branching from trunk when creating these sorts of things in the incubator. Otherwise the lineage of the code is unclear
[10:58:42 CDT(-0500)] <colinclark> 2. Follow our license and copyright guidelines. So I guess in this particular example, the copyright statement for Inline Edit's JavaScript code somehow got changed to include only OCAD instead of all the copyright owners--Cambridge, Berkeley, and U of T
[10:58:47 CDT(-0500)] <colinclark> Make sense?
[10:59:37 CDT(-0500)] <colinclark> So it seems to me that our next steps for bug parade might look something like this...
[10:59:47 CDT(-0500)] <colinclark> a. Remove the old demo directory from the incubator
[10:59:58 CDT(-0500)] <colinclark> b. Polish your new design, jhung
[10:59:58 CDT(-0500)] <jhung> 1. Didn't branch because it was unclear which inline edit branch was going to be used (described by or tooltip). So FLUID-3752 was put into incubator as just markup.
[11:00:19 CDT(-0500)] <colinclark> c. When Justin_o and mlam are ready, they can implement the new demo with you
[11:00:26 CDT(-0500)] <jhung> Do we know which approach we will be using so we can branch for FLUID-3752 properly?
[11:00:38 CDT(-0500)] <jhung> okay
[11:01:17 CDT(-0500)] <mlam> jhung: i should be ready first thing tmr
[11:01:39 CDT(-0500)] <colinclark> When mlam and Justin_o settle on a good working approach for Inline Edit, I think we'll fold their changes into trunk right away
[11:01:50 CDT(-0500)] <jhung> cool.
[11:01:56 CDT(-0500)] <colinclark> And we can work on the demo either directly in trunk or in the incubator depending on what is most convenient for everyone
[11:02:07 CDT(-0500)] <jhung> I'll remove existing FLUID-3752 now.
[11:02:12 CDT(-0500)] <Justin_o> colinclark, jhung: that sounds good
[11:02:14 CDT(-0500)] <colinclark> The King has included JIRAs for this work in the Bug Parade, so we should be good
[11:02:32 CDT(-0500)] <colinclark> golam: I think, since we haven't heard any complaints, that we're also ready to merge your branch into trunk, too
[11:02:35 CDT(-0500)] <colinclark> Along with the new demo
[11:02:48 CDT(-0500)] <golam> ok
[11:02:55 CDT(-0500)] <colinclark> jhung and jameswy: You guys feel good about replacing the old Progress demo with golam and jhung's shiny new one?
[11:04:13 CDT(-0500)] <Justin_o> jameswy: i think this is the link to try it from http://build.fluidproject.org/incubator/progress/FLUID-3751/src/webapp/demos/progress/html/progress.html
[11:05:22 CDT(-0500)] <jameswy> colinclark, Justin_o, jhung, golam: Yep, it's a good improvement.
[11:06:01 CDT(-0500)] <Justin_o> the only thing i'm slightly worried about is using the name of a real grocery store.. don't know if that matters or not
[11:07:01 CDT(-0500)] <jhung> justin_o: True. Although I don't know of many other helicopter grocery delivery services out there. (smile)
[11:07:25 CDT(-0500)] <jhung> justin_o: the name should change just to be clear.
[11:08:19 CDT(-0500)] <Justin_o> jhung: (smile) i wish i could get my groceries delivered by helicopter
[11:08:37 CDT(-0500)] <jhung> justin_o: that change should be done after the move to trunk. I'll file a new JIRA for it?
[11:09:23 CDT(-0500)] <Justin_o> jhung: probably don't need a new jira for it, since it's part of implementing the demo
[11:09:44 CDT(-0500)] <jhung> ok.
[14:40:28 CDT(-0500)] <jhung> mlam: I'm reassigning Fluid-3752 to you. (smile)
[14:40:42 CDT(-0500)] <mlam> ok (smile)
[14:45:14 CDT(-0500)] <mlam> jhung: do you have an image for me to build the demo?
[14:45:59 CDT(-0500)] <jhung> I've attached them to the JIRA.
[14:46:11 CDT(-0500)] <mlam> thanks
[14:46:33 CDT(-0500)] <jhung> mlam, let me know if that is enough or if you need clarification.
[14:49:14 CDT(-0500)] <mlam> ok, there should be enough for sure , thanks