Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow installing and removing bionics through debug menu #2507

Merged
merged 2 commits into from
Mar 30, 2023

Conversation

olanti-p
Copy link
Contributor

Summary

SUMMARY: Interface "Allow installing and removing bionics through debug menu"

Purpose of change

We already have 3 debug mutations that help with debugging bionics, but they have multiple shortcomings:

  1. Are clunky to use
  2. Don't allow removing bionics
  3. Don't work on NPCs

Describe the solution

Add new debug menu entry in Character section that allows installing and removing bionics, as well as editing power level and power cap.
For convenience, power manipulation functions can prompt in both joules and kilojoules.
Delete the clunkier debug mutations.

Describe alternatives you've considered

Leave debug mutations in - someone's bound to try to use them by mistake, and it's not really a mutation you're supposed to have, so I think it's fine to delete them outright. Test saves with that mutation will show a debugmsg on first load, but then they fix themselves.

@github-actions github-actions bot added JSON related to game datas in JSON format. src changes related to source code. labels Mar 30, 2023
@olanti-p olanti-p merged commit 83a8b76 into cataclysmbnteam:upload Mar 30, 2023
@olanti-p olanti-p deleted the debug-bionic-ui branch March 30, 2023 21:56
scarf005 pushed a commit to scarf005/Cataclysm-BN that referenced this pull request Mar 31, 2023
…team#2507)

* Add debug menu entry for installing/removing bionics

* Oops
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
JSON related to game datas in JSON format. src changes related to source code.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant