search

adamblan / whiteboard-issues

A github to publicly track Whiteboard issues
0 stars 0 forks source link

All office hours don't show up in the weekly schedule #35

Open robsimmons opened 11 years ago

robsimmons commented 11 years ago

Currently only the "special" ones for us do, and the recurring ones do not... which is kind of the opposite of what I would have expected.

adamblan commented 11 years ago

Perhaps the start dates are off by a week? Also, the all office hours only shows the upcoming week. Maybe that's the problem?

I'm confused, because its working fine for 151.

On Sep 18, 2013, at 3:37 PM, "Robert J. Simmons" notifications@github.com wrote:

Currently only the "special" ones for us do, and the recurring ones do not... which is kind of the opposite of what I would have expected.

— Reply to this email directly or view it on GitHub.

robsimmons commented 11 years ago

Doesn't make sense for start dates to be off by a week - start dates have gotta be inclusive, they show up on the "upcoming week" sign-up schedule.

On Wed, Sep 18, 2013 at 3:48 PM, Adam Blank notifications@github.comwrote:

Perhaps the start dates are off by a week? Also, the all office hours only shows the upcoming week. Maybe that's the problem?

I'm confused, because its working fine for 151.

On Sep 18, 2013, at 3:37 PM, "Robert J. Simmons" notifications@github.com wrote:

Currently only the "special" ones for us do, and the recurring ones do not... which is kind of the opposite of what I would have expected.

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHubhttps://github.com/adamblan/whiteboard-issues/issues/35#issuecomment-24693770 .

Robert J. Simmons simrob.com rjsimmon@cs.cmu.edu robsimmons@gmail.com Cell: 404-273-6890

adamblan commented 11 years ago

Ahh, I just looked. The "Weekly Office Hours" shows the "next week"---starting at Sunday.

So, what's going on is that it will show the ones starting this Sunday as of this Sunday.

I guess this isn't particularly intuitive behavior, but what exactly were you expecting? (I'm asking, because it's not clear to me what the right alternative might be.)

robsimmons commented 11 years ago

Fair enough: I guess I wanted to post weekly office hours starting Sunday and announce that I was doing so on Friday. The alternative would be a sliding window, but we've already got that view. It's okay, I guess, just a little unexpected. Perhaps "office hours for the week of X to Y" would be a more intuitive label.

On Wed, Sep 18, 2013 at 4:14 PM, Adam Blank notifications@github.comwrote:

Ahh, I just looked. The "Weekly Office Hours" shows the "next week"---starting at Sunday.

So, what's going on is that it will show the ones starting this Sunday as of this Sunday.

I guess this isn't particularly intuitive behavior, but what exactly were you expecting? (I'm asking, because it's not clear to me what the right alternative might be.)

— Reply to this email directly or view it on GitHubhttps://github.com/adamblan/whiteboard-issues/issues/35#issuecomment-24695509 .

Robert J. Simmons simrob.com rjsimmon@cs.cmu.edu robsimmons@gmail.com Cell: 404-273-6890

adamblan commented 11 years ago

Sounds reasonable. I will fix this in a bit (today).

adamblan commented 11 years ago

I'm not thrilled with how the header I made looks (not content-wise--visual-wise), but it's good enough for now.

robsimmons commented 11 years ago

While we love inclusive-exclusive bounds in computer science, the dates on that header should probably be inclusive-inclusive, yes? Next Sunday's office hours (on the 22nd) aren't on the calendar.

robsimmons commented 11 years ago

(To be clear: I don't think next Sunday's office hours should be on the calendar I think the dates should go through the 21st.)