Page 1 of 1

Ins Benefits in account

Posted: Mon Jun 02, 2008 9:44 am
by drtech
Jordan,
Question...did you want the ins benefits to not show on the account for speed or another reason or is that a bug? (that maybe I introduced yesterday when I compiled 5.7.) Been using it at the office today and hadn't had a chance to look at the code, but it doesn't show up until you mouse over it.

Re: Ins Benefits in account

Posted: Mon Jun 02, 2008 7:17 pm
by jordansparks
I'm not looking at the code right now, but the intention was to not make those expensive calls to the database until that panel was made visible by hover over. I don't remember actually testing it, so it's likely that I introduced a bug. I will take another look at it. But we cannot allow those expensive calls separate from the dataset.

Re: Ins Benefits in account

Posted: Tue Jun 03, 2008 6:27 am
by drtech
yes, I figured it was for speed and efficiency...can I make it part of the "data sheet"? do you mean include it as part of the business layer so everything is returned with one call? I will look at it too and try to see what exactly you mean.

Re: Ins Benefits in account

Posted: Tue Jun 03, 2008 2:30 pm
by jordansparks
I'm OK with not making it part of the business layer because I consider it "fluff". In fact, I think it's better to call it on an as-needed basis. This is assuming that it will not be needed until the popup panel shows. Anything that must show on the main screen right from the start, we should try hard to include in the business layer. Please don't touch it until after Wed, when I get back from vacation. I also don't want to clutter the business layer unnecessarily.

Re: Ins Benefits in account

Posted: Tue Jun 03, 2008 5:39 pm
by drtech
sure thing