search

wowsims / exporter

WoWSims Exporter addon repo
MIT License
1 stars 6 forks source link

Website Broken #11

Closed Motowatch closed 4 months ago

Motowatch commented 4 months ago

Your website won't accept Import from the Classic Cata Pre-Patch Addon, and the website equipment manual settings are incorrect/old.

FelixPflaum commented 4 months ago

Hey, can you post your export here so I can check what's wrong?

Motowatch commented 4 months ago

On May 5, 2024, at 1:06 PM, FelixPflaum @.***> wrote:

I could but I doubt it is a problem with the text string coming from the game addon. Multiple users have reported the exact same issue, 1) You export gear string from addon. 2) You go to website. 3) You paste in import string but 4) the Import button does nothing when you click it.

Also it’s not just the import function. If I try to adjust my gear manually, your interface's item selections are wrong/old. The Chaotic Skyflare Diamond has the wrong activation data for example. It has been changed to 3 red gems.

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Hey, can you post your export here so I can check what's wrong?

— Reply to this email directly, view it on GitHub https://github.com/wowsims/exporter/issues/11#issuecomment-2094880008, or unsubscribe https://github.com/notifications/unsubscribe-auth/AODW3AOXAYRXM4TFJOYUDLLZAZRIXAVCNFSM6AAAAABHHZOKPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJUHA4DAMBQHA. You are receiving this because you authored the thread.

Motowatch commented 4 months ago

Here is the export string from the game addon:

{"talents":"20100000000000000000-02000000000000000000-23203213211113002301","glyphs":{"minor":[{"spellID":57958},{"spellID":89401},{"spellID":57947}],"prime":[{"spellID":56416},{"spellID":63220},{"spellID":54934}],"major":[{"spellID":54938},{"spellID":54924},{"spellID":54926}]},"class":"paladin","unit":"player","race":"Human","name":"Motowatch","gear":{"items":[{"id":51162,"gems":[41285,40111],"reforging":154,"enchant":3817},{"id":50452,"gems":[40111],"reforging":140},{"id":51160,"gems":[40111],"reforging":154,"enchant":3808},{"id":50653,"gems":[40111],"reforging":154,"enchant":1099},{"id":51164,"gems":[40111,40111],"reforging":154,"enchant":3832},{"id":50659,"gems":[40111,40111],"reforging":139,"enchant":3845},{"id":51163,"gems":[40111,40111],"reforging":140},{"id":50987,"gems":[40111,40111,40111],"reforging":140},{"id":51161,"gems":[40111,40111],"reforging":161,"enchant":3823},{"id":51915,"gems":[40111,40111],"reforging":147,"enchant":1597},{"id":50402,"gems":[40111],"reforging":147},{"id":50678,"gems":[40111],"reforging":147},{"id":47115,"gems":[]},{"id":50351,"gems":[],"reforging":140},{"id":49623,"gems":[40111,40111,40111],"reforging":154,"enchant":3789},null,{"id":50455,"gems":[40111],"reforging":154}]},"professions":[{"name":"Blacksmithing","level":450},{"name":"Engineering","level":450}],"level":80,"spec":"retribution","realm":"Pagle"}

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ On May 6, 2024, at 8:33 AM, Thomas Haller @.***> wrote:

I could but I doubt it is a problem with the text string coming from the game addon. Multiple users have reported the exact same issue, 1) You export gear string from addon. 2) You go to website. 3) You paste in import string but 4) the Import button does nothing when you click it.

Also it’s not just the import function. If I try to adjust my gear manually, your interface's item selections are wrong/old. The Chaotic Skyflare Diamond has the wrong activation data for example. It has been changed to 3 red gems.

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ On May 5, 2024, at 1:06 PM, FelixPflaum @.***> wrote: Hey, can you post your export here so I can check what's wrong?

— Reply to this email directly, view it on GitHub https://github.com/wowsims/exporter/issues/11#issuecomment-2094880008, or unsubscribe https://github.com/notifications/unsubscribe-auth/AODW3AOXAYRXM4TFJOYUDLLZAZRIXAVCNFSM6AAAAABHHZOKPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJUHA4DAMBQHA. You are receiving this because you authored the thread.

1337LutZ commented 4 months ago

Thanks @Motowatch, I've just imported it in a incognito window to make sure I don't have anything else wrong and it seems to work fine: (it shows the meta warning, and 5 unspent talent points since your export is lvl 80)

image

You are correct about the gems, however these are WOTLK gems and the Sim is for level 85, so updating outdated lvl 80 items hasn't been the top priority.

Could you let us know which browser you are using? Perhaps it's a browser issue.

Motowatch commented 4 months ago

??? Still broken, not working. You understand this is a Cataclysm pre-patch issue, yes?

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ On May 6, 2024, at 9:21 AM, Adrian Klingen @.***> wrote: Thanks @Motowatch https://github.com/Motowatch, I've just imported it in a incognito window to make sure I don't have anything else wrong and it seems to work fine: image.png (view on web) https://github.com/wowsims/exporter/assets/1216787/6b1e74f0-7612-4954-817e-a5ba08a75e9d — Reply to this email directly, view it on GitHub https://github.com/wowsims/exporter/issues/11#issuecomment-2096007617, or unsubscribe https://github.com/notifications/unsubscribe-auth/AODW3AJI3LQT6ESUCG4EYHLZA57VPAVCNFSM6AAAAABHHZOKPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJWGAYDONRRG4. You are receiving this because you were mentioned.

1337LutZ commented 4 months ago

??? Still broken, not working. You understand this is a Cataclysm pre-patch issue, yes? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ On May 6, 2024, at 9:21 AM, Adrian Klingen @.***> wrote: Thanks @Motowatch https://github.com/Motowatch, I've just imported it in a incognito window to make sure I don't have anything else wrong and it seems to work fine: image.png (view on web) https://github.com/wowsims/exporter/assets/1216787/6b1e74f0-7612-4954-817e-a5ba08a75e9d — Reply to this email directly, view it on GitHub <#11 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AODW3AJI3LQT6ESUCG4EYHLZA57VPAVCNFSM6AAAAABHHZOKPCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOJWGAYDONRRG4. You are receiving this because you were mentioned.

Have you tried opening the link in a private/incognito window? As my testing concluded the import link seems to work fine.

Also, make sure to use the Cata link: https://wowsims.github.io/cata/ WOTLK sim is not compatible with the Cata pre-patch

kayla-glick commented 4 months ago

Hey @Motowatch 👋 I think there was some confusion in the thread. It seems like you're trying to use the Wrath of the Lich King site to run sims for your character in the pre-patch, is that correct?

The problem is that there's so much that's drastically different from Wrath to the Cata pre-patch like talents, glyphs, tons of gear changes, and plenty of core mechanic changes. While you could import to the Cataclysm site, everything is configured for level 85 and many specs are still being worked on before they go to Alpha for testing. Given that the pre-patch is only about 2 weeks, it just isn't worth the dev effort to update the site to support it. Sorry again for the inconvenience but I hope you'll find the Cataclysm site useful once we're at 85 🙏