[09:05:48 CDT(-0500)] <cindyli> Justin_o, michelled, with regards to the renaming of UIO to prefs framework, the last remaining issue as I know is whether the demo that uses grade components should be moved into manual tests. do we have a decision on that?
[09:08:27 CDT(-0500)] <michelled> I think it should be moved
[09:08:45 CDT(-0500)] <michelled> cindyli: we are expecting most people to use the schema, right?
[09:12:30 CDT(-0500)] <cindyli> michelled: yes, i agree. Justin_o, anastasiac, let me know if you have another opinion. otherwise, i will start moving it
[09:19:37 CDT(-0500)] <Justin_o> cindyli: i also think it should be moved
[09:20:15 CDT(-0500)] <cindyli> ok
[09:32:24 CDT(-0500)] <Justin_o> jhung: I merged in your GPII-204 pull request
[09:33:06 CDT(-0500)] <jhung> Thanks Justin_o
[10:01:21 CDT(-0500)] <cindyli> michelled: i sent a pull request for UIO renaming - https://github.com/fluid-project/infusion/pull/414 Can you review it when you have a chance?
[10:02:39 CDT(-0500)] <michelled> thanks cindyli, Justin_o will take a look at it
[10:02:48 CDT(-0500)] <cindyli> ok. thanks
[12:07:53 CDT(-0500)] <Justin_o> michelled: did you fix for the styling on the contrast panel go in?
[12:08:19 CDT(-0500)] <michelled> I think so Justin_o - I saw the pull request close
[12:08:42 CDT(-0500)] <michelled> and it looks fixed on the nightly build too
[12:08:53 CDT(-0500)] <Justin_o> michelled: great thanks
[12:08:59 CDT(-0500)] <michelled> np
[12:19:06 CDT(-0500)] <clown> jhernandez, ping
[12:44:33 CDT(-0500)] <Justin_o> cindyli: did you want help with the build related comments i left on the pull request?
[12:45:15 CDT(-0500)] <cindyli> Justin_o: yes
[12:45:40 CDT(-0500)] <cindyli> Justin_o: have you done with all the reviews?
[12:46:27 CDT(-0500)] <Justin_o> cindyli: i think so
[12:46:34 CDT(-0500)] <cindyli> i left a comment regarding moving UIO demo to framework demos. I don't see such a directory for framework demos, or you meant to create one?
[12:46:45 CDT(-0500)] <Justin_o> i replied to that one
[12:46:55 CDT(-0500)] <cindyli> cool. thanks, will take a look
[12:47:08 CDT(-0500)] <Justin_o> basically there is no directory for it.. but it's organized in the demo portal.
[12:48:20 CDT(-0500)] <cindyli> cool. got you, Justin_o
[12:49:42 CDT(-0500)] <Justin_o> anastasiac, cindyli: any thoughts about renaming generateUIOThemes. I'm thinking generatePEThemes or generatePrefsEditorThemes
[12:50:06 CDT(-0500)] <Justin_o> The first is shorter, but could be confused with progressive enhancement or something else
[12:50:25 CDT(-0500)] <cindyli> +1 on the second
[12:50:52 CDT(-0500)] <anastasiac> +1 on the second one, Justin_o
[12:51:04 CDT(-0500)] <Justin_o> cindyli, anastasiac: thanks
[12:52:45 CDT(-0500)] <jhernandez> clown: pong
[12:52:57 CDT(-0500)] <clown> hi jhernandez
[12:53:05 CDT(-0500)] <clown> did you see my email from Friday?
[12:53:09 CDT(-0500)] <jhernandez> of course
[12:53:18 CDT(-0500)] <jhernandez> :]
[12:53:34 CDT(-0500)] <clown> did you get chance to look at the new version of the tests?
[12:54:03 CDT(-0500)] * clown waiting for another "of course"
[12:54:15 CDT(-0500)] <jhernandez> heh
[12:54:23 CDT(-0500)] <jhernandez> of course, but didn't tried them
[12:54:49 CDT(-0500)] <clown> okay. I was just worried that you didn't know about them.
[12:54:51 CDT(-0500)] <jhernandez> it's time to try them, right now!
[12:55:03 CDT(-0500)] <clown> But you are aware of them, and it is a work in progress.
[12:55:08 CDT(-0500)] <clown> oh, okay. Cool.
[12:55:41 CDT(-0500)] * clown waiting for comments and requests for changes.
[12:55:53 CDT(-0500)] * jhernandez pulling your branch ...
[12:56:47 CDT(-0500)] * clown crosses fingers
[12:58:25 CDT(-0500)] <jhernandez> you don't have to
[12:58:32 CDT(-0500)] <jhernandez> you're a rock star!
[12:58:33 CDT(-0500)] <jhernandez>
[12:59:16 CDT(-0500)] <jhernandez> and ... re your friday's email
[12:59:27 CDT(-0500)] <jhernandez> you don't have to create another pull request
[13:00:16 CDT(-0500)] <jhernandez> just keep commiting on the same branch, and the commits will be included in tue pull request
[13:00:17 CDT(-0500)] <clown> ah, okay. I was wondering about that.
[13:00:31 CDT(-0500)] <clown> cool. I guessed correctly!
[13:00:35 CDT(-0500)] <clown> wonders never cease.
[13:00:43 CDT(-0500)] <jhernandez> yup!
[13:01:19 CDT(-0500)] <clown> what do you think about the console.log()s I put in? I put them there to get some idea of what was being tested, but is there a better way?
[13:06:02 CDT(-0500)] <jhernandez> i am ok with that
[13:06:11 CDT(-0500)] <clown> \o/
[13:06:16 CDT(-0500)] <jhernandez> (and with the latest changes ofc)
[13:08:08 CDT(-0500)] <clown> so, in terms of mechanics, does this mean you pull into your branch and this work is finished (for now)?
[13:09:45 CDT(-0500)] <jhernandez> merged
[13:11:08 CDT(-0500)] <jhernandez> now we have a packagekit add-on within their associated tests in jqUnit
[13:11:22 CDT(-0500)] <clown> okay.
[13:11:57 CDT(-0500)] <clown> what does this mean about my "packageBridgeTests" branch? I guess I just switch to your branch, and do an update.
[13:12:19 CDT(-0500)] <clown> something like git pull --rebase.
[13:12:39 CDT(-0500)] <jhernandez> now, we have almost the same with Android (still working on it because I still need to add install and remove capabilities)
[13:13:02 CDT(-0500)] <clown> that's good, jhernandez.
[13:13:09 CDT(-0500)] <clown> it's good that it's mostly "common".
[13:13:18 CDT(-0500)] <jhernandez> yeah, now switch back to my branch, fetch my repo and pull from it
[13:13:30 CDT(-0500)] <clown> Okay.
[13:14:50 CDT(-0500)] <Justin_o> cindyli: I've fixe the two build related issues in my FLUID-5161 branch
[13:15:02 CDT(-0500)] <Justin_o> cindyli: did you need help with any of the other parts?
[13:15:10 CDT(-0500)] <clown> I'm thinking the next step is more unit tests, but from a webserver perspective, along the lines of these tests: https://github.com/javihernandez/linux/tree/packagekitBridge/node_modules/gsettingsBridge/tests
[13:15:26 CDT(-0500)] <clown> (that's a weird url).
[13:15:42 CDT(-0500)] <cindyli> thanks, Justin_o, i will merge in your branch. can you take a look on contrast icons
[13:15:53 CDT(-0500)] <Justin_o> cindyli: sure will do
[13:16:00 CDT(-0500)] <cindyli> thanks, Justin_o
[13:16:05 CDT(-0500)] <Justin_o> cindyli: can you also double check my changes when you pull them in
[13:16:14 CDT(-0500)] <cindyli> sure
[13:16:18 CDT(-0500)] <Justin_o> thanks
[13:18:15 CDT(-0500)] <jhernandez> clown: more or less
[13:19:41 CDT(-0500)] <clown> I'll make a start on it, then.
[13:20:03 CDT(-0500)] <jhernandez> right now we should start thinking on the deviceReporter integration from the core architecture perspective
[13:20:26 CDT(-0500)] <clown> okay.