Jump to content
Sign in to follow this  
KingOfThePing

Naming Animals

Recommended Posts

I greet you, Aurorafolk. I bring, once again, a suggestion of utmost importance.

 

As many of you know, Pugs are snowflakes. With a right click you can name them. Completely without admin intervention. Other animals, non-snowflakey as they are, cannot be named. I suggest changing that. I feel like an idiot, whenever I press F1 to ask "can someone bls name this cat Mr. Pawsy, thank you". Since the code to name animals clearly already exists, please extend that to the corgis, cats, kittens and maybe even the fathouters. And cows, maybe, idk, perhaps chefs want to bond or want to name the thing they shove into the gibber. Also parrots. I can't think of more. Don't include rats, maybe. Maybe do, I'm not your boss.

 

That's all folks. Stay tuned for another highly important suggestion from me, thank you.

Share this post


Link to post

On this episode of "Doxx is KingOfThePing's Favorite Person": Already planned for this is a series of animal updates. This was going to be in the first one, along with another feature and some adjustments. Keep an eye out; I may get to it as soon as this weekend. 

edit: For some reason it's limited to guard dogs in code, which is why it works for pugs and other commandable dogs. I have most of the code written already, pending testing and stuff.

Edited by Doxxmedearly

Share this post


Link to post
19 minutes ago, Doxxmedearly said:

On this episode of "Doxx is KingOfThePing's Favorite Person": Already planned for this is a series of animal updates. This was going to be in the first one, along with another feature and some adjustments. Keep an eye out; I may get to it as soon as this weekend. 

edit: For some reason it's limited to guard dogs in code, which is why it works for pugs and other commandable dogs. I have most of the code written already, pending testing and stuff.

Please hold off on this. I'm already implementing a name change helper in my https://github.com/Aurorastation/Aurora.3/pull/7382 PR, and redundant code is not a great idea.  I'd be happy to adapt the name change helper to station pets after the PR is merged.

Share this post


Link to post

I will wait until your PR is resolved to handle it, but looking at it, doesn't seem to conflict with renaming most animals in the way that renaming dogs currently functions, which was what my work was based on (Which is as simple as extending the verb to most animals). 

Edited by Doxxmedearly

Share this post


Link to post
3 minutes ago, Doxxmedearly said:

I will wait until your PR is resolved to handle it, but looking at it, doesn't seem to conflict with renaming most animals in the way that renaming dogs currently functions, which was what my work was based on (Which is as simple as extending the verb to most animals). 

I was planning to rewrite doggo code too once the PR finishes so that everyone uses the same namechange helper support proc which includes stuff like admin notices of name changes.

Share this post


Link to post

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Sign in to follow this  

×
×
  • Create New...