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

Enable *plagiarism script commands #6304

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

Conversation

Emistry
Copy link
Contributor

@Emistry Emistry commented Oct 11, 2021

  • Addressed Issue(s):
    unable to control the plagiarism skill using npc
  • Server Mode:
    both
  • Description of Pull Request:
    enable to use script command to control skill plagiarism if needed.
    enable player to copy or erase skill using NPC

Copy link
Contributor

@aleos89 aleos89 left a comment

Just a quick first review!

doc/script_commands.txt Outdated Show resolved Hide resolved
doc/script_commands.txt Outdated Show resolved Hide resolved
doc/script_commands.txt Outdated Show resolved Hide resolved
doc/script_commands.txt Outdated Show resolved Hide resolved
src/map/pc.cpp Outdated Show resolved Hide resolved
src/map/pc.cpp Outdated Show resolved Hide resolved
src/map/pc.cpp Outdated Show resolved Hide resolved
src/map/script.cpp Outdated Show resolved Hide resolved
src/map/script.cpp Outdated Show resolved Hide resolved
src/map/script.cpp Outdated Show resolved Hide resolved
Copy link
Contributor

@aleos89 aleos89 left a comment

Just a few more things I could see since I no longer have to review on a mobile device!

src/map/pc.cpp Outdated Show resolved Hide resolved
src/map/pc.cpp Outdated Show resolved Hide resolved
src/map/pc.cpp Outdated Show resolved Hide resolved
src/map/pc.cpp Outdated Show resolved Hide resolved
src/map/pc.cpp Outdated Show resolved Hide resolved
src/map/pc.hpp Outdated Show resolved Hide resolved
src/map/script.cpp Outdated Show resolved Hide resolved
src/map/script.cpp Outdated Show resolved Hide resolved
src/map/script.cpp Outdated Show resolved Hide resolved
src/map/script.cpp Outdated Show resolved Hide resolved
@Emistry
Copy link
Contributor Author

@Emistry Emistry commented Feb 5, 2022

updated and rebased.

@aleos89
Copy link
Contributor

@aleos89 aleos89 commented Feb 5, 2022

So I'm good with this, but maybe the script name could use some loving? Something like plagiarizeskill and plagiarizeskillreset.

@Emistry
Copy link
Contributor Author

@Emistry Emistry commented Feb 18, 2022

So I'm good with this, but maybe the script name could use some loving? Something like plagiarizeskill and plagiarizeskillreset.

I would prefer the previous name since its easier to read, anyway updated it to plagiarizeskill and plagiarizeskillreset

@aleos89
Copy link
Contributor

@aleos89 aleos89 commented Feb 18, 2022

You can also set the alias if you'd like, so that we have both names available! I see no harm in that as long as both are documented in the script_commands. 👍🏼

@Emistry
Copy link
Contributor Author

@Emistry Emistry commented Feb 18, 2022

is okay, keep one is enough, wont take long to get used to it.

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.

None yet

3 participants