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

Stop people in refugee center from bashing and tell them to return to places. #62472

Closed
wants to merge 34 commits into from

Conversation

I-am-Erk
Copy link
Contributor

@I-am-Erk I-am-Erk commented Nov 29, 2022

Summary

Bugfixes "Adds NO_BASH and RETURN_TO_START_POS to most NPCs in refugee center."

Purpose of change

Partially addresses #62470

Describe the solution

Gives all the surface refugees and guards the listed traits, but allows them to still investigate sounds. I may give some of the quest giving NPCs the no investigate flag as well.

Created a novel guard class for the refugee center in the process rather than overriding traits. This gives them all an evac jacket which should be nice and uniform. These are the interior guards at the center, and so I have assumed these folks have minimal training and are using melee weapons, not guns. Later, in Experimental, I'd like to propose a quest line to help them improve their training and equipment.

Describe alternatives you've considered

As above - some of them could probably get no investigate as well.

Testing

TBA - at work.

@I-am-Erk I-am-Erk added NPC / Factions NPCs, AI, Speech, Factions, Ownership <Bugfix> This is a fix for a bug (or closes open issue) labels Nov 29, 2022
@github-actions github-actions bot added the [JSON] Changes (can be) made in JSON label Nov 29, 2022
@github-actions github-actions bot added the astyled astyled PR, label is assigned by github actions label Nov 29, 2022
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot added the json-styled JSON lint passed, label assigned by github actions label Nov 29, 2022
@I-am-Erk I-am-Erk marked this pull request as ready for review November 30, 2022 22:16
@MNG-cataclysm
Copy link
Contributor

Apparently, the new guard class you created is considered invalid by the tests.

@@ -8,7 +8,8 @@
"attitude": 0,
"mission": 3,
"chat": "TALK_FREE_MERCHANT_TEAMSTER",
"faction": "free_merchants"
"faction": "free_merchants",
"traits": [ { "trait": "IGNORE_SOUND" }, { "trait": "NO_BASH" }, { "trait": "RETURN_TO_START_POS" } ]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to move the traits to the NPC Class definition. They aren’t registered here.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm aware. Just been too burnt out from work to fix it for a few days.

@MNG-cataclysm MNG-cataclysm mentioned this pull request Jan 26, 2023
bombasticSlacks pushed a commit that referenced this pull request Jan 26, 2023
* Update NPC_Aleesha_Seward.json

* Update NPC_Alonso_Lautrec.json

* Update NPC_Boris_Borichenko.json

* Update NPC_Dana_Nunez.json

* Update NPC_Draco_Dune.json

* Update NPC_Fatima_Al_Jadir.json

* Update NPC_Garry_Villeneuve.json

* Update NPC_Guneet_Singh.json

* Update NPC_Jenny_Forcette.json

* Update NPC_John_Clemens.json

* Update NPC_Mandeep_Singh.json

* Update NPC_Mangalpreet_Singh.json

* Update NPC_Pablo_Nunez.json

* Update NPC_Rhyzaea_Johnny.json

* Update NPC_Stan_Borichenko.json

* Update NPC_Uyen_Tran.json

* Update NPC_Vanessa_Toby.json

* Update NPC_free_merchant_guard1.json

* Update NPC_free_merchant_guard2.json

* Update NPC_free_merchant_guard3.json

* Update NPC_free_merchant_guard_generic.json

* Update NPC_free_merchant_hostile_guard.json

* Update NPC_free_merchant_teamster.json

* Update NPC_free_merchant_teamster.json

* Create Generic_class_definitions

* Linting

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update NPC_free_merchant_guard1.json

* Update NPC_free_merchant_guard2.json

* Update NPC_free_merchant_guard3.json

* Update NPC_free_merchant_guard_generic.json

* Update NPC_free_merchant_hostile_guard.json

* Update Generic_class_definitions

* Update Generic_class_definitions

* Add worn override

* Apply my suggestions

* restart tests

Co-authored-by: I-am-Erk <45136638+I-am-Erk@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@MNG-cataclysm
Copy link
Contributor

As #63289 merged the content from this PR, it should probably be closed.

@Maleclypse
Copy link
Member

Completed by #63289

@Maleclypse Maleclypse closed this Feb 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions <Bugfix> This is a fix for a bug (or closes open issue) [JSON] Changes (can be) made in JSON json-styled JSON lint passed, label assigned by github actions NPC / Factions NPCs, AI, Speech, Factions, Ownership
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants