Skip to content
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

updatelivestats #3687

Closed
TSEFF opened this issue Aug 12, 2016 · 5 comments
Closed

updatelivestats #3687

TSEFF opened this issue Aug 12, 2016 · 5 comments

Comments

@TSEFF
Copy link

TSEFF commented Aug 12, 2016

I found the level_completion_percentage was not correct...

So I changed it to show the ETA to next level instead.. :P

if experience_per_hour != 0:
eta_next_lvl = float(require_xp/metrics.xp_per_hour())
else:
eta_next_lvl = 0

'level_completion': '{:,} XP ({:,.2f})hr'.format(require_xp,
eta_next_lvl),

@k4n30
Copy link
Contributor

k4n30 commented Aug 12, 2016

@TSEFF Why not create a pull request especially if you have already done the work

@TSEFF
Copy link
Author

TSEFF commented Aug 12, 2016

i am new to github... :P

@k4n30
Copy link
Contributor

k4n30 commented Aug 12, 2016

No worries, I've added the help wanted label so that someone can help you out :) Thanks for raising the issue

@k4n30
Copy link
Contributor

k4n30 commented Aug 16, 2016

@TSEFF has this been resolved?

@TSEFF
Copy link
Author

TSEFF commented Aug 17, 2016

@k4n30 yes... thank you

@k4n30 k4n30 closed this as completed Aug 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants