corowne / lorekeeper

A dA ARPG masterlist framework
MIT License
113 stars 125 forks source link

Making items use actual Rarities rather than arbitrary numbering #1113

Closed ScuffedNewt closed 4 days ago

ScuffedNewt commented 1 week ago

as the title states

ScuffedNewt commented 1 week ago

examples can be seen on item variants and a few other extensions that benefit from the use of rarities, plus I think logically it makes most sense for rarities to be consistent across the board

itinerare commented 1 week ago

On the other hand, sites may not use the same rarity systems between characters/traits and items-- so any solution for this needs to accommodate that possibility as well as allow conditional loot tables to work (i.e. still correlate rarities to numeric values).

ScuffedNewt commented 1 week ago

I think this should be solved on the rarity side, though, by making subcategories or divisions rather than on the item side. It is disruptive to the flow of how other things work especially considering most extensions use rarities as-is rather than creating a new system which ideally is how it should be

itinerare commented 1 week ago

Oh, I'm not arguing for any particular resolution-- I'm just saying that the kind of functionality that's presently there needs to be maintained one way or another. Making the rarity system more robust feels like a reasonable way about it, though.

ScuffedNewt commented 1 week ago

do you prefer if the rarity subcategories go in a new PR or as a separate issue to be discussed on how best to approach it?

itinerare commented 1 week ago

Just directly to new PR might be best, unless you have specific questions regarding the implementation?

ScuffedNewt commented 1 week ago

not really honestly, perhaps the only thing would be having varying searches based on the categorisations I'll mull it over since I won't get to it for a few hours for sure anyway

itinerare commented 1 week ago

Yeah, very fair. Sounds good, anyhow