[10:27:02 CDT(-0500)] <EricDalquist> ok athena
[10:27:06 CDT(-0500)] <EricDalquist> I got that minimized bug fixed
[10:27:11 CDT(-0500)] <EricDalquist> as hackey as it is
[10:34:29 CDT(-0500)] <drewwills> morning folks... i'd like to discuss adding a page like this one (https://wiki.jasig.org/display/~awills/uPortal+Practices+for+Tags%2C+Branches%2C+and+Releases) to the uP developers area (https://wiki.jasig.org/display/UPC/uPortal+Developers)
[10:35:07 CDT(-0500)] <drewwills> SSP wants to adopt a statement in this area by refering to accepted/actual practices in a leading Jasig project
[10:35:20 CDT(-0500)] <EricDalquist> https://wiki.jasig.org/display/UPC/Release+Strategy
[10:35:55 CDT(-0500)] <EricDalquist> you could merge your page with that page
[10:35:59 CDT(-0500)] <EricDalquist> and then have SSP point to that
[10:36:15 CDT(-0500)] <drewwills> fyi i can search for that page, but clicking the link comes up blank
[10:36:30 CDT(-0500)] <EricDalquist> athena: can you get to https://wiki.jasig.org/display/UPC/Release+Strategy
[10:36:46 CDT(-0500)] <athena> yep
[10:36:49 CDT(-0500)] <drewwills> bleh now it works
[10:52:26 CDT(-0500)] <athena> are you cutting the release today EricDalquist?
[10:52:33 CDT(-0500)] <EricDalquist> hopefully
[10:52:36 CDT(-0500)] <EricDalquist> I'm going through jira again
[10:52:42 CDT(-0500)] <EricDalquist> seeing if there is any other easy stuff
[10:52:48 CDT(-0500)] <athena> :
[11:11:06 CDT(-0500)] <drewwills> there you go, merged: https://wiki.jasig.org/display/UPC/Release+Strategy
[11:11:42 CDT(-0500)] <EricDalquist> very nice. thanks drewwills
[11:40:09 CDT(-0500)] <athena> EricDalquist: if you have a person attribute dao that uses an attribute from a previous dao in the query, does that just work, or do you have to do something special?
[11:40:29 CDT(-0500)] <EricDalquist> you have to set them up under a cascading dao
[11:40:32 CDT(-0500)] <athena> so if LDAP runs first and you have a JDBC dao after, can you just add one of the LDAP attributes to the JDBC dao query builder?
[11:40:34 CDT(-0500)] <EricDalquist> I think the default config has an example of that
[11:40:48 CDT(-0500)] <EricDalquist> I have dreams of making it automagical
[11:42:04 CDT(-0500)] <athena> hm.
[11:42:14 CDT(-0500)] <athena> not really obvious from the default config which part it is, if it's there
[11:42:30 CDT(-0500)] <EricDalquist> huh apparently not
[11:42:46 CDT(-0500)] <athena> i assuming it's the thing here: https://wiki.jasig.org/display/PDM15/Attribute+Aggregation
[11:42:52 CDT(-0500)] <EricDalquist> https://wiki.jasig.org/display/PDM15/Example+Configuration
[11:43:08 CDT(-0500)] <EricDalquist> the example xml file linked there
[11:43:13 CDT(-0500)] <EricDalquist> has a cascading dao in it
[11:43:27 CDT(-0500)] <EricDalquist> drewwills: for https://issues.jasig.org/browse/UP-3535 how is that work going?
[11:45:21 CDT(-0500)] <athena> so can this replace the mergingpersonattributedao?
[11:45:28 CDT(-0500)] <EricDalquist> yes
[11:45:41 CDT(-0500)] <EricDalquist> so merging calls the child daos in order each with the same query
[11:46:03 CDT(-0500)] <EricDalquist> cascading calls them in order adding the results of each previous dao to the next query
[11:46:53 CDT(-0500)] <athena> ok, so can probably just do a simple replacement, but add the echo thing to the beginning of the list
[11:46:58 CDT(-0500)] <EricDalquist> yeah
[11:47:42 CDT(-0500)] <athena> thanks!
[11:47:52 CDT(-0500)] <athena> sounds like some info that needs to go into the manual
[11:48:47 CDT(-0500)] <EricDalquist> yeah
[11:48:48 CDT(-0500)] <EricDalquist> that would be good
[11:48:52 CDT(-0500)] <EricDalquist> also can you also do a review of: https://issues.jasig.org/browse/UP-3249
[11:49:02 CDT(-0500)] <EricDalquist> steve has a pull request for that
[11:49:04 CDT(-0500)] <EricDalquist> but I don'
[11:49:07 CDT(-0500)] <drewwills> eric I haven't been able to start on that this week... i almost listed the fix version as 4.0.7 and version 4.0.6 in the applies to versions... but figured that might be confusing
[11:49:08 CDT(-0500)] <EricDalquist> t want to lose some functionality
[11:49:35 CDT(-0500)] <EricDalquist> sounds good drew just wanted to make sure I didn't miss it if you were almost done with iut
[11:50:21 CDT(-0500)] <drewwills> yeah thanks
[11:50:54 CDT(-0500)] <EricDalquist> also jen do you have any insight on https://github.com/Jasig/uPortal/pull/51
[11:52:24 CDT(-0500)] <EricDalquist> I'm thjinking that is ok as it appears that CAS_NEW_USER_URL is only configurable via changing it in universality.xml
[11:52:26 CDT(-0500)] <EricDalquist> er xsk
[11:52:27 CDT(-0500)] <EricDalquist> xsl
[12:05:32 CDT(-0500)] <athena> well
[12:05:36 CDT(-0500)] <athena> that link never worked
[12:05:47 CDT(-0500)] <athena> it was put in as a placeholder for schools that wanted to use it
[12:05:57 CDT(-0500)] <athena> so i guess the real question is what do we actually want there?
[12:06:09 CDT(-0500)] <athena> i don't think it's great to have links that don't work
[12:06:24 CDT(-0500)] <athena> though if we take it out, anyone that wants to use it has to figure out how to create it themselves
[12:19:14 CDT(-0500)] <EricDalquist> ok
[12:19:22 CDT(-0500)] <EricDalquist> perhaps we leave it but undefine the variable
[12:19:29 CDT(-0500)] <EricDalquist> and add an xsl:if
[12:19:36 CDT(-0500)] <EricDalquist> to only display it if the var is set
[12:19:42 CDT(-0500)] <EricDalquist> though then someone needs to document that variable
[12:19:48 CDT(-0500)] <athena> yeah
[15:52:50 CDT(-0500)] <chris_cousquer> hi athena
[15:53:00 CDT(-0500)] <athena> heya
[15:54:48 CDT(-0500)] <chris_cousquer> is it normal that I receave in my mail box some mail like "You can merge this Pull Request by running:" for pull resquest from Aaron grant ?
[15:55:21 CDT(-0500)] <athena> yes, probably
[15:55:27 CDT(-0500)] <athena> i assume it's on a project for which you're a committer?
[15:55:44 CDT(-0500)] <chris_cousquer> I think I don't have the right to...
[15:55:53 CDT(-0500)] <athena> you might be able to change the way github sends you notifications if they're irritating
[15:56:16 CDT(-0500)] <athena> which project is the pull request for?
[15:56:28 CDT(-0500)] <chris_cousquer> UP