Skip to content

Scripts/ExilesReach: Update and add conversation for quests 58208/59928#31759

Open
Saburo-Arasaka wants to merge 8 commits intoTrinityCore:masterfrom
Saburo-Arasaka:58208_59928
Open

Scripts/ExilesReach: Update and add conversation for quests 58208/59928#31759
Saburo-Arasaka wants to merge 8 commits intoTrinityCore:masterfrom
Saburo-Arasaka:58208_59928

Conversation

@Saburo-Arasaka
Copy link
Copy Markdown
Contributor

Add conversation for quests 58208/59928
Add Garrick
Reworked npc scripts for quests 58208/59928

Add conversation for quests 58208/59928
Add Garrick
Copy link
Copy Markdown
Member

@mdX7 mdX7 left a comment

Choose a reason for hiding this comment

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

i'd prefer to use templates for npc_first_mate_brace_for_impact_private and not use so many member variables.

template<uint32 Path, uint32 NpcID, uint32 Conversation, uint32 Actor0, uint32 Actor1>

@Saburo-Arasaka Saburo-Arasaka requested a review from mdX7 March 30, 2026 13:17
@imstuckbro
Copy link
Copy Markdown

usually a database sql file should actually have .sql extension format

@Saburo-Arasaka Saburo-Arasaka changed the title Scripts/ExilesReach Add conversation for quests 58208/59928 Scripts/ExilesReach Update and add conversation for quests 58208/59928 Mar 30, 2026
@Saburo-Arasaka Saburo-Arasaka changed the title Scripts/ExilesReach Update and add conversation for quests 58208/59928 Scripts/ExilesReach: Update and add conversation for quests 58208/59928 Mar 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants