demilich1 / metastone

Hearthstone simulator written in Java with full GUI support
GNU General Public License v2.0
132 stars 80 forks source link

Journey to Un'Goro Update #333

Open webadict opened 7 years ago

webadict commented 7 years ago

Oh ho ho! Guess who's here with an update on Journey to Un'Goro?!

So, there are a BUNCH of changes that need to be made for just the few cards I've seen, and I plan to go over a vast majority of them right now. Bare in mind that this will be a list of all changes that are planned for v1.4.0, and if there's an issue or a change you'd like to see, you're free to leave a comment here.

Biggest Changes:

Big Changes:

Minor Changes:

That's all I have for now, but this will be modified as more information appears. I'll push some updates as soon as I get Permanents and Quests working, but it is very, very, VERY likely that they will need to be tested to ensure that nothing breaks.

mayuso commented 7 years ago

MIght be interesting to show the progress as a checklist:

You are doing an incredible job and never stop to amaze us, we would love to keep up with your progress and help as much as possible, but it's harder than you think xD I seriously believe a checklist will help both us (those that want to help) and other users that want to know how the changes are going.

This is just my opinion, feel free to ignore it if it makes no sense.

webadict commented 7 years ago

Yeah, that might be a good idea. I'll throw it up soon.

webadict commented 7 years ago

It took forever, but Quests... Are FINALLY working, though I think they might need to be implemented a bit better.

DominusMaximus commented 7 years ago

I received this error when I tried to run your latest build:

Metastone\game\src\main\java\net\demilich\metastone\game\spells\trigger\secrets\Secret.java:27: error: incompatible types: net.demilich.metastone.game.spells.trigger.secrets.Secret cannot be converted to net.demilich.metastone.game.spells.trigger.types.Secret event.getGameContext().getLogic().secretTriggered(owner, this);

webadict commented 7 years ago

Yeah, I noticed that while uploading. I moved the secrets to their own folder, but it didn't update. I'll upload a copy with working Quests today.

webadict commented 7 years ago

See if the new one works. It looks like it should be fixed.

DominusMaximus commented 7 years ago

It's running now.

To see the new cards I had to update the CardSet.java file.

Other than that an error pops up for The Marsh Queen:

22:25:35 ERROR - java.lang.IllegalArgumentException: No enum constant net.demilich.metastone.game.spells.desc.filter.Operation.EQUALS

I'll be testing out the new cards in the coming days :-)

webadict commented 7 years ago

Yeah, that's my most up to date copy, so enjoy it. I'm pushing Permanents to @demilich1, so that he doesn't get bombarded by 20 different changes at once.

@demilich1: I never got around to making those test cases... My bad. It took like 45 minutes to get these changes separated from the rest.

DominusMaximus commented 7 years ago

Should I post any issues to a new thread, or to this one?

webadict commented 7 years ago

Either this thread or post them in my local branch's issue thread.

DominusMaximus commented 7 years ago

I can't reach the issues page of your branch. Does this apply to you?

http://stackoverflow.com/questions/16406180/is-there-a-way-to-add-issues-to-a-github-forked-repo-without-modifying-the-orig

1.) The Charge attribute isn't working.

2.) Errors for Living Mana, Ozruk, Molten Blade and Shifter Zerus popped up upon running the latest build. I attached a file with them.

errors.txt

webadict commented 7 years ago

lol, yeah, that was my issue.

webadict commented 7 years ago

Alright, not sure why ValueProvider isn't working for you, but I fixed a lot of issues. Charge is fixed, Secrets are fixed, and a few other errors are fixed.

webadict commented 7 years ago

Alright, next push will be Quests, later tonight.

webadict commented 7 years ago

Quest PR is now in review. I'll start pushing cards in the next PR, along with a few bug fixes.

webadict commented 7 years ago

Added Adapt! It's currently in my branch. I'll push something eventually.

Added GROUP cards, so that you can literally group specific effects together. Right now, each "card" has to be defined, but eventually, this will be expanded (Such as Spare Parts.)

webadict commented 7 years ago

As far as I can tell, I'll probably have all the cards done either this weekend or next weekend, but I won't have time to work on them until the weekend.

webadict commented 7 years ago

Added around 1/3 of the remaining cards. Will add more tomorrow.

unobserver-effect commented 7 years ago

When metastone 4.0 comes out, how can I keep modified/custom cards? And to install it, should I just run the installer again or do something ealse? (I alredy have the 3.0)

Watermelon86 commented 7 years ago

@y0urMom I highly recommend getting an account on Pastebin.com You can store all your custom cards there and download them all whenever you more to a new computer or if you accidently delete or mess up one.

TarCreeper commented 7 years ago

@webadict Congratulations for the work, but when the 1.4.0 will exit? Thanks :)