-
Notifications
You must be signed in to change notification settings - Fork 71
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Total Ratted - Dashboard Summary Stat #587
Conversation
src/Traits/Stats.php
Outdated
*/ | ||
public function getTotalCharacterRattingIsk(array $character_ids): ?float | ||
{ | ||
return CharacterWalletJournal::selectRaw('SUM(amount) as total_ratting_value') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Any way to avoid selectRaw()
here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
☕ yeah, just late and my copy-pasta game was weak.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated.
Maybe a change for the icon ? Anyway, big thanks 👍 What about coins ? |
Yeah that works, too. Done. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be sum('amount'), isn't it ?
Yeah I really need to go to bed, lol. |
No hurry 🙂 |
@leonjza any love for this one? :) |
Gonna merge that this weekend at last. It's EoY and business is even more crazy with all that COVID party. |
Adding a new Home statistic for total ratted stat.
Attempts to solve eveseat/seat#835
Fixed layout formatting for tablet/mobile