Versions Compared

Key

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

...

[12:47:40 CDT(-0500)] <b-sure> ok thanks drew_wills. I'm gonna poke around in the source as well. I'm thinkning if we can't get past this particular issue, we'll need to start looking into loading the default layout for all of our users.

[12:56:52 CDT(-0500)] <drew_wills> b-sure it looks like folders and channels are mixed in one of the <dlm:positionSet> elements... that looks strange to me

[12:57:15 CDT(-0500)] <drew_wills> how many users are affected?

[12:58:31 CDT(-0500)] <b-sure> hi. I"m not sure how many users are affected. by chance the user is in my work group.

[12:59:30 CDT(-0500)] <drew_wills> i have to hop on a call... bbl

[12:59:56 CDT(-0500)] <b-sure> is the mixed folders and channels for both of the pasted layouts or just the second one? reason I ask is because there are only about 40 or so out of about 12000 imports that errored

[13:00:31 CDT(-0500)] <b-sure> and the 2nd layout of the paste is the representative of that group

[14:07:03 CDT(-0500)] <EricDalquist> athena: you think I'm safe adding start and end datetimes to Term in the courses portlet?

[14:07:56 CDT(-0500)] <athena> yes

[14:08:15 CDT(-0500)] <EricDalquist> should I then remove year as well?

[14:10:25 CDT(-0500)] <athena> no

[14:10:45 CDT(-0500)] <athena> i think probably we'll need a flexible model that can accomodate a bunch of different styles of representing terms

[14:11:01 CDT(-0500)] <EricDalquist> ok

[14:11:02 CDT(-0500)] <athena> so we could have a general Term object that has a bunch of different fields, some of which might be null

[14:11:07 CDT(-0500)] <athena> or we could have different implemetnations

[14:11:42 CDT(-0500)] <athena> don't know if we'll need some sort of TermId interface for querying a term in the long term

[14:12:06 CDT(-0500)] <athena> for now we can probably just use a string key, and a school could create/parse combinations of year + season if they need to

[14:14:01 CDT(-0500)] <EricDalquist> yeah

[14:44:35 CDT(-0500)] <athena> what does this error message mean? "Your merge target has changed since viewing this pull request - try the merge again"

[14:45:03 CDT(-0500)] <EricDalquist> from github?

[14:45:13 CDT(-0500)] <athena> yeah, while trying to merge a pull request

[14:45:31 CDT(-0500)] <EricDalquist> perhaps there was a commit made to the target branch between when you opened the pull request page and when you clicked merge?

[14:46:05 CDT(-0500)] <athena> doesn't seem to be

[14:46:30 CDT(-0500)] <athena> maybe try a force-refresh

[14:46:35 CDT(-0500)] <athena> wasn't sure if you'd seen that before

[14:46:55 CDT(-0500)] <athena> huh, now it's working

[14:47:00 CDT(-0500)] <athena> wonder if something was goign on the server

[14:47:04 CDT(-0500)] <athena> was being really slow, too

[14:47:09 CDT(-0500)] <athena> now back to it's happy speedy self

[14:47:20 CDT(-0500)] <EricDalquist> weird