/the-blue-alliance-android

New issue

Current Weak Empty Again #322

Closed
phil-lopreiato opened this Issue Feb 15, 2015 · 3 comments

Assignees

No one assigned

Labels

bug

Projects

None yet

Milestone

No milestone

2 participants

@phil-lopreiato @fangeugene
@phil-lopreiato
Member
phil-lopreiato commented Feb 15, 2015

Need to fix up week detection, it seems
@phil-lopreiato phil-lopreiato added the bug label Feb 15, 2015
@fangeugene
Member
fangeugene commented Feb 15, 2015

It was working yesterday and showed preseason.

On Sun, Feb 15, 2015, 11:46 AM Phil Lopreiato notifications@github.com
wrote:

Need to fix up week detection, it seems

—
Reply to this email directly or view it on GitHub
#322
.
@phil-lopreiato
Member
phil-lopreiato commented Feb 15, 2015

It's Sunday, so it's a new week, so I'm guessing that caused it to happen. I'm just going to get rid of the 'Current Week' label - I think an auto selected Week # will be fine. It was the different label that was causing the content to not be there (lack of key in the hashmap storing all events).
@phil-lopreiato
Member
phil-lopreiato commented Feb 15, 2015

device
@phil-lopreiato phil-lopreiato added a commit that closed this issue Feb 15, 2015
@phil-lopreiato phil-lopreiato Don't name an event tab 'Current Week' 
It messes with resolving the real content for that tab. It'll be okay to
leave all the labels as Week n, with the current one selected. Closes #322
 9537ab1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment