-
Notifications
You must be signed in to change notification settings - Fork 18
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
added a translate command #246
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some changes needed
cogs/commands/misc.py
Outdated
@@ -143,6 +143,17 @@ async def chat(self, ctx: Context, message: str = None, gpt4: bool = False): | |||
async def gpt4(self, ctx: Context, message: str): | |||
await self.chat(ctx, message, True) | |||
|
|||
@commands.hybrid_command() | |||
async def translate( | |||
self, ctx: Context, source: str, target: str, message: clean_content |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add a help text to explain what source and target are (see
Line 15 in 4d30f63
@commands.hybrid_command(help=LONG_HELP_TEXT, brief=SHORT_HELP_TEXT) |
Also might be an idea to set a deafult value of source to en or auto
cogs/commands/misc.py
Outdated
await ctx.send(translated) | ||
|
||
@commands.hybrid_command() | ||
async def joeltech(self, ctx: Context): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tbh im not entirely sure if we need this given :joel_tech: can already be written
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the command should be Joel not Joel tech for this to be effective
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed it for now, will make something mightier in the future :))
use of camelCase in pytho brings me pain but fine |
No description provided.