fluid-work IRC Logs-2013-05-30

[13:25:43 CDT(-0500)] <cindyli> hi, Bosmon

[13:47:07 CDT(-0500)] <Bosmon7> Hi cindyli

[13:47:13 CDT(-0500)] <Bosmon7> I am looking at your latest bug report now

[13:47:21 CDT(-0500)] <Bosmon7> But there seems to be a typo in your test case

[13:47:27 CDT(-0500)] <cindyli> oh, where

[13:47:35 CDT(-0500)]

[13:47:46 CDT(-0500)] <Bosmon7> Whereas the option itself is deeperSubComponentOption: {

[13:47:58 CDT(-0500)] <cindyli> ah, exactly. sorry. trying again

[13:48:43 CDT(-0500)] <cindyli> good catch. the test case works. sigh…

[13:49:02 CDT(-0500)] <Bosmon7> great!

[13:49:03 CDT(-0500)] <cindyli> have to figure out the cause again

[13:49:21 CDT(-0500)] <Bosmon7> Can you describe the symptoms you are seeing?

[13:49:56 CDT(-0500)] <cindyli> regarding these lines: https://github.com/cindyli/infusion/blob/FLUID-5010/src/webapp/components/uiOptions/js/FullPreviewUIOptions.js#L30-35

[13:50:49 CDT(-0500)] <cindyli> to distribute "outerPreviewEnhancerOptions" into preview.enhancer, i was using

[13:50:56 CDT(-0500)] <cindyli> distributeOptions: [{

[13:50:56 CDT(-0500)]

[13:50:57 CDT(-0500)] <cindyli> removeSource: true,

[13:50:57 CDT(-0500)] <cindyli> exclusions: [],

[13:50:57 CDT(-0500)]

[13:50:57 CDT(-0500)] <cindyli> }],

[13:51:27 CDT(-0500)] <cindyli> the options cannot reach preview.enhancer

[13:51:33 CDT(-0500)] <Bosmon7> Btw, you can now omit the empty exclusions: [] block

[13:51:37 CDT(-0500)] <Bosmon7> That was fixed along with your other fixes

[13:51:43 CDT(-0500)] <cindyli> cool.

[13:52:14 CDT(-0500)] <Bosmon7> Is "preview" a legitimate subcomponent now?

[13:52:18 CDT(-0500)] <Bosmon7> How is it defined

[13:52:42 CDT(-0500)] <cindyli> defined here: https://github.com/cindyli/infusion/blob/FLUID-5010/src/webapp/components/uiOptions/js/FullPreviewUIOptions.js#L45-57

[13:53:16 CDT(-0500)] <Bosmon7> Yes, but I think we need to eliminate "derivedDefaults" as wlel

[13:53:28 CDT(-0500)] <Bosmon7> If "preview" is not genuinely configured as a subcomponent it will not be visible to IoC

[13:53:46 CDT(-0500)] <Bosmon7> This "derivedDefaults" rubbish should just be in a grade somewhere

[13:54:05 CDT(-0500)] <cindyli> not yet. since due to FLUID-5025, the mapping code for uiOptions hasn't been replaced by IoCSS

[13:54:13 CDT(-0500)] <Bosmon7> FLUID-5025!

[13:54:20 CDT(-0500)] <cindyli> ya

[13:54:21 CDT(-0500)] <cindyli> https://github.com/cindyli/infusion/blob/FLUID-5010/src/webapp/components/uiOptions/js/UIOptions.js#L84-88

[13:54:55 CDT(-0500)] <Bosmon7> FLUID-5025 can't be fixed either

[13:55:04 CDT(-0500)] <Bosmon7> Since we plan not to support preInit or postInit

[13:55:14 CDT(-0500)] <Bosmon7> I am very sorry that I will need to close both of these JIRAs : P

[13:55:17 CDT(-0500)] <cindyli> what a loop

[13:55:26 CDT(-0500)] <Bosmon7> Unfortunately you will need to refactor the whole component at once

[13:55:41 CDT(-0500)] <cindyli> ok, i see. will try

[13:58:57 CDT(-0500)] <Bosmon7> Good luck

[14:28:22 CDT(-0500)] <system64> Hello michelled!

[14:30:19 CDT(-0500)] <michelled> hi system64! how are things?

[14:31:11 CDT(-0500)] <system64> Thanks for selecting me

[14:32:24 CDT(-0500)] <michelled> system64: you're welcome - I'm looking forward to working with you on this project

[14:32:50 CDT(-0500)] <system64> I have been trying out Meteor these days, Is it possible to use Meteor instead of Express (as proposed)

[14:33:34 CDT(-0500)] <michelled> Bosmon, anastasiac: with all the new framework changes, I'm feeling like the framework documentation needs a big overhaul.

[14:34:02 CDT(-0500)] <michelled> system64: yes, if you find something you'd prefer to use, go for it

[14:34:25 CDT(-0500)] <michelled> anastasiac: is there a list somewhere of documentation that's going to be updated?

[14:35:00 CDT(-0500)] <anastasiac> michelled, no, there is no list, except "everywhere"

[14:35:07 CDT(-0500)] <anastasiac> it would be good to start one

[14:35:24 CDT(-0500)] <anastasiac> so that everyone has a place to drop notes on what's changed

[14:35:58 CDT(-0500)] <michelled> the reason that I bring it up, is because with the modifications to inline edit that were pushed today, I think we need to change how we recommend people use 'undo'

[14:37:31 CDT(-0500)] <anastasiac> I can start a wiki page outlining the info that needs to be documented. Then, Bosmon can add notes to that page on the "new way" of using Undo. Does that seem reasonable?

[14:37:46 CDT(-0500)] <anastasiac> then, I can work on massaging the notes into actual docs changes

[14:37:53 CDT(-0500)] <michelled> Bosmon: would that work for you?

[14:38:13 CDT(-0500)] <Bosmon7> That's fine

[14:38:19 CDT(-0500)] <Bosmon7> I should have added a test case demonstrating the "new way"

[14:39:25 CDT(-0500)] <michelled> anastasiac: it looks like this page might be a start on your list: http://wiki.fluidproject.org/display/docs/API all the things marked with *

[14:39:26 CDT(-0500)] <Bosmon7> Which basically involved attaching a "fluid.undo" instance as a standard IoC subcomponent

[14:39:49 CDT(-0500)] <Bosmon7> But we were committed to strong API backwards capability with InlineEdit since it was a fully released component

[14:40:07 CDT(-0500)] <Bosmon7> We should still mark the "old method" as deprecated, however -

[14:40:27 CDT(-0500)] <Bosmon7> Our plan is to make an "Infusion 1.9" release which is the last one for which we commit to support for obsolete framework features and APIs

[14:40:36 CDT(-0500)] <Bosmon7> And eliminate them all in "Infusion 2.0" following shortly afterwards

[14:40:39 CDT(-0500)] <anastasiac> michelled, Bosmon: http://wiki.fluidproject.org/display/fluid/Infusion+1.5+Documentation+Changes I'll also announce this page to the list

[14:41:28 CDT(-0500)] <Bosmon7> I guess it's something that the community needs to discuss further, but it is seeming like the release we currently label as "1.5" will in fact end up being "1.9"

[14:43:38 CDT(-0500)] <system64> michelled: Thanks, could you suggest any improvements I could make to my timeline or my approach in general?

[14:44:25 CDT(-0500)] <michelled> system64: I think your plan looks good. we might want to tweak it once you get started

[14:44:33 CDT(-0500)] <michelled> you're sure to run into things as you implement

[14:49:46 CDT(-0500)] <system64> michelled: Since this my first time working on GSoC project, could you tell me how will I be working with the community?

[14:52:38 CDT(-0500)] <michelled> system64: sharing out what you've done on the list and in irc is a good start

[14:52:45 CDT(-0500)] <michelled> asking questions here is also great

[14:53:28 CDT(-0500)] <michelled> and as you work, you might come across other places where you can contribute, such as improving documentation or fixing bugs that you come across

[14:53:43 CDT(-0500)] <Bosmon7> Or at least creating test cases for them

[14:54:30 CDT(-0500)] <michelled>

[14:59:54 CDT(-0500)] <system64> Sure, I'll try to contribute in any way I can.