AssistantNMS / AssistantNMS/App

🚀 Feature request: Use proper data for upgrades

Open
#264 2 comments 0 reactions 0 assignees View on GitHub
idea
Dominant language
Dart
Stars
50
Forks
3
PR merge metrics
No merged PRs in 30d

Description

### 🔖 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:
- Use available item data from the Module
- Add the stats from the Upgrade
- The final item page would be for the Module, but that is more easily understood than the Upgrade I guess.

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.
- Look through `NMS_REALITY_GCPRODUCTTABLE.EXML` for upgrades
- Take the following data from this file:
- `NameLower` ("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 details
- Search for `DeploysInto` property (`UP_LASERX`)
- Look for this ID in `NMS_REALITY_GCPROCEDURALTECHNOLOGYTABLE.EXML`
- Display upgrade stats from this file

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.
![grafik](https://github.com/user-attachments/assets/cc7ef60b-c636-40f8-b1f0-6c8352c1b200)

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?

- [X] I checked and couldn't find similar issue

### 🏢 Have you read the Code of Conduct?

- [X] I have read the [Code of Conduct](https://github.com/AssistantNMS/App/blob/main/.github/CODE_OF_CONDUCT.md)

### Are you willing to submit PR?

None

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.