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

Th17 - New Th, Troop, Spell, Hero, & Equipment #248

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

MagicTheDev
Copy link
Collaborator

updated relevant static files & enums

@@ -262,7 +264,7 @@ def in_game_name(self) -> str:

SPELL_ORDER = ELIXIR_SPELL_ORDER + DARK_ELIXIR_SPELL_ORDER

HOME_BASE_HERO_ORDER = ["Barbarian King", "Archer Queen", "Grand Warden", "Royal Champion"]
HOME_BASE_HERO_ORDER = ["Barbarian King", "Archer Queen", "Grand Warden", "Royal Champion", "Minion Prince"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

I would vote to keep it the order of unlocking, matching the ingame order in the hero hut, so minion prince in the middle

"Magic Mirror"
"Magic Mirror",
"Henchmen Puppet",
"Dark Orb"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please add the lavaloon puppet

@@ -14,8 +14,7 @@
import zipfile
from collections import defaultdict

import lzma
import zstandard
Copy link
Collaborator

Choose a reason for hiding this comment

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

good catch with the duplicate imports

APK_URL = "https://d.apkpure.net/b/APK/com.supercell.clashofclans?version=latest"

def get_fingerprint():
import aiohttp
import asyncio
Copy link
Collaborator

Choose a reason for hiding this comment

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

can we move the imports all to the top? there is another one in decompress

@mathsman5133
Copy link
Owner

Happy to hear thoughts, but does anyone actually look through the json files? Ie. do we need them "beautified"?

I did a quick test and just about halved the buildings.json file by loading and re-dumping the contents into a single line json file.

If people actually look through them I don't hugely mind, but if we have the option it'd be nice to keep this static data small-ish for pypi and releases, and not having enormous git diffs.

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.

3 participants