Skip to content
This repository has been archived by the owner on Oct 19, 2021. It is now read-only.

[BUG] Users can't see "Total Matches" on Seasons #166

Closed
mrc4tt opened this issue Oct 27, 2019 · 7 comments
Closed

[BUG] Users can't see "Total Matches" on Seasons #166

mrc4tt opened this issue Oct 27, 2019 · 7 comments
Labels
bug Something isn't working

Comments

@mrc4tt
Copy link

mrc4tt commented Oct 27, 2019

Hi,

Some users can't see "Total Matches" on Seasons.

If I'm logged out of the get5page. I can see it, but others can't.

What can I do?
I can't find any errors.

Here; the user can't see "Total Matches"
screen1

User can see them if He clicking on Week 1
screen2

I did tell the user to try, logout. - He can't see the "Total Matches" (Where it should show: 12 - but it displays: 0)
screen3

Here, he did log on, but can't see it.
screen4

@mrc4tt mrc4tt added the bug Something isn't working label Oct 27, 2019
@PhlexPlexico PhlexPlexico changed the title [HELP] Users can't see "Total Matches" on Seasons [BUG] Users can't see "Total Matches" on Seasons Oct 27, 2019
@PhlexPlexico
Copy link
Owner

I believe it's a bug in the template. I need to look into it more, but from what I can see, the map function isn't working as intended.

Possible error line: here

In your get5 instance, would you mind putting this line right below line 46 in season.py

app.logger.info("Our total matches are: {}".format(seasoned_matches))

Restart your Apache instance, and then tell the user to navigate to the page again. And provide the output from the log. And also once when you check the page as well. I need to verify it's just in the template, and not the middleware itself.

Thanks!

@PhlexPlexico
Copy link
Owner

Wait, nevermind. I think I know where the problem is. Let me push out a quick fix.

PhlexPlexico added a commit that referenced this issue Oct 27, 2019
Was not including matches total count in the users seasons, which was leading to a count of zero.
@PhlexPlexico
Copy link
Owner

PhlexPlexico commented Oct 27, 2019

e82249b Should fix in development branch.

@mrc4tt
Copy link
Author

mrc4tt commented Oct 27, 2019

@PhlexPlexico danke <3

@PhlexPlexico
Copy link
Owner

Works now?

@mrc4tt
Copy link
Author

mrc4tt commented Oct 27, 2019

@PhlexPlexico Yes! ;)

@PhlexPlexico
Copy link
Owner

Perfect. Going to close this issue now since that seems to have done it. Thanks for the report! 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants