Create takina willyoumarryme. #1

Closed
Yunexiz wants to merge 3 commits from patch-1 into main
Yunexiz commented 2025-01-14 09:08:03 +03:00 (Migrated from github.com)

I humbly request you merge this. At the mercy of your choice, I shall await a new command in Takina. All shall fair well for Takina's wedding(s).

I humbly request you merge this. At the mercy of your choice, I shall await a new command in Takina. All shall fair well for Takina's wedding(s).
iostpa (Migrated from github.com) reviewed 2025-01-14 09:08:03 +03:00
orangci (Migrated from github.com) requested changes 2025-01-14 09:13:06 +03:00
orangci (Migrated from github.com) left a comment

All these mistakes aside why would I add this brother 🙏

All these mistakes aside why would I add this brother 🙏
orangci (Migrated from github.com) commented 2025-01-14 09:09:52 +03:00
            title=f"{ctx.author.mention} proposed to {BOT_NAME}",
```suggestion title=f"{ctx.author.mention} proposed to {BOT_NAME}", ```
orangci (Migrated from github.com) commented 2025-01-14 09:11:43 +03:00
            await ctx.reply(embed=embed, mention_author=False)
```suggestion await ctx.reply(embed=embed, mention_author=False) ```
orangci (Migrated from github.com) commented 2025-01-14 09:11:52 +03:00
            await ctx.reply(embed=embed, mention_author=False)
```suggestion await ctx.reply(embed=embed, mention_author=False) ```
orangci (Migrated from github.com) commented 2025-01-14 09:12:00 +03:00
            await ctx.reply(embed=embed, mention_author=False)
```suggestion await ctx.reply(embed=embed, mention_author=False) ```
orangci (Migrated from github.com) commented 2025-01-14 09:12:22 +03:00
            title=f"{interaction.user.mention} proposed to {BOT_NAME}",
```suggestion title=f"{interaction.user.mention} proposed to {BOT_NAME}", ```
orangci (Migrated from github.com) commented 2025-01-14 09:12:31 +03:00
        description="Takina replies to whether or not marriage is feasible."
```suggestion description="Takina replies to whether or not marriage is feasible." ```
orangci (Migrated from github.com) requested changes 2025-01-14 09:13:59 +03:00
orangci (Migrated from github.com) commented 2025-01-14 09:13:35 +03:00
            await interaction.send(embed=embed, ephemeral=True)
```suggestion await interaction.send(embed=embed, ephemeral=True) ```
orangci (Migrated from github.com) commented 2025-01-14 09:13:41 +03:00
            await interaction.send(embed=embed, ephemeral=True)
```suggestion await interaction.send(embed=embed, ephemeral=True) ```
orangci (Migrated from github.com) commented 2025-01-14 09:13:47 +03:00
            await interaction.send(embed=embed, ephemeral=True)
```suggestion await interaction.send(embed=embed, ephemeral=True) ```
Yunexiz commented 2025-01-14 09:15:20 +03:00 (Migrated from github.com)

All these mistakes aside why would I add this brother 🙏

Pretend you didnt see that. Also I had no idea you had the bot name constant.

> All these mistakes aside why would I add this brother 🙏 Pretend you didnt see that. Also I had no idea you had the bot name constant.
Yunexiz commented 2025-01-14 09:15:55 +03:00 (Migrated from github.com)

Maybe should've checked, anyway. Back to your first point. You should merge this because it is a very requested feature.

Maybe should've checked, anyway. Back to your first point. You should merge this because it is a very requested feature.
orangci commented 2025-01-14 09:18:06 +03:00 (Migrated from github.com)

very requested feature

requests by simps are not relevant

> very requested feature requests by simps are not relevant
Yunexiz commented 2025-01-14 09:20:18 +03:00 (Migrated from github.com)

very requested feature

requests by simps are not relevant

Om would agree. This is very important to me and santinder aswell.

> > very requested feature > > requests by simps are not relevant Om would agree. This is very important to me and santinder aswell.
orangci commented 2025-01-14 09:23:12 +03:00 (Migrated from github.com)

Om would agree. This is very important to me and santinder aswell.

if you apply my suggested changes and move it somewhere in cogs/sesp/isadev ill merge this

> Om would agree. This is very important to me and santinder aswell. if you apply my suggested changes and move it somewhere in cogs/sesp/isadev ill merge this
Yunexiz commented 2025-01-14 09:25:11 +03:00 (Migrated from github.com)

Om would agree. This is very important to me and santinder aswell.

if you apply my suggested changes and move it somewhere in cogs/sesp/isadev ill merge this

You're on!

> > Om would agree. This is very important to me and santinder aswell. > > if you apply my suggested changes and move it somewhere in cogs/sesp/isadev ill merge this You're on!
Yunexiz commented 2025-01-14 09:26:54 +03:00 (Migrated from github.com)

I'm going to sleep though, maybe in a couple hours.

I'm going to sleep though, maybe in a couple hours.
Satindar31 commented 2025-01-14 19:43:05 +03:00 (Migrated from github.com)

very requested feature

requests by simps are not relevant

Om would agree. This is very important to me and santinder aswell.

My name is Satindar 😭

> > > very requested feature > > > > requests by simps are not relevant > > Om would agree. This is very important to me and santinder aswell. My name is Satindar 😭

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: c/takina#1
No description provided.