-
-
Notifications
You must be signed in to change notification settings - Fork 80
Gym defenders implementation #1112
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
Gym defenders implementation #1112
Conversation
|
By the way the threshold numbers are definitely not one/two thirds. I think it's 73% or something similar. |
|
Yeah it's ~47% and then ~73% as the threshold, or more precisely, 7/15 and then 11/15. The minimum is 20% or 1/5. See also: https://pogo.gamepress.gg/gym-motivation-what-you-should-know |
|
I made another small change (dropped defender info from extra text), including hearth thing @Mygod mentioned: sanderDijkxhoorn#3 |
feat(client): drop defenders from extra info, fix defenders hearth
|
I merged it, could someone try to run this PR and check if all functions correctly? |
|
@sanderDijkxhoorn I'm running it on prod at top of #1113. Everything looks good related to defenders. |
|
By the way, do we want to display some of these other fields? |
Waste of space... Unless we would toggle ON CLICK between
|
|
I guess if people are okay with it, I will merge and implement the changes myself. |
|
Looks fucking good. UI done by vibe coding. @sanderDijkxhoorn @lenisko Merge? |
Those icons look a little odd, replace it with berry and such? Why does the top one start with 1d xxxx |
|
Minor complaints about UI will be only accepted in the form of commits or PRs. :) |
|
🎉 This PR is included in version 1.36.0-develop.19 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
|
@Mygod this looks like a 💩 We just literary been fighting for any vertical space, and you just added a bunch of useless stuff on the same page. That's why I proposed additional data on toggle/click. |
|
@lenisko I just care about functionality. If you want to make it look less 💩, PR welcome. :) |
|
🎉 This PR is included in version 1.36.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |



Implementing #1101
Latest design:
I made a little oopsy with the last PR... here is a cleaner version of the PR, no GitHub workflow commits.