#dev-log

1 messages ยท Page 4 of 1

regal archBOT
#
wookie184
[python-discord/bot] branch deleted: move\_security\_cog
oak estuaryBOT
odd spireBOT
regal archBOT
#
TizzySaurus
[python-discord/sir-lancebot] New comment on pull request #1129: Add ability to use replied message's content as \`\.randomcase\` argument\.

I think it would be nice if this supported randomcaseing replies to embeds, given this command already supports that through message links, although I don't really mind.

This isn't really possible to implement without making the code very message, since clean_text_or_reply doesn't have this feature. Users will at least still have the option of doing via message links (and said message link could be in the replied message).

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
regal archBOT
#
wookie184
[python-discord/bot] New review comment on pull request #2307: Talentpool Cleanup

It was refreshed pretty much every time it was accessed which made it pretty pointless, as if you make an API call every time anyway it defeats the point of a cache. There were also a bunch of times where it was refreshed even though it didn't need to be, so i'd expect this PR actually makes less API calls now.

It would be possible to reimplement it properly (in _api.py so it's more contained), although it's not a very high-throughput cog so it's not really necessary and is just extra ...

odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
#
gustavwilliam
[python-discord/site] New branch created: ot\-etiquette\-update
odd spireBOT
regal archBOT
oak estuaryBOT
regal archBOT
#
gustavwilliam
[python-discord/branding] New branch created: lemoji/holding\-back\-tears
odd spireBOT
regal archBOT
#
ionite34
[python-discord/bot] New branch created: rules\-fix
#
minalike
[python-discord/site] branch deleted: ot\-etiquette\-update
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
gustavwilliam
[python-discord/bot] New comment on issue #1295: Improving dormant help channel embed

What about a simple embed like this?

<img width="538" alt="Screenshot 2022-10-25 at 07 27 09" src="https://user-images.githubusercontent.com/65498475/197689997-a88a11ff-f888-4e81-958e-93380b7b4e33.png">

Here's the raw text:

This help channel has been closed and it's no longer possible to send messages here. If your question wasn't answered yet, feel free to ask in a new help channel by checking out [#โ“๏ฝœhow-to-get-help](/guild/267624335836053506/channel/704250143020417084/). To maximize your chances of getting a response, check out th...
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] New review comment on pull request #2310: Fix for rules greedy parsing freezing

Even in that case, it would still only take microseconds. All this loop does it try to convert to an int, on failure it calls .lower() and then does a dict lookup.

All of those are very fast things, so likely won't impact performance enough to warrant the increased complexity of adding a limit of 50 iterations.

Also, I am entirely fine with this being a separate issue/PR, rather than blocking this one.

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/bot-core] New branch created: Python\-3\.11
#
ChrisLovering
[python-discord/bot-core] New tag created: v8\.3\.0\-beta
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
ionite34
[python-discord/snekbox] New branch created: bytes\-output
regal archBOT
#
mbaruh
[python-discord/bot-core] Issue opened: #152 get\_or\_fetch\_member can raise discord\.errors\.HTTPException

https://github.com/python-discord/bot-core/blob/6ae78684ee0e61d8a6ad8bb779ea30f2e5294d0d/botcore/utils/members.py#L22-L26

This can raise discord.errors.HTTPException for an invalid user ID (for example a snowflake too far into the future). This should be handled as the user not being found instead.

I'm 99% sure I can make this error in prod, but would rather save a Sentry alert.

regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
jchristgit
[python-discord/infra] New branch created: add\-role\-creation\-command
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
wookie184
[python-discord/bot] New comment on pull request #2285: Add slicing tag

Yes, I consider that feasible, but keep in mind that sentences must start with a capital letter. In line 6:

- *slicing* is a way of accessing a part of a sequence by specifying a start index, stop index, and step to take elements at.
+ *Slicing* is a way of accessing a part of a sequence by specifying a start index, stop index, and step to take elements at.

Other than that, I'm blown away at your proposal and I'm definitely accepting it.

Thanks :) And yeah that cap...

#
Xithrius
[python-discord/sir-lancebot] branch deleted: randomcase\-reply\-enhancement
night lilyBOT
#
Sir Lancebot

Connected!

#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
swfarnsworth
[python-discord/bot] New comment on pull request #2285: Add slicing tag

Thank you for taking the initiative to create this tag. I think slicing is a great candidate for having a tag, and that we've been without one for too long.

We need to be very intentional about what and how much information is conveyed through a tag. In that light, there's a lot that I'd like to see different about your proposed tag. Let me propose this alternative:

<pre>

aliases: ["slice", "seqslice", "seqslicing", "sequence-slice", "sequence-slicing"]
embed:
title: "Sequen...

night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
MarkKoz
[python-discord/bot] New comment on pull request #2230: Add nomodule tag

Explains well in detail about the module not found and how to fix but doesn't necessarily mention about what a module actually is. Seems more like a temporary fix rather than teaching the reader what exactly a module is and how to avoid module not found.

Tags must be limited in scope. Otherwise they become long and loose their effectiveness. I don't think it's possible to explain what a module is in this tag in addition to what the error is without making the content long and overwhelming....

odd spireBOT
regal archBOT
regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
#
wookie184
[python-discord/site] New branch created: messages\-in\-past\-n\-days\-endpoint
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
#
Xithrius
[python-discord/sir-lancebot] New branch created: Diabolical5777/main
#
Xithrius
[python-discord/sir-lancebot] branch deleted: Diabolical5777/main
regal archBOT
regal archBOT
#
mbaruh
[python-discord/site] New branch created: mbaruh\-bump\-psql
odd spireBOT
regal archBOT
#
mbaruh
[python-discord/bot] New branch created: mbaruh\-bump\-psql
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
mbaruh
[python-discord/site] branch deleted: mbaruh\-bump\-psql
odd spireBOT
regal archBOT
regal archBOT
#
jchristgit
[python-discord/sir-lancebot] New review comment on pull request #1095: Use buttons for bookmark command\.

There's a possible race condition here if a user clicks the button multiple times in very short succession (assuming that's possible with ratelimits). In that case, we send out responses multiple times before the self.clicked.append line is hit. The fix would probably be appending to clicked directly below the if statement block.
Since we expect uniqueness of the users, self.clicked should also probably be a set -> self.clicked = {author.id}.

regal archBOT
#
shenanigansd
[python-discord/bot] Issue opened: #2313 Update d\.py intents tag to clarify that privleged intents require activation in D\.py 2\.0

Raise by Roie#0089 here:

As of Discord.py 2.0, privileged intents are now required to be explicitly activated.
We would like to add something in the tag body to help clarify this - for example -

Starting from version 2.0, privileged intents must now be passed into the bot constructor as a required keyword argument intents and also be explicitly enabled on the developer portal.

odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
#
ks129
[python-discord/sir-lancebot] New branch created: bug/1109/wolfram\-api\-error\-handling
#
ks129
[python-discord/sir-lancebot] Pull request opened: #1136 Catch more status codes in Wolfram commands

Relevant Issues

Closes #1109
<!-- Link the issue by typing: "Closes #" (Closes #0 to close issue 0 for example). -->

Description

Added catching of all non-200 HTTP codes that didn't get caught before.

Did you:

  • [x] Join the Python Discord Community?
  • [x] Read all the comments in this template?
  • [x] Ensure there is an issue open, or link relevant discord discussions?
  • [x] Read and agree to the [contributing guidelines](htt...
odd spireBOT
regal archBOT
#
ks129
[python-discord/bot] New branch created: bug/2299/timeit
regal archBOT
odd spireBOT
regal archBOT
#
Xithrius
[python-discord/sir-lancebot] branch deleted: bug/1109/wolfram\-api\-error\-handling
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
Robin5605
[python-discord/bot] Pull request opened: #2316 Add safeguard when banning staff

Closes #2314

Created a subclass of botcore.utils.Interactions.ViewWithUserAndRoleCheck that will be sent when a moderator attempts to ban staff (helper role or above) in bot/exts/moderation/infraction/_utils.py. Only the !ban command author and moderators are allowed to interact with this view. If cancelled, the view is deleted (NOTE: not the entire message). Similarly, if the timeout (10 seconds) is reached, the view will be deleted (once again, not the entire message). If confirme...

odd spireBOT
odd spireBOT
regal archBOT
#
Robin5605
[python-discord/bot] New review comment on pull request #2316: Add safeguard when banning staff

The superclass uses it in their on_timeout function: https://github.com/python-discord/bot-core/blob/main/botcore/utils/interactions.py#L65-L70

    async def on_timeout(self) -> None:
        """Remove the view from ``self.message`` if set."""
        if self.message:
            with contextlib.suppress(NotFound):
                # Cover the case where this message has already been deleted by external means
                await self.message.edit(view=None)
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
#
wookie184
[python-discord/bot] branch deleted: mbaruh\-bump\-psql
oak estuaryBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
shtlrs
[python-discord/bot] New review comment on pull request #2316: Add safeguard when banning staff

Question: Can't the code used in confirm and cancel be moved to a "private" method instead ?
It's so similair, and the only thing changing is the value

async __set_value(self, interaction: Interaction, button: Button, value: bool):
        await interaction.response.defer()
        self.value = value
        await super().on_timeout()
        self.stop()
   
@discord.ui.button(label="Cancel", style=ButtonStyle.red)
    async def cancel(self, interaction: Interaction, but...
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
#
HassanAbouelela
[python-discord/snekbox] Pull request opened: #153 Bump To 3\.11

This PR bumps the snekbox version to the latest stable python version.

  • [x] Bump version in dockerfile
  • [x] Bump version in documentation
  • [x] Relock dependency versions
  • [ ] Possibly look into bumping dependency versions?
  • [ ] Upgrade and test runtime packages

I'll upgrade where possible to avoid the image having to compile a bunch of packages itself, to speed up builds, however we might have to revisit in the future as I imagine a lot of packages don't have wheels yet.

regal archBOT
regal archBOT
#
HassanAbouelela
[python-discord/snekbox] New branch created: 3\.11\-upgrade2
regal archBOT
regal archBOT
#
CaedenPH
[python-discord/bot] Issue opened: #2317 Delete 'Applied mute to โ€ฆ' message

What feature are you proposing?

When a user sends multiple messages, they get muted automatically with the python discord bot, and a message is sent, confirming that this user is muted. The feature that I am suggesting is deleting the message after a minute or so.

What is your justification for the proposed feature?

Topical channels that arenโ€™t necessarily used very often may have periods of empty messages, and when a mute message is shown it makes the channel look untidy and messy.

odd spireBOT
regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] New branch created: help\-channel\-rewrite
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
Xithrius
[python-discord/sir-lancebot] New branch created: Diabolical5777/main
#
Xithrius
[python-discord/sir-lancebot] branch deleted: Diabolical5777/main
#
Xithrius
[python-discord/sir-lancebot] branch deleted: mentions\-fix
#
Xithrius
[python-discord/sir-lancebot] branch deleted: fix\-928
#
Xithrius
[python-discord/sir-lancebot] branch deleted: sentry\_aiohttp
#
Xithrius
[python-discord/sir-lancebot] branch deleted: sebastiaan/backend/apply\-proper\-check\-name
#
Xithrius
[python-discord/sir-lancebot] branch deleted: sebastiaan/backend/test\-ci\-linting
#
Xithrius
[python-discord/sir-lancebot] branch deleted: redis\-persist
#
Xithrius
[python-discord/sir-lancebot] branch deleted: change\-permissions
#
Xithrius
[python-discord/sir-lancebot] branch deleted: issue\-command\-hotfix
#
Xithrius
[python-discord/sir-lancebot] branch deleted: latex\-limits
umbral grove
#

Context: branches that haven't been updated for 2 years.

regal archBOT
#
Xithrius
[python-discord/bot] branch deleted: feat/1169/codespaces
#
Xithrius
[python-discord/bot] branch deleted: mfa\-token\-filter
regal archBOT
#
ChrisLovering
[python-discord/modmail-plugins] branch deleted: test
#
ChrisLovering
[python-discord/modmail-plugins] branch deleted: kick\-plugin
#
ChrisLovering
[python-discord/modmail-plugins] branch deleted: ChrisLovering\-patch\-1
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] branch deleted: bug/2299/timeit
#
ChrisLovering
[bot:main] 7 new commits

8bd4eac set rules command's help upon cog_load - Amrou Bellalouna
fb13976 reinstate the docstrings of "rules" and update ... - Amrou Bellalouna
57053b8 remove Information Cog loading log - Amrou Bellalouna
4a72d92 remove the "nยฐ " from rules help - Amrou Bellalouna
c087ebd remove blank line between different rule keywords - Amrou Bellalouna

odd spireBOT
regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
#
wookie184
[python-discord/sir-lancebot] Pull request review submitted: #1095 Use buttons for bookmark command\.

I get this error clicking "Recieve Bookmark"

2022-11-02 18:09:50 | discord.ui.view | ERROR | Ignoring exception in view  for item <Button style= url=None disabled=False label='Receive Bookmark' emoji=None row=None>
Traceback (most recent call last):
  File "sir-lancebot-i1MlkyY3-py3.10\lib\site-packages\discord\ui\view.py", line 425, in _scheduled_task
    await item.callback(interaction)
  File "sir-lancebot\bot\exts\utilities\bookmark.py", line 56, in button_callback
    if i...
#
ChrisLovering
[python-discord/sir-lancebot] New comment on pull request #1095: Use buttons for bookmark command\.

I get this error clicking "Recieve Bookmark"

2022-11-02 18:09:50 | discord.ui.view | ERROR | Ignoring exception in view <SendBookmark timeout=180.0 children=1> for item <Button style=<ButtonStyle.success: 3> url=None disabled=False label='Receive Bookmark' emoji=None row=None>
Traceback (most recent call last):
  File "sir-lancebot-i1MlkyY3-py3.10\lib\site-packages\discord\ui\view.py", line 425, in _scheduled_task
    await item.callback(interaction)
  File "...
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
wookie184
[python-discord/bot] New comment on issue #2270: Allow evaluating code via a context menu?

At the moment I'm not really convinced this is worth having, as I'm not sure what benefit this adds over just using the command.

It seems potentially confusing if it only worked on your own messages, but also I think allowing people to run it on other's code will often result in errors (when using the eval command the code needs to be completely standalone, often code people post wouldn't be).

regal archBOT
regal archBOT
regal archBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
jchristgit
[python-discord/site] branch deleted: bot\-tags
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/bot-core] New branch created: prepare\-for\-pypi\-release
#
ChrisLovering
[python-discord/bot-core] branch deleted: prepare\-for\-pypi\-release
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
wookie184
[python-discord/bot] branch deleted: 2306\-nominations\-cleanup
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
ChrisLovering
[snekbox:main] 5 new commits

eba537a upgrade actions/checkout to solve deprecation w... - onerandomusername
483eda3 update all gha packages - onerandomusername
a42aed2 downgrade docker/setup-buildx-action to a valid... - onerandomusername
eb4dd4c remove the unnecessary and possibly invalid kub... - onerandomusername
ffd073a Merge pull request #156 from onerandomusername/... - ChrisLovering

regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
shtlrs
[python-discord/bot] Pull request opened: #2319 2304 link previous nomination threads

Note to core devs:
This PR needs to be merged before this one

Closes #2304

As requested by @bast0006, we want to be able to go back easily in time and look for all the nomination threads of a particular user.

This currently links all the previous threads (as long as they haven't been deleted) to the nomination history of a particular user.
It also lists the thread ids in the .md file when getting a review of a user.

**Pre...

odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot-core] New branch created: add\-lock\-utils
regal archBOT
regal archBOT
#
wookie184
[python-discord/site] New comment on pull request #792: Add "thread\_id" as a column in the nomination table

However, I wanted your opinion on when should we allow editing the thread_id of a nomination. And my guess is, it should follow the rest, e.g.:

1. It shouldn't be allowed when we're ending a nomination

2. It shouldn't be allowed on an inactive nomination

Let me know if I missed something

Hmm, I hadn't thought about validation. I don't really mind and don't think it matters too much, although those two things sound good for now. We can always make it mor...

odd spireBOT
regal archBOT
regal archBOT
#
Preocts
[python-discord/sir-lancebot] Issue opened: #1142 Enhance spoopy reactions for Halloween

Description

I would propose increasing the pattern match for "spooky" in the Halloween reacts to include such fun alternatives as "spoopy", "spookier", "spoopiest", and such.

Reasoning

Following the spirit of the best season; "spoopy" is time honored meme'ry, ripe historic use. It seems a shame to ignore such valuable potential and, indeed, causes me to feel a pang of sadness when a happy little ghost fails to appear whilst I am being spoopy.

![image](https://user-images.git...

odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
#
discretegames
[python-discord/site] Pull request opened: #794 Proofread, reorganized, and added to all the "Python on Windows" guides

As [discussed in #dev-contrib](#dev-contrib message) I wanted to write a guide for the site on how to add Python to Path since it comes up so often in help channels. wookie already wrote a lot of good stuff on that and other Windows matters (see https://github.com/python-discord/site/pull/773) which here I expanded on and added lots of instructional screenshots.

**The changes I've made are as follows (they're all on the `add...

odd spireBOT
regal archBOT
#
shtlrs
[python-discord/site] New review comment on pull request #794: Proofread, reorganized, and added to all the "Python on Windows" guides

We can add a section here that helps people identify the currently active virtual environment in PyCharm.
If you look at the bottom right, you will see your project root's name between parenthesis along with the python version like this.
image

If you click on that, you can see the full path to your virtual env here
![image](https://user-images.githubusercontent.com/48383734/20061847...

regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
thegamecracks
[python-discord/bot] Pull request opened: #2320 Update intents tag to better reflect discord\.py's API changes

To resolve #2313, this PR adds a short disclaimer near the top of the intents tag indicating that discord.py v2.0.0 requires its users to specify intents.
The starting paragraphs have also been revised to provide a clearer distinction between Discord's standard and privileged intents.
The code snippet now demonstrates the message_content intent since it is (generally) a requirement when writing prefix commands, especially with commands.Bot.

odd spireBOT
regal archBOT
regal archBOT
#
wookie184
[python-discord/.github] New branch created: wookie184\-add\-shtlrs
#
thegamecracks
[python-discord/bot] Pull request opened: #2321 Update discord\.py tags to refer to the stable version of the documentation

Currently the latest version of discord.py's documentation is referenced, which is being synchronized to the development branch of discord.py (i.e. master branch). Their branch may introduce changes to the documentation that are irrelevant / inaccessible to users that installed discord.py from PyPI. To avoid potential confusion with those links, this PR changes them to use the stable version of the docs instead.

This also fixes the link to BucketType in the [customcooldown](https://g...

odd spireBOT
regal archBOT
odd spireBOT
oak estuaryBOT
regal archBOT
#
HassanAbouelela
[python-discord/bot-core] New branch created: fix\-attributes
#
ChrisLovering
[python-discord/.github] branch deleted: wookie184\-add\-shtlrs
regal archBOT
#
shtlrs
[python-discord/bot] New review comment on pull request #2320: Update intents tag to better reflect discord\.py's API changes

We don't have access to the standard intents in the developer portal. What do you think about mentioning that ? We'd want to have people ask the question "Why don't I see these standard intents, am I doing something wrong ?"

We could say the the ones we can define in the portal are the privileged ones.
And maybe, if we want, we could add the link to a detailed technical list of these intents.

#
ChrisLovering
[python-discord/bot-core] branch deleted: fix\-attributes
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
shtlrs
[python-discord/bot] New comment on issue #2297: Add a \`doesntwork\` or similar tag

I think this would be quite similar to a recent tag we had about how to ask a question.

The problem was it became somewhat toxic and people started spamming it.

We have alot of new joiners in our community, so this is a scenario that we can expect quite often.

So it would be history repeating itself IMO.

I think it's better if we leave potential helpers ask the question themselves, it would also help us to explicitly identify people that show '' hostility '' towards beginners.

regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
shtlrs
[python-discord/site] New comment on pull request #789: Messages in past n days endpoint

I guess we could have the user ids as comma-separated-values, but we'd be running the risk of having too long query strings :/
I think it's fine to have this as a POST endpoint, even though it "sounds" wrong.

The only way to make this a GET would be to fetch the data of one user, but it would basically be a waste of bandwidth.

I'm saying this because I don't know what was discussed in #dev-core, so just shooting out ideas.

As for

#
Keyacom
[python-discord/bot] Pull request opened: #2322 Edited several tags
  • environments.md:
    • Added envs alias
    • Moved Python Environments to the embed's title field
    • Unwrapped sys.executable since it's a REPL example
  • identity.md: Converted the example with lists to a snippet from a REPL session.
  • inline.md: Added info about including a backtick in an inline code snippet.
  • listcomps.md: if in list comprehensions is an expression, not a statement, so I changed that.
  • local-file.md: Short explanation of rb, changed modal verb i...
odd spireBOT
regal archBOT
#
shtlrs
[python-discord/site] New comment on pull request #789: Messages in past n days endpoint

Thoughts on changing this to return 0 for users with no messages?

I didn't do it originally because it would also return 0 for invalid IDs which could potentially be misleading, but now I'm thinking it would probably be more useful if it just returned 0 for the ID if nothing was found.

I think it would be a good idea to return 0 for users with no messages.

Currently, if at least one id is invalid, the request won't be successful and will return with the encountered errors.

...

regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
shtlrs
[python-discord/bot] Pull request opened: #2323 Create an admin task when votes are 2 weeks old

closes #2226

This is still a draft since some details regarding GitHub helpers need haven't been discussed thoroughly.

The idea is to fetch active nominations that are more than 2 weeks old, and create admin issues for them in Github so that staff will do the follow up.

As soon as a vote is tracked, we add a "๐Ÿšฉ" reaction (which will probably change) to it to make sure we don't create duplicate issues for it.

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
wookie184
[python-discord/site] New comment on pull request #789: Messages in past n days endpoint

I was referring to the case where we can't tell if the ID is correct or not. For cases where it's not an integer or is negative the API user has clearly done something wrong so I think it makes sense to return an error response.

The issue is that with the current query we can't tell between users with no messages in that period and users that don't exist (but with the ID in the correct form). I think i'll change it so that does return 0 anyway though, as it's probably more useful.

#
shtlrs
[python-discord/site] New comment on pull request #789: Messages in past n days endpoint

I was referring to the case where we can't tell if the ID is correct or not. For cases where it's not an integer or is negative the API user has clearly done something wrong so I think it makes sense to return an error response.

The issue is that with the current query we can't tell between users with no messages in that period and users that don't exist (but with the ID in the correct form). I think i'll change it so that does return 0 anyway though, as it's probably more useful.

...

regal archBOT
#
HassanAbouelela
[python-discord/bot] New comment on pull request #2323: Create an admin task when votes are 2 weeks old

Fwiw, you can target one PR to another branch (i.e it would look something like main <- nomination history PR branch <- this PR branch).

What that would do is basically try applying the changes from the final branch (3) in this chain to the middle branch (2), so when you check out the 3rd branch, it would have all the changes from branch 2 as well, but the PR itself will only have the changes of branch 3.

For the admin repo, there's an issue template which Chris can probably fetch for...

regal archBOT
odd spireBOT
regal archBOT
oak estuaryBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
discretegames
[python-discord/site] New review comment on pull request #794: Proofread, reorganized, and added to all the "Python on Windows" guides

Hmm, I see a python.exe in my C:\Users\<user>\AppData\Local\Microsoft\WindowsApps. When the store version is installed that exe runs python, and there's a pip.exe in there too, but when it's not installed it opens the store - aka it is the app execution alias. So I've been assuming it gets looked up on the path normally. If you toggle off the app execution alias that python.exe is removed.

So I think it's ok. I double checked and with `C:\Users<user>\AppData\Local\Microsoft\WindowsA...

#
discretegames
[python-discord/site] New review comment on pull request #794: Proofread, reorganized, and added to all the "Python on Windows" guides

I'd prefer to mention it. Lots of times I see tutorials that don't mention the software versions involved or date of writing and I have to guess if it'll work for my setup.

Mentioning Win 11 might help people from clicking away thinking "oh this must be a Win 10 guide since Win 11 is new".

(Though btw I have not explicitly tested the guides in Win 11 since I'm still on Win 10.)

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
discretegames
[python-discord/site] New review comment on pull request #794: Proofread, reorganized, and added to all the "Python on Windows" guides

I think it's important to mention the "Manage app execution alias" setting somewhere - I didn't even know about it until writing the guides. (Though it is mentioned in /microsoft-store/#confusing-app-execution-alias-behaviour come to think of it.)

Whether to suggest it or not I have no opinion on. Though I believe it does work more or less like a normal path entry as explained here: https://github.com/python-discord/site/pull/794#discussion_r1020826357

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
discretegames
[python-discord/site] New comment on pull request #794: Proofread, reorganized, and added to all the "Python on Windows" guides

One thing I wanted to be sure to resolve is whether or not the yellow downloads button on https://www.python.org/downloads/ really does download the best version the suits the computer, since I said it did in the "Recommended Installed" tutorial:

Go to python.org/downloads and click the big yellow "Download Python 3.x.x" button near the top of the page. That should start the download of the latest Windows Python installer that best suits your computer....

#
Anonymous4045
[python-discord/sir-lancebot] Issue opened: #1145 Show previous conversation prompts after \.topic command is reacted to

Description

After a member invokes the "topic" command, they have the ability to re-roll the embed to choose a different conversation prompt. When they do this, the previous contents of the message is overwritten. Instead, the content should be edited to include the new prompt with the old prompt(s) above it, with a number corresponding to the order it was created in.

Reasoning

When the person who invokes the command sees a topic they don't want to comment on, they tend to re-r...

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
Xithrius
[python-discord/site] New branch created: maskduck\-another\-random/main
#
Xithrius
[python-discord/site] branch deleted: maskduck\-another\-random/main
#
Xithrius
[python-discord/site] New branch created: fix\-verification\-ssl
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
Xithrius
[python-discord/site] branch deleted: fix\-verification\-ssl
odd spireBOT
regal archBOT
regal archBOT
#
shtlrs
[python-discord/site] New review comment on pull request #733: Added discord\-app\-commands\.md 
Upon resumption of the most popular discord API wrapper library for python, `discord.py`, while catching on to the latest features of the discord API, there have been numerous changes with addition of features to the library. Some additions to the library are -> Buttons support, Select Menus Support, Forms (AKA Modals), Slash Commands (AKA Application Commands) and a bunch of more handy features! All the changes can be found [here](https://discordpy.readthedocs.io/en/latest/mig...
regal archBOT
#
ChrisLovering
[python-discord/bot] New review comment on pull request #2316: Add safeguard when banning staff

AH, I see what is going on. Discord.py usually deals with the response to the callback automatically, but since you're calling self.timeout() manually, that s stopping the interaction, so Discord.y does not respond to Discord to say the interaction was handled.

I think what we want to do here is

await interaction.message.edit(view=None)
self.stop()

At the end of both button callbacks You could even edit the embed at the same time for feedback to the person who interacted...

odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot-core] New branch created: d\.py\-2\.1
oak estuaryBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] New branch created: snekbox\-3\.11\-default
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT