Clicking on AMR Gear Ranking leads to a 404 #3279

Open
Detrel opened this Issue Dec 23, 2016 · 4 comments

Projects

None yet

4 participants

@Detrel
Detrel commented Dec 23, 2016

Please include a brief overview of the issue in the title above, as well as completing the following:
Clicking on the askmrrobot or wowhead link sends me to a 404 page. (I'm wondering if it's because my realm name has a ' character in it: Vek'nilash

Simulationcraft Version

710-03 but was present in previous version as well

If Simulationcraft is crashing, please include your dxdiag file - Instructions here - https://us.battle.net/support/en/article/Obtaining-System-Files

simc_gui.simc file or contents

This file is saved in the same folder as the program after each simulation. Make sure to run the simulation that is giving you issues, then include the file here by either uploading to github, copy/pasting the contents, or using http://pastebin.com/

Reproduction Steps:

Run a sim. Click the link. Opens default browser with a 404 on the AMR site.
Latest URL: http://www.askmrrobot.com/wow/optimize/us/veknilash/Detrel#spec=WarriorArms;weights=Strength:1.00,AttackPower:0.95,CriticalStrike:0.97,Haste:1.41,Mastery:1.28,Versatility:0.98,MainHandDps:3.58
Please include any other steps necessary to recreate the problem, or include logs/data to show what is occurring in game if there is an issue with how something is modeled.

@lgkern
Contributor
lgkern commented Jan 5, 2017

Can you give a link to a working URL from a character from Vek'nilash on AMR's website so we can know the correct formatting for it?

@Collisionc
Contributor

This on AMR's end, they've changed how that part of their site works and I doubt we'll be able to interface with it since they would rather you use their simulator. =\

@Collisionc Collisionc self-assigned this Jan 12, 2017
@Collisionc Collisionc changed the title from Clicking on Gear Ranking leads to a 404 to Clicking on AMR Gear Ranking leads to a 404 Jan 12, 2017
@Collisionc
Contributor

I removed the link from the report as it's useless.

@yellowsix

Yeah this is something that Theck added a while back I believe -- since updating for Legion that link format and related behavior isn't supported/relevant on AMR anymore, so it is safe to remove it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment