Open Lenni009 opened 1 month ago
I expect this to also mess with URLs, so the API needs to be adjusted accordingly.
To add to this: Sentinel Exosuit Fragment and Sentinel Weapons Shard are duplicated in the app.
One of them has dev details, the other does not.
Maybe it would be good to keep them as separate items, with their respective name, icon, description, etc., but link to each other. So in the case of the Sentinel Exosuit Fragment, the module would have a link "Deploys into: Forbidden Exosuit Module", with a chip styling similar to how cooking and refining products are styled. The Forbidden Exosuit Module would have a link "Deployed from: Sentinel Exosuit Fragment".
As for the name of the Forbidden Exosuit Module (and other upgrades), use the subtitle. Another example would be "A-Class %NAME% Upgrade" (taken from the Subtitle in the PROCEDURALTECHNOLOGYTABLE), which is deployed from "Neutron Cannon Module" (taken from the Name in the PRODUCTTABLE).
That way you would still have two items, both with proper data, and linked to each other. Then there should hopefully be no more confusion and no lost data.
🔖 Describe the solution you'd like
Upgrades are currently a bit confusing. This probably stems from the fact they they can be in "Module" or "Upgrade" form. The "Modules" have the item data in the game files, while the "Upgrades" only have template data. This is already implemented with the Neutron Cannon upgrades, but not with other upgrades, for example Mining Beam.
My solution would be to combine these two:
This is primarily intended for the web app for now, but transferring this to mobile should be trivial, the generated data source is the same after all. I'm using the X-Class Mining Beam Upgrade as an example here, your script would hopefully handle everything on its own.
NMS_REALITY_GCPRODUCTTABLE.EXML
for upgradesNameLower
("Suspicious Mining Beam Module")Icon
(TEXTURES/UI/FRONTEND/ICONS/U4PRODUCTS/PROCTECH/PROCTECH.X.LASER.DDS
)Subtitle
("Banned Mining Beam Upgrade")Description
("A black-market modification for the Mining Beam. Use FE_ALT1 to begin upgrade installation process. The module is flexible, and exact upgrade statistics are unknown until installation is complete. Potential upgrades include: power output, heat toleration and fuel efficiency.")Category
,Type
, etc. for the dev detailsDeploysInto
property (UP_LASERX
)NMS_REALITY_GCPROCEDURALTECHNOLOGYTABLE.EXML
Here's a mockup for how it would look on the web Not too crazy, it's basically just the name, subtitle and description changed.
Also note that the item now has dev details from the module item.
Implementing this would close #48 and #214.
🩹 Describe alternatives you've considered
No response
👀 Have you checked if this bug report has been created already?
🏢 Have you read the Code of Conduct?
Are you willing to submit PR?
None