Skip to content

Added WalkSpeed to API#47

Open
Nemot1 wants to merge 1 commit into
Polytoria:mainfrom
Nemot1:api-player-walkspeed
Open

Added WalkSpeed to API#47
Nemot1 wants to merge 1 commit into
Polytoria:mainfrom
Nemot1:api-player-walkspeed

Conversation

@Nemot1

@Nemot1 Nemot1 commented May 25, 2026

Copy link
Copy Markdown
Contributor

Added WalkSpeed in yaml file (yaml/types/Player.yaml) and description for this

Signed-off-by: Nemot1 <spostis101@gmail.com>
@retrievaaaa

Copy link
Copy Markdown

this is redundant as its already in NPC, which Player inherits from

@Nemot1

Nemot1 commented May 25, 2026

Copy link
Copy Markdown
Contributor Author

this is redundant as its already in NPC, which Player inherits from

Alright, but from a documentation and user experience point, many new developers creators looking specifically at the Player class, for example, on forum most of the new developers are kinda confused with understanding all this stuff.

In future also should be added more context and stuff into v2 documentation. As i said earlier, most of new developers is hard to understand information due to how small info about polytoria creator. Anyways, i think thats right opinion on it, so why not make some context between NPC and Player to make readers understand differences between them, there is already context that Inherits NPC, but as i think that WalkSpeed should be at least in here, however, if don't wanna to or you don't like this i can at any time close PR

@retrievaaaa

Copy link
Copy Markdown

It would be inconsistent with the rest of the docs. I don't think redocumenting every property on every subclass is a good idea personally

but its up to willem to decide 🤷

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

Successfully merging this pull request may close these issues.

2 participants