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

Scripts scripted tarindrella #161

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

Conversation

zTerragor
Copy link
Member

Scripts: Scripted Tarindrella.
Need test Port from last SkyFire

Scripts: Scripted Tarindrella.
Scripts: Scripted Tarindrella.
Scripts: Scripted Tarindrella.
@zTerragor zTerragor added the enhancement New feature or request label Nov 16, 2023
@zTerragor zTerragor self-assigned this Nov 16, 2023
SPELL_CLEANSE_SPIRIT = 66056

};
#define TARINDRELLA_TEXT_ON_COMPLETE "This totem has been corrupting the eggs! It seems a greater threat looms. The Gnarlpine remain tainted by something most foul."
Copy link

Choose a reason for hiding this comment

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

Change to Creature Text

Copy link
Member Author

Choose a reason for hiding this comment

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

#define TARINDRELLA_CREATURE_TEXT_ON_COMPLETE "This totem has been corrupting the eggs! It seems a greater threat looms. The Gnarlpine remain tainted by something most foul."

Copy link

Choose a reason for hiding this comment

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

creature text non exist in broadcast_text,seems wrong.

Copy link

Choose a reason for hiding this comment

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

Copy link

Choose a reason for hiding this comment

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

According to video,This script seems wrong.

@zTerragor zTerragor requested a review from leelf00 November 17, 2023 14:48
@leelf00
Copy link

leelf00 commented Nov 17, 2023

@leelf00
Copy link

leelf00 commented Nov 17, 2023

@leelf00
Copy link

leelf00 commented Nov 17, 2023

Her 2 quest video.No creature text is shown.Script maybe wrong

@@ -0,0 +1 @@
UPDATE `creature_template` SET `ScriptName`='npc_tarindrella' WHERE `entry`=49480;
Copy link

Choose a reason for hiding this comment

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

Her two quests show in video after 4.0.3 is different from this script.

Choose a reason for hiding this comment

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

missing to add code script cpp for this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants