jasig-ssp IRC Logs-2013-04-11

Owned by Apereo Infrastructure
Last updated: Apr 11, 2013
2 min read

[10:46:40 CDT(-0500)] <js70> failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:2.3.2:compile (default-compile) on project ssp: Compilation failure

[10:46:41 CDT(-0500)] <js70> [ERROR] /Users/jamesstanley/Documents/opensource/SSP/src/main/java/org/jasig/ssp/service/impl/TemplateServiceImpl.java:[45,1] method does not override or implement a method from a supertype

[10:48:39 CDT(-0500)] <TonyUnicon> ok ill fix it right now

[10:48:46 CDT(-0500)] <js70> kool

[10:49:47 CDT(-0500)] <TonyUnicon> fixed

[10:49:49 CDT(-0500)] <TonyUnicon> pull

[10:49:51 CDT(-0500)] <TonyUnicon> sorry bout that

[10:58:12 CDT(-0500)] <js70> np and thanks

[10:59:09 CDT(-0500)] <js70> I needed some coffee anyway.

[11:04:04 CDT(-0500)] <TonyUnicon> hehe, i'm already two cups in

[11:04:51 CDT(-0500)] <TonyUnicon> Dan, I introduced plan rows, rowcounts, and filters to the get API call

[11:04:57 CDT(-0500)] <TonyUnicon> if you want to take a quick look

[11:05:12 CDT(-0500)] <TonyUnicon> I think that should mirror the doc

[11:06:24 CDT(-0500)] <dmccallum54> great thx

[11:07:19 CDT(-0500)] <dmccallum54> updating that api doc is #4 on my todos today

[11:07:24 CDT(-0500)] <dmccallum54> will take a look then

[11:12:05 CDT(-0500)] <TonyUnicon> we mentioned before that there was no rule against exposing API calls that will make our lives easier for our UI

[11:12:33 CDT(-0500)] <dmccallum54> yes

[11:12:38 CDT(-0500)] <TonyUnicon> I think exposing one to fetch just the current map for a student is a case like this

[11:12:46 CDT(-0500)] <dmccallum54> no objection

[11:12:52 CDT(-0500)] <TonyUnicon> ok cool

[11:13:44 CDT(-0500)] <TonyUnicon> I think we will have a little heartburn serializing and deserializing the plan model and being able to deal with single instances may help

[11:15:34 CDT(-0500)] <dmccallum54> if it makes life easier go for it.

[11:15:51 CDT(-0500)] <dmccallum54> lets' just not delete support for the API to get at the entire collection of plans for a user

[11:16:29 CDT(-0500)] <TonyUnicon> for a student yes?

[11:16:57 CDT(-0500)] <TonyUnicon> that should be what the current 'get' call represents ATM

[11:18:54 CDT(-0500)] <dmccallum54> yes, always "person scoped" where person is usually a student

[11:21:02 CDT(-0500)] <dmccallum54> merge commit alert! merge commit alert!

[11:23:03 CDT(-0500)] <dmccallum54> TonyUnicon looked at PlanController. get() looks good w/r/t currently spec'd API

[11:30:11 CDT(-0500)] <TonyUnicon> yes merge commit was my fault

[11:30:13 CDT(-0500)] <TonyUnicon> slip of the finger

[11:30:20 CDT(-0500)] <TonyUnicon> thanks for looking at the controller

[13:14:03 CDT(-0500)] <dmccallum54> SSP-1000!

[13:14:05 CDT(-0500)] <dmccallum54> what's my prize?

[13:15:26 CDT(-0500)] <TonyUnicon> http://www.nbcuniversalstore.com/img/product/resized/518/00044960-832518_catl_600.jpg?k=cf0b3250&amp;pid=44960&amp;s=catl&amp;sn=nbcuniversalstore

[13:15:42 CDT(-0500)] <dmccallum54> sweet

[13:16:17 CDT(-0500)] <TonyUnicon> may not fetch as much as an oscar on ebay though

[13:18:45 CDT(-0500)] <TonyUnicon> Patty_ how come remote staff was not invited to ride the mechanical bull at a golf club this weekend? (smile)

[13:19:04 CDT(-0500)] <TonyUnicon> I hope there is an open bar, it will make the mechanical bull sort of pointless