DestinyItemManager / DIM

Destiny Item Manager
https://destinyitemmanager.com
MIT License
2.07k stars 644 forks source link

Armory: Properly handle multiple intrinsic options #9139

Open bhollis opened 1 year ago

bhollis commented 1 year ago

Old screenshot, no longer happens:

image

Improvements to be made for enhanced intrinsics in Armory:

  1. [ ] Fix error/warning as described below
  2. [ ] Allow previewing/selecting from all enhanced intrinsics
robojumper commented 10 months ago

I think this is improved now that items (including fake items like the Armory item) are forced to one intrinsic at most. It's still not great because you can't preview any others.

There's also an error/warning when you open the armory from a crafted weapon with an enhanced intrinsic

> [applySocketOverrides] Tried to override to a socket that didn't exist in the options

because only the crafting template has these available as options, not the actual item. You also can't preview enhanced intrinsics in these weapons.