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

Fixed NPC tracking events and more #69

Merged

Conversation

Leidtier
Copy link
Contributor

@Leidtier Leidtier commented Aug 2, 2024

  • Fixed NPC tracking events not working.
    • Added MantellaTargetListenerScript.psc to QuestAliases of MantellaConversationParticipantsQuest
    • Deleted MantellaEvents quest that wasn't doing anything
  • Added 9 more possible QuestAlias to MantellaConversationParticipantsQuest so conversations can have a maximum of 12 participants
  • Made tracking of time and weather optional
    • Added MCM options for tracking time and weather to MantellaMCM_PlayerTrackingSettings.psc. Moved them to a new category Track World Events on the right column together with the location tracking option.
  • Fixed extra tracking of location through events as this is now part of the context
  • Fixed looking at the wrong slot for helmets (should be 31, was 30)

- Fixed NPC tracking events not working.
  - Added `MantellaTargetListenerScript.psc` to QuestAliases of `MantellaConversationParticipantsQuest`
  - Deleted `MantellaEvents` quest that was doing anything
- Added 7 more possible QuestAlias to `MantellaConversationParticipantsQuest` so conversations can have a maximum of 10 participants
- Made tracking of time and weather optional
  - Added MCM options for tracking time and weather to `MantellaMCM_PlayerTrackingSettings.psc`. Moved them to a new category *Track World Events* on the right column together with the location tracking option.
- Fixed extra tracking of location through events as this is now part of the context
- Fixed looking at the wrong slot for helmets (should be 31, was 30)
@Leidtier
Copy link
Contributor Author

Leidtier commented Aug 4, 2024

One second on the merge here. I made a mistake that assigned both tracking scripts to the player.

@Leidtier
Copy link
Contributor Author

Leidtier commented Aug 4, 2024

Fixed the issue of the PC getting both tracking scripts assigned.

@art-from-the-machine art-from-the-machine merged commit 2e13ba9 into art-from-the-machine:main Aug 8, 2024
@Leidtier Leidtier deleted the fix-npc-tracking branch August 8, 2024 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants