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

Bug or feature: choosing a Skill tree at higher lvl counts as switch #37

Closed
liquidox opened this issue Mar 5, 2013 · 6 comments
Closed

Comments

@liquidox
Copy link

liquidox commented Mar 5, 2013

A player levels a Pet, without choosing any skill tree.

Then say, at Level 3, picks a skill tree, the switch-penalty is applied, and the Pet can/will lose levels.

Any Pet without a skill tree will always be weaker than one with, so it's likely fine to not apply the penalty in this case?

However, I'd prefer a config option, that prevents a pet from levelling up, without choosing a skill tree. Ideally it would inform/remind the player to choose one, when the Pet has enough XP to level up.

Don't know how complicated this is to add, it's not too critical ;)

@scomans
Copy link
Member

scomans commented Mar 5, 2013

The part with the skilltree switch penalty works fine for me. Even when the mypet had no skilltree before.

@liquidox
Copy link
Author

liquidox commented Mar 5, 2013

Eh did you misunderstand? I meant it also costs levels/xp when you choose a skill-tree for the first time, and your pet already had levels...

@scomans
Copy link
Member

scomans commented Mar 5, 2013

I know what you mean and as I said this works for me. When my pet doesn't have a skilltree and I use /pcst no exp will be removed. Do you have MyPet.Skilltree.AutomaticAssignment enabled (true) in the config?

@liquidox
Copy link
Author

liquidox commented Mar 5, 2013

Yes, have that enabled, but currently have two skilltrees implemented, so no tree is auto-picked.

Anyways, this issue was only tested and reported by a player, so I'll have to verify it myself, maybe he was mistaken.

@scomans
Copy link
Member

scomans commented Mar 5, 2013

When you have this option enabled the plugin will autopick the first skilltree that is usable for this player. Maybe this player didn't know that he already had a skilltree.

@scomans scomans closed this as completed in 06554d8 Mar 5, 2013
@liquidox
Copy link
Author

liquidox commented Mar 5, 2013

Aaah didn't know that either, that explains that, thanks :D

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

No branches or pull requests

2 participants