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

Achievement stuff #2690

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

DeltaFire15
Copy link
Contributor

@DeltaFire15 DeltaFire15 commented Jun 30, 2024

About The Pull Request

We have some custom NSV achievements that have been broken for a long while (if they ever worked)
This fixes them and or makes them work in the first place.

Also adds a bunch of new achievements that fit NSV, aswell as a score counter for people making torpedoes the old fashioned way.

I don't have a database setup so I can't actively test this too much, but it compiles and should work.

Modifies star system config. If this PR is merged or testmerged, the star_system default json must be updated to the one provided here (or manually compared). You must also either reset the persistent json, or update the values of the systems changed here manually (two values in total, so should be managable)

Why It's Good For The Game

A sense of pride and accomplishment for completing objectives.

Changelog

🆑
fix: The NSV-specific achievements should work now.
add: Some more NSV-specific achievements.
/:cl:

@DeltaFire15 DeltaFire15 added Config Update Fix Overmap This issue or PR is related to overmap interactions labels Jun 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Config Update Fix Overmap This issue or PR is related to overmap interactions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant