arghblargh / feh-inheritance-tool Goto Github PK

Just wanted to let you know it says Death Knight learns close guard 3 at 5 star, but he has it at 4 star!
Also, I was just wondering if you planned on adding Cormag? :^)

Yo, I don't know how to JS but I can provide (at least) french translation for the skills if you want :)

I was thinking about adding some jsons files for the different languages.

Minor thing.

Flier Shigure grants ward fliers at 4 stars, not 5,

Whenever I try to open a wiki build for Brave Lucina the screen gets blank
Thanks in advance

Chrom with the -Res IV has a Resistance stat of 13, not 14 like your app suggests.
http://i.imgur.com/h2ft51C.png

All four split (A) skills featuring HP don't reflect their changes in the stat bracket

Hello.

Reporting a thing from the Feh skill inheritance tool.

Galeforce seems to not be showing on the list of specials for some characters, Velouria and Keaton as examples, female Kana too.

Equiping Fury on at least Finn and Ryoma is cause the whole window to go blank

It reads "grants +2 to Atk/Spd/Def/Res" when it grants +4 to Atk/Spd/Def/Res

Zephiel should appear below Zelgius, yet does not seem to be on the list at all.

In the FE:H inheritance for skill slot B, it does not show kana being able to get Wrath when she indeed can https://puu.sh/BZbkT/8431920a5b.png
https://cdn.discordapp.com/attachments/204852088545607682/510598688356433941/Screenshot_20181109-163632.png

https://imgur.com/a/cyyg7

1: Skill called Affinity Breaker in the B Slot doesn't exist

2: Special Fighter Available on non-armour characters

3: Sothe missing 4-star lv.40 and 4-star lv.1

4: Blizzard inheritable, it's Gunnthra's Prf

5: All swords, even Prfs, inheritable by Athena

Your tool is saying it's inherited by 5* Ogma, even though it can be inherited by the 4* version.
Great work and thank you very much! 👍

Refining Staves doesn't appear to add to the SP cost of a build. Tested Elise with Grav/Trilemma, and Lucius with Pain/Candlelight

Atk Ploy shows requiring 3870 SP, like so

https://gyazo.com/b59f698b7fcdd4c55172a853980a1ffe

Should be 630.

Flame Siegmund has a refinement option for both an "x" skill that breaks the page, and a/s/d/r which don't exist.

For some units, skills they can't inherit aren't listed, like offensive specials for healers for example. Others are listed even if they're inherit restricted. Here's a list of errors I've noticed, things inheritable when they shouldn't be:

Breath(Fierce, Steady, Warding) inheritable by all Cavalry, Fliers, Ranged Units
Sturdy Impact inheritable by Armours and Cavalry
Steady Stance 4 inheritable by Healers(3 can be given to healers, 4 cannot)
Null C-Disrupt and Null-Follow Up inheritable by Armours, Cavalry, and Fliers
Dull Close inheritable by Ranged Units
Smoke Skills inheritable by Healers

I've noticed some skills, like Blarserpent, have their Gronn and Rauor equivalents already added, even though we don't have access to them yet, but others like Joint Hone Spd don't have their equivalents. Not sure if intentional or not, just figured I'd mention it

Selecting Lucina's "x" Refinement makes the page go blank.

When I load up certain heroes, such as Alphonse or Alm, I will get an error that looks something like:

Uncaught (in promise) TypeError: Cannot read property '1' of null
    at Object.<anonymous> (helper.js:142)
    at <anonymous>

I've seen it on line 142 and 155 so far, but there may be others. This has the unfortunate side effect of killing the JS when it's trying to grab builds from the wiki, so the build select dropdown isn't populating for any hero that triggers this error.

Title

So if you change off of it, you can't change back.

Wo Gun+ isn't available as a inheritable Axe from Libra, and Walhart is listed as coming with Panic Ploy in his C slot, but he doesn't. Only checked Wo Gun availability on Groom Marth.

Missing the guard effect that the 4th tier skill has. Warding Stance 4 will have it too.

New Year Azura Hagoita+ is a melee weapon with possible refinement for A,S,D,R as most simple melee.

As mentioned above, the HP seal and skill is adding 7 HP when it should only be adding 5.

Safeguard+ doesn't have access to its refines in the Inheritance Tool for some reason.
Refine options are, ofc
Atk (HP+5, Atk+2)
Spd (HP+5, Spd+3)
Def (HP+5, Def+4)
Res (HP+5, Res+4)
(info taken from https://fireemblem.gamepress.gg/weapon-skills/safeguard)

The Skill refinement for Mystletainn doesn't reflect the Fury 3 effect on Eldigan's stats

If I select Lachesis the screen goes blank.

Not sure if this is due to Flora or the build itself but trying to load the (only) build "Ice Ice Baby" adds all the build's skills except its A skill. The skill can be manually added, but isn't automatically added when the build is loaded.

I've tried a few other units at random and they seem to work fine. Flora only has one curated build though so I can't check if it's due to Flora herself or due to the build (maybe being formatted incorrectly or something).

I've noticed on both Seth and normal Eirika, when calculating at +10, putting a Boon on will decrease other stats

Do you plan to add in dragonflowers to the tool? If you need a list of who can get how many, I could put one together

Might: 14. At start of turn, if unit's HP ≤ 75% and unit's attack triggers Special, grants Special cooldown count-1, and deals +1- damage when Special triggers. If unit initiates combat, grants Spd+6 during combat.

The typo is +1- instead of +10

When upgrading a Falchion and not selection the "X" upgrade, the description says:
Effective against dragon foes. At the start of every third turn, restores 10 HP.

It should say:
Effective against dragon foes. At the start of every second turn, restores 10 HP.