Skip to content

Add daddy as a command alias#385

Closed
extendedclip wants to merge 1 commit into
masterfrom
daddy
Closed

Add daddy as a command alias#385
extendedclip wants to merge 1 commit into
masterfrom
daddy

Conversation

@extendedclip
Copy link
Copy Markdown
Member

@extendedclip extendedclip commented Jul 28, 2020

Please read

Please make sure you checked the following:

  • You checked the Pull requests page for any upcoming changes.
  • You documented any public code that the end-user might use.
  • You followed the contributing file

Type

  • Internal change (Doesn't affect end-user).
  • External change (Does affect end-user).
  • Other: __________

Description

Provide additional information if needed.

Do not delete this PR. Lets talk about it. I want to know why its so bad... I personally like the alias and would use it over /papi any day.

@PiggyPiglet
Copy link
Copy Markdown
Member

So long it's behind a permission like @Sxtanna said, there's no reason this can't be added back.

@LemmoTresto
Copy link
Copy Markdown
Member

I'm all for having the alias, afaik it was only removed due to someone complaining. So I'd agree of having it behind a permission.

@extendedclip
Copy link
Copy Markdown
Member Author

The problem is /daddy will still return the no permission message.

Copy link
Copy Markdown
Member

@Funny-cube Funny-cube left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pls no drama

Copy link
Copy Markdown
Contributor

@Sxtanna Sxtanna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merge #255 before this. It's 5 months old.

Copy link
Copy Markdown
Member

@iGabyTM iGabyTM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a big fan of this idea to be honest.

@Sxtanna Sxtanna removed the request for review from BlitzOffline July 29, 2020 21:56
@Sxtanna Sxtanna closed this Jul 29, 2020
@BomBardyGamer
Copy link
Copy Markdown

BomBardyGamer commented Jul 30, 2020

This should've been merged just for the memes lol

@Andre601 Andre601 deleted the daddy branch August 4, 2020 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.