#dev-log

1 messages · Page 99 of 1

regal archBOT
#
ChrisLovering
[python-discord/bot] branch deleted: fix\-trigger\_typing
oak estuaryBOT
odd spireBOT
regal archBOT
#
HassanAbouelela
[python-discord/forms-frontend] branch deleted: make\-code\-selectable
regal archBOT
#
Bluenix2
[python-discord/site] New branch created: cj9\-qualifier
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] New branch created: limit\-ext\-cog\-to\-1\-actionn\-at\-a\-time
#
ChrisLovering
[python-discord/bot] Pull request opened: #2205 Limit the ext cog to 1 action at a time

Now that cog loading actually has lots of logic in it, rather than spawning tasks to do the actual load work, loading cogs can take time.

Previously we gave no feedback to the caller that something was actually happening. This could mean the caller would try to run the command again, possibly getting the cogs into a bad state.

This change both enforces only 1 action at a time, and gives the caller feedback that their action is in progress.

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
Robin5605
[python-discord/site] New review comment on pull request #733: Added discord\-app\-commands\.md

We also want to explain why we're binding it to self

- Next up `super().__init__(...)` runs the `__init__` function of the **Client** class, this is equivalent to `discord.Client(...)`. Then, `self.tree = discord.app_commands.CommandTree(self)` creates a CommandTree which acts as the container for slash commands, and binds it to the `discord.Client` subclass instance, so wherever you have access to it, you will also have access to the command tree.
regal archBOT
#
sentry-io[bot]
[python-discord/bot] Issue opened: #2206 ResponseCodeError: Status: 400 Response: \{'deletedmessage\_set': \[\{'embeds': \{'0': \["Tag embed must contain one of th\.\.\.

Sentry Issue: BOT-33V

ResponseCodeError: Status: 400 Response: {'deletedmessage_set': [{'embeds': {'0': ["Tag embed must contain one of the fields {'title', 'description', 'video', 'image', 'fields'}."]}}, {'embeds': {'0': ["Tag embed must contain one of the fields {'title', 'description', 'video', 'image', 'fields'}."]}}, {'embeds': {'0': ["Tag embed must contain one of the fields {'title', ...
#
ChrisLovering
[python-discord/site] New branch created: embed\-required\-fields
#
sentry-io[bot]
[python-discord/site] Issue opened: #736 ResponseCodeError: Status: 400 Response: \{'deletedmessage\_set': \[\{'embeds': \{'0': \["Tag embed must contain one of th\.\.\.

Sentry Issue: BOT-33V

ResponseCodeError: Status: 400 Response: {'deletedmessage_set': [{'embeds': {'0': ["Tag embed must contain one of the fields {'title', 'description', 'video', 'image', 'fields'}."]}}, {'embeds': {'0': ["Tag embed must contain one of the fields {'title', 'description', 'video', 'image', 'fields'}."]}}, {'embeds': {'0': ["Tag embed must contain one of the fields {'title', ...
#
ChrisLovering
[python-discord/site] branch deleted: cj9\-qualifier
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
dawnofmidnight
[python-discord/snekbox] New branch created: dawnofmidnight/typing\-extensions
#
ChrisLovering
[python-discord/snekbox] branch deleted: dawnofmidnight/typing\-extensions
odd spireBOT
regal archBOT
#
ChrisLovering
[bot:main] 7 new commits

adac8e8 Always prepend guild name to guild invite filters. - ChrisLovering
19f67db Include the list type when telling mods a new a... - ChrisLovering
111db69 Don't ping mods for autoban filters - ChrisLovering
af4339e Improved output format of guild autoban notific... - ChrisLovering
11f329f Rename filtering send log kwarg for consistency... - ChrisLovering

#
ChrisLovering
[python-discord/bot] branch deleted: always\-prepend\-guild\-name\-to\-guild\-invite\-filters
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot-core] New branch created: dont\-connect\-statsd\-if\-none
#
ChrisLovering
[python-discord/bot-core] branch deleted: misc\-bug\-fixes
regal archBOT
#
ChrisLovering
[python-discord/bot-core] branch deleted: dont\-connect\-statsd\-if\-none
regal archBOT
#
D0rs4n
[python-discord/sir-robin] New branch created: cj\-add\-role\-creation
#
HassanAbouelela
[python-discord/forms-frontend] branch deleted: snekbox\-results\-feedback
#
HassanAbouelela
[python-discord/forms-frontend] Pull request opened: #474 Display Test Failures & Refactor Formpage

This PR includes a few things:

  • An update to the error message component (see 34d57f2). This is mostly standalone, but it's useful for another change
  • Unittest failures now display the reason of failure. This was included in this PR, as the restructuring of formpage made it easier to accomplish
  • Refactoring of formpage. This is where the bulk of the changes are, but it's mostly just a straight copy-paste of old code into new files. It does contain a few changes - which should hopefully ...
regal archBOT
#
HassanAbouelela
[python-discord/forms-frontend] New branch created: cleanup\-codeowners
#
HassanAbouelela
[python-discord/forms-frontend] branch deleted: cleanup\-codeowners
regal archBOT
#
HassanAbouelela
[python-discord/forms-frontend] New review comment on pull request #474: Display Test Failures & Refactor Formpage

It's always a string right now, but there's no way to legally assert that unless you add a type check.

Since we're setting the error variable based on this, another check will need to be added to handle the case where the error variable was not set. This never happens though, so it would be a bunch of fun error handling for an impossibility.

The larger issue here is that state is a map with no hard defined rules on type. The same is true of many things around the codebase, such as setting t...

#
HassanAbouelela
[python-discord/forms-frontend] New review comment on pull request #474: Display Test Failures & Refactor Formpage

That's one option, but mainly this is going to be important once we're switching between the sending state, and some other state. Some state change is going to be necessary.

Validation error can be merged into the initial state, but I set it to its own type here in case it proves useful for something else (such as moving the validation logic, or rewriting the scroll logic).

regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
HassanAbouelela
[python-discord/forms-frontend] branch deleted: display\-test\-failures
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
LiquidPulsar
[python-discord/modmail-plugins] New review comment on pull request #25: Snippet search

Is the strip necessary here? Aliases shouldn't differ by leading/trailing whitespace unless someone intentionally modifies them, and you seem to subscribe to that idea given the content = snippets[name] which grabs an unstripped version (ie if the differences mattered we would be stripping that one too).

Working with that added strip yields:

    names_by_content = defaultdict(set)
    for name, content in snippets.items():
        names_by_content[content.strip()].add(name)
...
#
LiquidPulsar
[python-discord/modmail-plugins] New review comment on pull request #25: Snippet search

Looks like the enumerate is to ensure tuple sorting is done properly in the case of equal scores, idk how much it matters particularly tho

Could replace 89-99 with:

        for i, (names, content) in enumerate(grouped_snippets):
            group_score = max(score(query, name, content) for name in names)
            #or score(query, names, content) if modifying score

            if group_score >= THRESHOLD: #saves sorting time?
                scored_groups.append((group_scor...
#
LiquidPulsar
[python-discord/modmail-plugins] New review comment on pull request #25: Snippet search

The common_word_count function is being called multiple times redundantly. Perhaps modify to take a list of names: typing.Iterable[str], eg

return (
    max(
        common_word_count(query, name)
        for name in names
    )
+ common_word_count(query, content)
) / len(words(query))

If we're being even more pedantic, would try to minimise calls to words but i guess we don't necessarily care about the efficiency that much. I just kept this as the heuristic functi...

regal archBOT
#
kosayoda
[python-discord/site] branch deleted: cj9\-frameworks\-reorder
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/forms-frontend] New branch created: update\-refresh\-timeout\-logic
odd spireBOT
regal archBOT
#
HassanAbouelela
[python-discord/forms-frontend] branch deleted: update\-refresh\-timeout\-logic
odd spireBOT
regal archBOT
#
D0rs4n
[python-discord/sir-robin] New branch created: pr\-integrate\-cj\-mgmt
#
D0rs4n
[python-discord/sir-robin] branch deleted: cj\-add\-role\-creation
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
wookie184
[site:main] 4 new commits

d4f717e Remove embed validators for deleted messages - ChrisLovering
084357d Remove embed validators from old migrations - ChrisLovering
9d312d0 Merge branch 'main' into embed-required-fields - ChrisLovering
cf4695a Merge pull request #737 from python-discord/emb... - wookie184

#
wookie184
[python-discord/site] branch deleted: embed\-required\-fields
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
regal archBOT
#
shenanigansd
[python-discord/bot] New branch created: shenanigansd\-patch\-1
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] Pull request review submitted: #2207 Capitalize the "D" in "Discord"

This PR is a masterpiece. There is no praise in any language that can describe the excellence of this PR and the importance of it to be included in the human history. This by itself has started a new era, An Era of Utopias ; A Golden Age. This PR is truly historical and has inspired me to drop all my professions, talents and job to observe this masterpiece repeatedly. The sheer knowledge of this PR has prevented my inevitable demise and has kept me from decaying out of the fabric of reality. ...

#
ChrisLovering
[python-discord/bot] branch deleted: shenanigansd\-patch\-1
oak estuaryBOT
odd spireBOT
regal archBOT
regal archBOT
#
MarkKoz
[python-discord/snekbox] Pull request opened: #147 Create v1 cgroups as needed

Resolves #101

NsJail puts subprocess cgroups under parent cgroups. However, it does not create the parents automatically. Thus, snekbox has to do this. Previously, for v1, only the memory and pids cgroup parents were being created. Now, all four controllers supported by NsJail are also supported by snekbox.

It looks at the NsJail config to determine if a cgroup parent needs to be created. Basically, if a value is non-default, then NsJail will create a cgroup and thus will rely on a ...

regal archBOT
regal archBOT
regal archBOT
#
camcaswell
[python-discord/sir-robin] New branch created: redis\-debug
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
D0rs4n
[python-discord/sir-robin] New branch created: feat\-integrate\-code\-jam\-management
odd spireBOT
regal archBOT
#
D0rs4n
[python-discord/sir-robin] Pull request opened: #49 Feat integrate code jam management

Okay, so Code Jam Management has been integrated into Sir Robin's CJ extension..but What changed?

First and foremost.. Roles!

I have added back team roles, and have updated the flows (later on those) accordingly!

Flows:

Creation flow:

So, cj create behaves a tiny bit differently. From now on, when you issue the command with a csv, you'll be sent a prompt to confirm, whether we actually want all that.. 😎 , you'll see how many channels, and roles will be created. Upon confir...

#
D0rs4n
[sir-robin] Branch feat\-integrate\-code\-jam\-management was force-pushed to `f4ba2f1`

Compare changes

regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
janine9vn
[python-discord/sir-lancebot] Issue opened: #1069 Add message replies support for \`\.uwu\`

Description

I think it would be great that if you replied to a message with .uwu and no other text, that lancebot would uwu-ify the replied-to message. Alternatively, add support for .uwu in a context menu!

Reasoning

We must allow .uwu in all its forms. But I think it would be a fun feature to implement and a good way to get familiar with our code base.

Proposed Implementation

If there is no message content, check if there is a message that has been replied to, and if ...

regal archBOT
regal archBOT
regal archBOT
#
janine9vn
[python-discord/bot] New branch created: tag\-dashmpip
regal archBOT
#
janine9vn
[python-discord/bot] Pull request opened: #2209 Add tag to use \`python \-m pip\` to install packages

In the wild west of #python-general, we see a lot of issues where folks will use pip to install a package. Unfortunately, the pip they're using to install the package is not in the same version of Python that's used to actually run their code (we're going to specifically not talk about cases where a venv is created automatically for them and they don't realize for this PR and tag). This could be for a variety of reasons and will often lead down a rabbit hole of altering what's on PATH. ...

odd spireBOT
regal archBOT
#
D0rs4n
[python-discord/bot] branch deleted: tag\-dashmpip
oak estuaryBOT
odd spireBOT
regal archBOT
#
shenanigansd
[python-discord/sir-lancebot] New branch created: gh1069
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/metricity] branch deleted: remove\-opt\-out
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/sir-lancebot] branch deleted: gh1069
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/bot-core] New branch created: fix\-stats\-close
#
ChrisLovering
[python-discord/bot-core] branch deleted: fix\-stats\-close
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
oak estuaryBOT
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/metricity] New branch created: botcore\-migration
#
MarkKoz
[python-discord/snekbox] branch deleted: feat/nsjail/101/create\-all\-cgroups
#
Xithrius
[python-discord/sir-lancebot] New branch created: fix\-928
regal archBOT
#
HassanAbouelela
[python-discord/forms-backend] New branch created: verify\-webhook\-user\-exists
odd spireBOT
regal archBOT
#
HassanAbouelela
[python-discord/forms-backend] branch deleted: verify\-webhook\-user\-exists
odd spireBOT
regal archBOT
regal archBOT
#
Xithrius
[python-discord/sir-lancebot] Pull request opened: #1071 Avoidance of \`KeyError\` from \`"draft"\` key when pulling GitHub PR data\.

Closes #928

Moved FetchError handling to fetch_data function, so no placeholder variables in returns need to be used.

FetchError is no longer a data class, but an actual error which inherets from Exception.

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](h...
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/metricity] branch deleted: botcore\-migration
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/metricity] New branch created: dependabot
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
mbaruh
[python-discord/site] New branch created: more\_frameworks
odd spireBOT
regal archBOT
regal archBOT
#
kosayoda
[python-discord/site] branch deleted: more\_frameworks
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
Xithrius
[python-discord/bot] branch deleted: limit\-ext\-cog\-to\-1\-actionn\-at\-a\-time
oak estuaryBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
janine9vn
[python-discord/bot] Issue opened: #2211 Features we need before we can trial threads in topical channels

Description

We'd love to trial people having the ability to create threads within topical channels to have more focused and less disruptive help sessions.

In some of the busier topical channels, you'll often have multiple help chats going on at the same time. This can make it overall difficult to follow, a bit harder to help. It also may not be feasible to shift to a help channel because some of the help may be back and forth over a few hours, depending on availability of the people an...

#
ChrisLovering
[python-discord/metricity] New branch created: update\-staff\-channel\-logic
#
ChrisLovering
[python-discord/bot] New comment on issue #2211: Features we need before we can trial threads in topical channels

Ensure reasonable default timeouts for threads, and once archived can only be brought back by the thread creator or a staff member/mod
Make sure bringing back a thread also respects the cooldown, re-archiving it if the user is currently on cooldown.

On question I have right now is what wee would expect to happen if the name is found in violation of our filters. Maybe just hard-deleting and notifying mods?

regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
MarkKoz
[python-discord/site] branch deleted: swfarnsworth/resources
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
HassanAbouelela
[python-discord/site] New comment on pull request #740: Update Django Version To 4\.0

Background on 7b40cd8

I've worked on this for a while, so here's a bit more information on why that part of the test suite was dropped. After some debugging, The custom setup for migration tests instantiates a few model instances. Going from django 3.1.14 (what we were on) to 3.2.0 (the version directly after) introduces behavior that breaks this. At some point between the setup and the actual test the (already created) instances' _meta model gets updated to the latest model, whi...

regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
SebastiaanZ
[python-discord/site] New comment on pull request #740: Update Django Version To 4\.0

I'm hoping Sebastiaan could provide some context on why it was added in the first place. If it's still relevant, I'll work on reintroducing it.

It was introduced because we were writing code that would directly affect import production data, namely a data migration that would directly touch all of our infraction records.

This allowed is to test the code that we were writing.

These days, there are a few packages available that basically implement this functionality without having to do it...

regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
D0rs4n
[sir-robin] Branch feat\-integrate\-code\-jam\-management was force-pushed to `c13935f`

Compare changes

regal archBOT
#
ks129
[sir-robin:main] 18 new commits

05c88f9 Add role creation to the Code Jam creation flow - D0rs4n
6184740 Integrate Code Jam Management into CJ Create, a... - D0rs4n
481cd22 Make compliant and port the code jam tests from... - D0rs4n
515ab15 Integrate Roles, and Code Jam Mgmt into the CJ ... - D0rs4n
ea9de19 Add relevant test for bulk delete of channels, ... - D0rs4n

#
ks129
[python-discord/sir-robin] branch deleted: feat\-integrate\-code\-jam\-management
thin oysterBOT
#
Sir Robin

Connected!

regal archBOT
#
HassanAbouelela
[python-discord/metricity] branch deleted: update\-staff\-channel\-logic
regal archBOT
#
janine9vn
[python-discord/bot] New branch created: tag\-aliases
regal archBOT
#
janine9vn
[python-discord/bot] Pull request opened: #2213 Add Aliases for Tags Support

This PR adds support for adding aliases and alternative names for tags and showcases it with two existing tags.

It uses the metadata functionality and will look for the "aliases" key. It then adds the alias names to the Tags dictionary that gets searched through when trying to match a tag directly.

This doesn't check for duplicate aliases as there wasn't a good way to do that validation ahead of time. It also didn't feel like a good solution to iterate through all TagIdentifiers to find...

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
MarkKoz
[python-discord/snekbox] Issue opened: #148 API Documentation

I'm not happy with the current state of documentation for the API, as it requires looking through docstrings in the source code. I've been considering using an OpenAPI specification and generating a website with documentation using SwaggerUI or ReDoc.

Because snekbox is intended to be hosted by the users, I believe users would find the documentation more convenient and accessible if hosted on GitHub pages. The alternative would be to add a route to Falcon, but this would require users to b...

#
D0rs4n
[python-discord/sir-robin] New branch created: pr\-cj\-infr\-notes
#
D0rs4n
[python-discord/sir-robin] Pull request opened: #50 Add support for viewing and adding CJ Infractions notes
  • From now on the cj info command has a view attached to it. You can either view all the notes added to the specific participant, or you can add one. Adding a note is accomplished by triggering a modal from the view, which then calls the code jam management api on submission.A new kind of interaction check has been implemented for this view, since from now on, all of the admins can use the view regardless of the original author.
regal archBOT
#
ionite34
[python-discord/sir-lancebot] New comment on issue #1072: \.uwu command incorrectly transforms emojis in their raw text form

Couldn't we make use of the emojis library, that is already a dependency? alexandrevicenzi/emojis

Well, built-in emojis do work right now. And third-party emojis (such as those in python-discord) would work if the bot is in the server.

However third-party emojis wouldn't be able to be sent since SirLanceBot isn't in those servers, and as such doesn't have access to post those emojis. This is what this issue is for.

night lilyBOT
#
Sir Lancebot

Connected!

oak estuaryBOT
thin oysterBOT
#
Sir Robin

Connected!

night lilyBOT
#
Sir Lancebot

Connected!

oak estuaryBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
HassanAbouelela
[python-discord/site] New comment on pull request #741: Run Tests On A Non\-Durable Database

While trying to merge both databases as per Mark's suggestion, I made an interesting discovery. At first, the merge seemed to slow the tests again, despite nothing having really changed. Upon closer inspection, the port I defined on the service, was slightly different from the one we currently have (127.0.0.1:port:port current vs port:port my change).

I can't explain it, and it's possible I have something messed up in my networking, but I can reproducibly demonstrate it makes an order ...

odd spireBOT
regal archBOT
#
MarkKoz
[python-discord/site] New review comment on pull request #741: Run Tests On A Non\-Durable Database

For future reference, there isn't really a point in changing this as its deprecated

https://github.com/compose-spec/compose-spec/blob/master/spec.md#version-top-level-element

A Compose implementation SHOULD NOT use this version to select an exact schema to validate the Compose file, but prefer the most recent schema at the time it has been designed.

Maybe there are still some tools that are not respecting this advice?

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
mbaruh
[python-discord/bot] New branch created: mbaruh/regex\-tag\-fix
odd spireBOT
regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
#
HassanAbouelela
[python-discord/site] Pull request opened: #742 Add GitHub Artifacts API Route

Description

This PR adds an API route, which uses a github app (not to be confused with OAuth apps) to return a download URL for the static preview artifact. This should allow us to get around the rate limiting issues that we've been experiencing in #728.

A bit more info on my approach:
Q. Why a GitHub app?
A. It provides the most granularity in terms of permissions. We can get very specific about ...

odd spireBOT
regal archBOT
#
shenanigansd
[python-discord/sir-lancebot] New branch created: type\-hint\-fixes
odd spireBOT
regal archBOT
regal archBOT
#
camcaswell
[python-discord/bot] New branch created: revival\-of\-code\-role
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] branch deleted: revival\-of\-code\-role
oak estuaryBOT
odd spireBOT
regal archBOT
#
shenanigansd
[python-discord/bot] Issue opened: #2216 Allowing reaction roles

Several popular bots have the capabilities for "reaction roles", where a bot will send a message in a channel, with one reaction per role, and when a user reacts with a reaction, they will be assigned the specified role.

Here a couple of examples:
https://dyno.gg/docs/en/modules/reactionroles
https://mee6.xyz/tutorials/allow-users-to-self-assign-roles-in-discord-with-the-mee6-bot-reaction-roles

Replacing !subscribe with reaction roles will greatly reduce the amount of spam and confu...

regal archBOT
regal archBOT
#
TizzySaurus
[python-discord/sir-lancebot] Issue opened: #1074 Use replied message as argument to \`\.uwu\` when text isn't provided\.

Description

Allow .uwu to work on replied messages whenever no text argument is provided.

Reasoning

Pretty self-explanatory; means you can just reply to a message instead of having to manually copy-paste the content as the command argument.

Proposed Implementation

A simple if statement checking whether a text argument was provided, and if not whether a message was replied to.

Would you like to implement this yourself?

  • [x] I'd like to implement this fea...
regal archBOT
#
TizzySaurus
[python-discord/bot-core] Issue opened: #100 Add a \`CleanedTextOrReply\` converter

As per sir-lancebot#1074, we should add a CleanedTextOrReply converter which will allow an argument to be provided either by text, or by replying to a message (if an invocation has both then use the text).

There's currently no file for converters, so we'd need a new file -- either botcore/converters.py as per bot structure or botcore/utils/converters.py as per sir-lancebot structure.

I'd like to implement this myself.

regal archBOT
#
shenanigansd
[python-discord/sir-lancebot] New branch created: gh1076
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ionite34
[python-discord/sir-lancebot] Pull request opened: #1078 Fix: \.uwu incorrectly transforms emojis in their raw text form

Relevant Issues

Fixes #1072

Description

6db84b38568d3f53c1b1bec4c66c1c8f2ee7b1ac

  • Added pattern RE_EMOJI to capture emoji markdowns
  • Implemented Emoji data class for emoji information and displayability inferencing
  • Added function _ext_emoji_replace to replace external undisplayable emojis with random emoticons
  • Added call to _ext_emoji_replace at the end of _uwuify

Fixes / formatting

202d53b3f7ebe4d6041a2e5a4c1419bc16b27e11

  • Added type checking annotatio...
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
thin oysterBOT
#
Sir Robin

Connected!

odd spireBOT
odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] New branch created: 3\.11\-snekbox
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
Preocts
[python-discord/bot] New comment on issue #2217: Infraction display durations off\-by\-one

Related to: https://github.com/python-discord/bot/issues/2130

To echo @mbaruh in the other issue, I'm also in favor of removing the redundancy here. It would simplify the overall process to use the discord timestamp. There might be an edge case for having duration (in N hours/days) for visual cues on the rare cases !mute ... 15M becomes !mute ... 15m. Beyond that, the timestamp rendered in Discord serves the purpose well.

#
Bluenix2
[python-discord/site] New branch created: cj9\-changes
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] New comment on pull request #2218: Allow snekbox to run in both 3\.10 and 3\.11

Being able to run in 3.11 sounds good.

I'm not sure that the button to change versions is worth the added clutter given that in the vast majority of cases of eval commands the version wouldn't make a difference.

Was having a separate command like !eval3.10, or an optional argument like !eval 3.10 considered?

This PR does add optional argument to the commands too.

The thoughts behind the button is if the user runs into a 3.11 problem, they can easily switch to 3.10 an...

regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
TizzySaurus
[python-discord/bot] New branch created: fix\-bot\-2168
#
wookie184
[python-discord/site] New review comment on pull request #743: Make finalizing codejam 9 changes

Pending discussion on the server, this wording feels a little clumsy. Would this be better?

            An exception to this rule is that communication with databases and files is allowed for accessing resources or for storage purposes. 
            For example, you may use PostgreSQL as a database but not its `NOTIFY` command.
            Working with subprocesses (through stdin/stdout or <code>multiprocessing.Pool()</code>/<code>concurrent.futures.ProcessPoolExecutor()</co...
odd spireBOT
regal archBOT
regal archBOT
#
TizzySaurus
[python-discord/bot-core] New branch created: bot\-core\-100
#
TizzySaurus
[python-discord/bot-core] Pull request opened: #101 Add \`clean\_text\_or\_reply\` util\.

Closes #100.

Does what the title says.

This util function will clean text if provided, else try and clean the referenced message, or if text isn't provided and was unable to get a reference message will return None.

This is useful for commands such as lancebot's .uwu and .randomcase where users can either provide a text argument, or reply to a message and have the bot use the replied message's content.

odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
Xithrius
[python-discord/sir-lancebot] branch deleted: type\-hint\-fixes
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
regal archBOT
#
HRLO77
[python-discord/bot] New comment on issue #2221: \!matchcase tag

It should have a short summary such as
"Match case is a 3.10+ feature that allows you to execute statements based on the result of an operation"
`resp_code = requests.get('https://pythondiscord.com')
match int(str(resp_code)[0]): # get the first letter of the response
case 2: # resp_code started with 2
print('Request was successful!')
case 3: # resp_code started with 3
print('You are being redirected.')
case 4: # resp_code started with 4
print('Cl...

regal archBOT
regal archBOT
oak estuaryBOT
regal archBOT
odd spireBOT
regal archBOT
oak estuaryBOT
regal archBOT
regal archBOT
#
wookie184
[python-discord/site] branch deleted: cj9\-changes
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
sid-maddy
[python-discord/bot] New comment on issue #2222: Add support for embedding images/videos in tag responses

Unfortunately, it seems like Discord doesn't support sending videos in embeds.
Trying to call the Embed.set_image with a video URL results in the embed just showing a loading animation.

Since the point of this feature was to add support for images and videos, we'll have to go with the other approach of adding a new command instead.

Would it be a good idea to rework the tags command to send a media lin...

regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot-core] New branch created: add\-interaction\-utils
regal archBOT
#
HassanAbouelela
[python-discord/bot-core] branch deleted: add\-interaction\-utils
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
MarkKoz
[python-discord/bot] New comment on pull request #2218: Allow snekbox to run in both 3\.10 and 3\.11

I get the following error when trying to use the delete button

2022-07-16 13:56:31 | discord.ui.view | ERROR | Ignoring exception in view <ViewWithUserAndRoleCheck timeout=180.0 children=2> for item <DeleteMessageButton style=<ButtonStyle.secondary: 2> url=None disabled=False label='Delete' emoji=None row=None>
Traceback (most recent call last):
  File "/home/mark/repos/python/bot-pydis/.venv/lib/python3.9/site-packages/discord/ui/view.py", line 412, in _scheduled_task
    await it...
#
ChrisLovering
[python-discord/bot-core] New branch created: fix\-delete\-button
#
ChrisLovering
[python-discord/bot] New comment on pull request #2218: Allow snekbox to run in both 3\.10 and 3\.11

I get the following error when trying to use the delete button

2022-07-16 13:56:31 | discord.ui.view | ERROR | Ignoring exception in view <ViewWithUserAndRoleCheck timeout=180.0 children=2> for item <DeleteMessageButton style=<ButtonStyle.secondary: 2> url=None disabled=False label='Delete' emoji=None row=None>
Traceback (most recent call last):
  File "/home/mark/repos/python/bot-pydis/.venv/lib/python3.9/site-packages/discord/ui/view.py", line 412, in _scheduled_task

...

#
MarkKoz
[python-discord/bot] New comment on pull request #2218: Allow snekbox to run in both 3\.10 and 3\.11

Error

2022-07-16 14:17:41 | botcore.utils.scheduling | ERROR | Error in task Task-674 139899212692800!
Traceback (most recent call last):
  File "/home/mark/repos/python/bot-pydis/bot/exts/utils/snekbox.py", line 466, in run_job
    response = await self.send_job(ctx, python_version, code, args=args, job_name=job_name)
  File "/home/mark/repos/python/bot-pydis/bot/utils/lock.py", line 112, in wrapper
    raise LockedResourceError(str(namespace), id_)
bot.errors.LockedResourceErr...
#
ChrisLovering
[python-discord/bot-core] branch deleted: fix\-delete\-button
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[bot:main] 8 new commits

3ad7cd9 Add required config and constants for snekbox 3.11 - ChrisLovering
234bd00 Allow users to eval code in either 3.11 or 3.10 - ChrisLovering
32d44d1 Update snekbox tests to reflect current behaviour - ChrisLovering
c4a7722 Use generic views from bot0core for snekbox - ChrisLovering
b20b72a Infer the snekbox invoker from context - ChrisLovering

oak estuaryBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/site] New branch created: snekbox\-311\-and\-metricity
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/bot-core] New branch created: remove\-view\-on\-timeout
#
ChrisLovering
[python-discord/bot-core] New tag created: v7\.4\.0\-beta1
#
sid-maddy
[python-discord/bot] New branch created: sid/feature/tags/print\-return
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] New branch created: snekbox\-snags
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
#
Bluenix2
[python-discord/site] New branch created: cj9\-changes
#
janine9vn
[python-discord/sir-robin] Pull request review submitted: #50 Add support for viewing and adding CJ Infractions notes

Works like a charm and I enjoy the usage of buttons to view and add notes!

Since we have a full CJ system up and running now, we should document the required .env vars for testing out the functionality, but that's very out of scope for this PR.

I think from a running-this-event perspective it might be nice to have the viewing of a note and the relay of a note being added to not be ephemeral. We often discuss notes and the like with other event team members, so it'll be helpful if eve...

#
Bluenix2
[python-discord/site] New branch created: cj9\-qualifier\-desc
odd spireBOT
regal archBOT
#
mbaruh
[python-discord/site] branch deleted: cj9\-qualifier\-desc
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] Issue opened: #2226 Create an admin task when votes are 2 weeks old

Description

We want a way to notify admins when helper votes are 2 weeks old, so that they can step in and get it over the line.

This should be done via a call to GitHub API to create an issue. The github org name, repo name, and the auth token should be constants to allow the to be different in dev & prod.

Suggested implementation

Have a discord.py task that runs daily to check if there are any votes that meet the stale threshold, if so create the issue and add an emoji to th...

regal archBOT
#
AbooMinister25
[python-discord/sir-lancebot] Pull request opened: #1080 Fix issue with \.uwu failing to uwuify embeds in replies

Relevant Issues

Closes #1075

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

Description

In uwu_command, if command was invoked referring to a replied message, store the message's embeds in an embeds variable, otherwise set it to None.
Add an else statement in uwu_command after if embed is not None to fallback to embeds which was defined earlier.
Add a check in uwu_command to avoid adding a leading >>> if the uwuifi...

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
D0rs4n
[sir-robin:main] 9 new commits

3aaa9cf Add support for viewing and adding CJ Infractio... - D0rs4n
01ff98c Enhance code consistency in the CJ extensions - D0rs4n
f6f2e22 Add a utility function to fetch a CJ user durin... - D0rs4n
4ec3d59 Limit CJ info view interactions to admins and i... - D0rs4n
b649b3c Abort Code Jam interaction upon category name c... - D0rs4n

#
D0rs4n
[python-discord/sir-robin] branch deleted: pr\-cj\-infr\-notes
thin oysterBOT
#
Sir Robin

Connected!

regal archBOT
regal archBOT