Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

[11:08:14 CDT(-0500)] <dmccallum54> https://issues.jasig.org/browse/SSP-1526

[11:08:21 CDT(-0500)] <dmccallum54> question #1 about Notes permissions

[11:14:20 CDT(-0500)] <pspaude1> Can linux ci be reset? No rush, eventually need to test SSP-1540. Thanks!

[11:14:57 CDT(-0500)] <JasonElwood> building now

[11:15:05 CDT(-0500)] <pspaude1> Cool , thanks!

[12:26:15 CDT(-0500)] <JasonElwood> Paul- ok to build Linux CI again?

[12:37:38 CDT(-0500)] <pspaude1> Jason my current iteration issues are done. Do you want help on QA?

[12:39:46 CDT(-0500)] <JasonElwood> Unless Dan has something else, that would be great. We need to hit the app from all directions

[12:40:29 CDT(-0500)] <dmccallum54> paul can you pls take a look at https://issues.jasig.org/browse/SSP-1529

[12:40:34 CDT(-0500)] <pspaude1> No not Dan.... last time tons of issues (smile)

[12:40:36 CDT(-0500)] <dmccallum54> and see if it's something you feel confident dealing with

[12:40:50 CDT(-0500)] <dmccallum54> if so, pls assign it to yourself and have a grand ol' time

[12:40:59 CDT(-0500)] <pspaude1> Just kidding, no problem I'll take a look, don't mind helping out where needed!

[16:29:52 CDT(-0500)] <pspaude1> Dan, per SSP-1529. So, the remove/reset button for track should stay the same? Kind of seems oddly named. I added a cancel button and you don't want it to go back to server and reload correct?

[16:37:25 CDT(-0500)] <dmccallum54> pspaude1 sorry… gimme another sec

[16:39:36 CDT(-0500)] <pspaude1> No problem still looking at everything and seeing how it works.

[16:43:49 CDT(-0500)] <dmccallum54> ok paul

[16:43:51 CDT(-0500)] <dmccallum54> so

[16:44:16 CDT(-0500)] <dmccallum54> i dont object to going back to the server if that's the cleanest way to discard unwanted changes

[16:45:04 CDT(-0500)] <dmccallum54> my main concern there would just be making sure the list and detail views don't get all out of whack

[16:45:16 CDT(-0500)] <dmccallum54> by that i mean...

[16:45:38 CDT(-0500)] <dmccallum54> let's say we've been clobbering fields left and right in the model object held by the store that backs the list view

[16:45:44 CDT(-0500)] <dmccallum54> we hit cance

[16:45:46 CDT(-0500)] <dmccallum54> cancel

[16:46:06 CDT(-0500)] <dmccallum54> that reloads a model object from the server that the form then points to, but not the list view

[16:46:32 CDT(-0500)] <dmccallum54> so that if you, say, click a different journal entry, then click back on the original journal entry, oops, all your clobbering is still there

[16:52:18 CDT(-0500)] <pspaude1> Sorry answerwing q's to our docs person. Ok, that makes a lot of sense I've done some limited testing and I'll look for that. Actually, the list view is working the best, I'm trying to understand the confidentiality and other fields how to reset those. At first glance server call seems easiest, but if you know of better ways let me know.

[16:53:19 CDT(-0500)] <dmccallum54> the client side option is to defer all form-model bindings to save time

[16:54:46 CDT(-0500)] <dmccallum54> i think some of our forms elsewhere already do that. and you'll some of it going on here as well. e.g. in Ssp.controller.tool.journal.TrackTreeViewController

[16:55:40 CDT(-0500)] <dmccallum54> i'm pretty sure there's a more idiomatic Ext.js way of doing all of this, but we're so far afield of that i doubt it's worth even trying

[16:56:01 CDT(-0500)] <pspaude1> So, it doesn't change the model object store until save correct?

[16:56:12 CDT(-0500)] <dmccallum54> that's the "save time binding" option, yeah

[16:58:19 CDT(-0500)] <dmccallum54> Ssp.controller.person.CaseloadAssignmentViewController has reams and reams (and reams) of code for doing stuff like this (late, manual form->model bindings

[16:58:35 CDT(-0500)] <dmccallum54> looks like the idiomatic Ext.js mechanism is Form.updateRecord()

[16:58:41 CDT(-0500)] <dmccallum54> which we use sparingly

[16:58:46 CDT(-0500)] <dmccallum54> in our great wisdom, no doubt

[17:00:09 CDT(-0500)] <pspaude1> Cool, I'll take a look. This seems like a good way to go, since I'm already having issues with parts not refreshing

[17:00:18 CDT(-0500)] <dmccallum54> k. thx

[17:00:22 CDT(-0500)] <pspaude1> I'll defer to the wisdom and not use it (smile)