RoyaleAPI / cr-api-ux

User Experience and content issue tracker for RoyaleAPI website.
https://royaleapi.com
21 stars 16 forks source link

Info page donations #379

Open BlackKat101 opened 5 years ago

BlackKat101 commented 5 years ago

Describe the bug On the Clan Info page of RoyaleAPI, when you click on 7d, the donations are correct for that week. When you click on 14d or 28d it still only shows the donations for the current week and does not include the donations for the full period

To Reproduce Steps to reproduce the behavior:

  1. Go to 'https://royaleapi.com/clan/9JJQ0GVU'
  2. Click on 'Info'
  3. Look at donations
  4. Click on 14d
  5. Look at donations again, you will see they are the same as previously

Expected behavior I would expect that when you clicked on 14d, that it would give you a total of each clan member's current week donations plus their donations for the previous 7d period

Screenshots If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context Add any other context about the problem here.

smlbiobot commented 5 years ago

What happened: the clan history was out of service for a week, and so there was a time period where there were no data. That’s why when you click on 14d, it looks the same as 7d. This page will get a revamp soon and it should not matter after that.

BlackKat101 commented 5 years ago

Awesome, thanks for the info. I look forward to seeing the new revamped page once it is done!!

Kind regards Hannah

On Thu, 8 Nov 2018, 14:22 SML, notifications@github.com wrote:

What happened: the clan history was out of service for a week, and so there was a time period where there were no data. That’s why when you click on 14d, it looks the same as 7d. This page will get a revamp soon and it should not matter after that.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/RoyaleAPI/cr-api-ux/issues/379#issuecomment-436976403, or mute the thread https://github.com/notifications/unsubscribe-auth/AqusudDdTzq9FhH9H3vp6bbKmnYcWMxhks5utCIMgaJpZM4YQUDi .

6Sean commented 5 years ago

@BlackKat101 Is this issue completed/resolved?