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
Rematch 5 Support #45
Comments
Relevant information for us from Gello: https://www.warcraftpets.com/community/forum/viewtopic.php?p=149251&sid=87e17bc5c4b93d1dfbc356c918f00e7d#p149251 They really don't want us to store script information in the team data, but are offering to do API for extending Rematch. I didn't have time to look into it since sadly. |
So, I've gotten a working edit. It is not backwards compatible, at all (that can be done, but will still leave the file even more messy than it already is) One major issue though... In Rematch4 teams are saved with a target ID (number) or a team name (string). As such PBS does the same. On the initial load of Rematch5 those teams are converted to a new format which uses a a unique id (string). It should be possible to convert those over, and have them matchup. Unless if the user been using Rematch5 without PBS, and have made changes to their teams, like myself. |
I can't wait to try it out. I have and will never pet battle without this mod. Thanks for all the hard work and dedication |
Does this mean you have a version that works with Rematch 5? |
Technically yes it does work with with Rematch5, but there are some carveouts with it, mainly around the script database migration.
|
If I backed up my saved variables and then basically reloaded all the teams does that sound like a good way to approach it? Also - it that patched version linked here? Having a crush week at work and a bit brain broken - so being really basic about my assumptions. |
You have to change the code yourself from his changes to the file or I am GitHub dumb. They work great though. Had to redo the changes when the mod updated last week. |
The initial load needs to happen either at the same time, or after Rematch5.
You can find my branch Here. Download the Github Zip. Looks like you can copy the contents of the folder that contains the |
Yeah after that version, I did rebase my branch, so its up to date. |
after today's most recent Rematch beta I get the following errors: 1x Interface\FrameXML\Bindings.xml:1 tdBattlePetScript/Rematch/Rematch.xml:1 tdBattlePetScript/Rematch/Rematch.xml(17): error: not well-formed (invalid token) 1x tdBattlePetScript/Rematch/Rematch.xml:1 tdBattlePetScript/Rematch/Rematch.xml(17): error: not well-formed (invalid token) Line 17 appears to be |
That's really weird. Not happening for me. Have tried restarting the client? My edit didn't do any change to the |
I'll try restarting in a sec - I disables PBS in the addon list before loading in the first time with the new version of rematch, did a rematch reset everything, turned PBS back on - got that error I have tried reloading a few times but not restarting the client |
Uh, if you did a Either copy in your backup, or if your comfortable editing the file: delete the |
I have both restarted the client and deleting the tdBattlePetScript.lua from both the account and character level (I had a back up) and I am still getting the same error I'm considering clearing out my rematch saved variables too to see what happens |
With Rematch beta-15, guessing and hoping for matching teams is no longer needed. My branch has been updated to use that. It should* now reset the scripts back to the Rematch4 backup, if you call |
Please let me know if we need anything asked/changed on the Rematch end to facilitate this. |
Since this will be the new version can we get an update that these work together again. Without scripts is like life without coffee
The text was updated successfully, but these errors were encountered: