PathOfBuildingCommunity / PathOfBuildingCommunity/PathOfBuilding
Foulborn variants for jewels which change radius in item db
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Lua
- Sterne
- 5.4k
- Forks
- 2.4k
- Ø Merge
- 1 T. 12 Std.
- Gemergte PRs (30 T.)
- 26
Beschreibung
Check version
- I'm running the latest version of Path of Building and I've verified this by checking the changelog
Check for duplicates
- I've checked for duplicate open and closed issues by using the search function of the issue tracker
What platform are you running Path of Building on?
Windows
How is Path of Building expected to behave?
In #9946 I noticed that adding support for this wasn't possible. It seems that the item parsing doesn't really consider radius a mod, which means that it didn't seem to be possible to add variants which change the radius. I wasn't able to support this item in that PR because it only processes mods instead of, say, the raw item text.
How does Path of Building behave?
unique db only has regular variant
How to reproduce the issue
No response
PoB for PoE1 build code
Screenshots
No response
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, den in diesem Issue beschriebenen Pfad zum Parsen von Items und die Behandlung der Datenbank für einzigartige Items nachzuverfolgen, mit Fokus darauf, wie Mods und roher Item-Text verarbeitet werden. Fertig ist es, wenn die Datenbank für einzigartige Items Varianten von Foulborn-Juwelen enthält, deren Radiusänderungen korrekt dargestellt werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- lua
- Bereich
- databases
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Ruhig
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100