Skip to end of metadata
Go to start of metadata

You are viewing an old version of this page. View the current version.

Compare with Current View Page History

« Previous Version 24 Next »

[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> :(smile)

[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 (smile)

[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

[15:57:05 CDT(-0500)] <athena> dunno - you're right, you're not listed as a committer

[15:57:14 CDT(-0500)] <athena> i'm not quite sure how github decides who to send email to

[15:57:34 CDT(-0500)] <chris_cousquer> :-D

[15:57:36 CDT(-0500)] <athena> maybe you're watching the repository?

[15:57:38 CDT(-0500)] <athena> https://github.com/settings/notifications

[15:57:43 CDT(-0500)] <chris_cousquer> yes

[15:58:01 CDT(-0500)] <athena> that's likely it then - looks like you can update settings at that link

[15:58:07 CDT(-0500)] <athena> wow, apparently i'm watching 60 repositories (tongue)

[15:58:10 CDT(-0500)] <EricDalquist> lol

[15:58:11 CDT(-0500)] * athena needs extra eyes

[15:59:02 CDT(-0500)] <chris_cousquer> so that's it I did this morning cause github has change its notification policy

[15:59:33 CDT(-0500)] <chris_cousquer> I have miss something...

[16:00:41 CDT(-0500)] <chris_cousquer> oh btw I did a pull request on attribute lang on the html tag

[16:01:21 CDT(-0500)] <chris_cousquer> something that render

[16:01:37 CDT(-0500)] <chris_cousquer> html lang="en-US"

[16:02:16 CDT(-0500)] <EricDalquist> yeah I saw that

[16:02:37 CDT(-0500)] <EricDalquist> I was hoping someone with more knowledge of what the heck that does would weigh in on it (smile)

  • No labels