Skip to content

"Race advancements" and "Global statistics" are functional#564

Open
Youraveragebub wants to merge 9 commits into
TeamGalacticraft:mainfrom
Youraveragebub:feature/space-race-stats
Open

"Race advancements" and "Global statistics" are functional#564
Youraveragebub wants to merge 9 commits into
TeamGalacticraft:mainfrom
Youraveragebub:feature/space-race-stats

Conversation

@Youraveragebub
Copy link
Copy Markdown

You can even hide yourself from these statistics that are visible server wide. You can compete in teams to see who has the most advancements completed (although atm there is no reward for it, just a flex). Global statistics are server wide statistics similar to vanilla but only for galacticraft stats. It is a leaderboard type system too (which you can hide yourself from) and the "server" tab shows all the advancements completed by the server as a whole. It shows you the first 3 players to complete an advancement.

You can even hide yourself from these statistics that are visible server wide. You can compete in teams to see who has the most advancements completed (although atm there is no reward for it, just a flex).
Global statistics are server wide statistics similar to vanilla but only for galacticraft stats. It is a leaderboard type system too (which you can hide yourself from) and the "server" tab shows all the advancements completed by the server as a whole. It shows you the first 3 players to complete an advancement.
@Youraveragebub
Copy link
Copy Markdown
Author

I haven't tested the "teams" tab in global stats btw so let me know if I gotta do something about it because idk how to test it now.

Copy link
Copy Markdown
Member

@Roelymole Roelymole left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is looking good, but I am concerned that the information it is showing needs to be more focused on the space race.

Some refactoring of the SpaceRaceScreen.java file is in order before I can give more specific feedback about most of that.

Comment thread src/main/java/dev/galacticraft/impl/network/GCApiPackets.java Outdated
Comment thread src/main/java/dev/galacticraft/impl/network/GCApiServerPacketReceivers.java Outdated
Comment thread src/main/java/dev/galacticraft/mod/client/gui/screen/ingame/SpaceRaceScreen.java Outdated
Comment thread src/main/java/dev/galacticraft/mod/content/GCStats.java
Comment thread src/main/java/dev/galacticraft/mod/client/gui/screen/ingame/SpaceRaceScreen.java Outdated
Comment thread src/main/java/dev/galacticraft/mod/client/gui/screen/ingame/SpaceRaceScreen.java Outdated
Comment thread src/main/java/dev/galacticraft/mod/data/GCTranslationProvider.java
Comment thread src/main/java/dev/galacticraft/mod/client/gui/screen/ingame/SpaceRaceScreen.java Outdated
I ordered as many of those imports as I could. (I hope there aren't any that slipped through and I doubt there will be)
I added a config file for the categories of the Server stats (formerly global stats). There is NO defaults, however everything is in # basically quick suggestions if you want to add or remove anything.
Translations should (hopefully) be in place.
Buttons scaling with GUI is now implemented.
Moved the text down by one pixel (line 55 SpaceRaceButton.java)
Last but not least I added a bunch of files with decent names so you can easily look through it.
Also wildcard imports and my intellij don't work together for SOME reason.
src\main\java\dev\galacticraft\impl\network\GCApiServerPacketReceivers.java
(again because I can't read)
It now reads from codec backed JSON files for the default values.
Also have split up the spaceracescreen.java into multiple files.
Maybe done?
Race Adv and server stats screens both contain code exactly for those tabs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants