suriyun-production / mmorpg-kit-docs

This is document for MMORPG KIT project (https://www.assetstore.unity3d.com/#!/content/110188?aid=1100lGeN)
https://suriyun-production.github.io/mmorpg-kit-docs
49 stars 10 forks source link

maxLevel in BaseSkill #2609

Closed Eniotnacram closed 2 weeks ago

Eniotnacram commented 2 weeks ago

Hello,

Is it possible to make the MaxLevel int in BaseSkill a property instead or in any other way that it is possible to override in upper classes?

Or remove the range of it? Because I would like my max skill level to be 200 but it's getting locked to 100 without any modifs. If I make the Maxlevel "new" in my class, it gives serialization errors in the editor.

insthync commented 2 weeks ago

Hmm, how is it being locked?

Eniotnacram commented 2 weeks ago

Because of the range set from 1 to 100 in the baseSkill

insthync commented 2 weeks ago

Cannot set its max level to 200?

Eniotnacram commented 2 weeks ago

If I modify the BaseSkill yes, but that's a core modif and not convenient. I just don't get why there is a range for that. We could remove it and if we want 1000 levels we could set it up

Eniotnacram commented 2 weeks ago

As per the screen, because of the range, the max level is set to 100. So in Skill, or any other class that inherits base skill, the max level is 100 and cannot be changed as it's not overridable. The only option is to modify the BaseSkill class.

Screenshot 2024-11-11 at 8 59 08 AM

insthync commented 2 weeks ago

Oh, i see, it actually must not have range limit

Eniotnacram commented 2 weeks ago

Oh, I see, it actually must not have range limit

Yes, that would be better, and more flexible.