#dev-log

1 messages · Page 18 of 1

odd spireBOT
odd spireBOT
odd spireBOT
odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
#
jchristgit
[python-discord/site] branch deleted: the-pythondiscord-workers-serverless-webscale-platform
#
jchristgit
[python-discord/site] branch deleted: the-pythondiscord-workers-serverless-webscale-platform
#
jchristgit
[python-discord/site] branch deleted: the-pythondiscord-workers-serverless-webscale-platform
dusk berry
#

^ tests, please ignore

regal archBOT
#
jchristgit
[python-discord/workers] New branch created: kill-github-filter-worker
regal archBOT
#
dependabot[bot]
[python-discord/sir-lancebot] New branch created: dependabot/pip/pydis-core-10.5.0
#
dependabot[bot]
[python-discord/sir-lancebot] Pull request opened: #1408 Bump pydis-core from 10.4.0 to 10.5.0

Bumps pydis-core from 10.4.0 to 10.5.0.

Release notes
Sourced from pydis-core's releases.

v10.5.0
What's Changed

Mark dependencies using tilde version specifiers. This is to allow users of pydis core to use newer versions of these libraries without us having to cut a new release. by @​ChrisLovering in python-discord/bot-core#197

Full Changelog: https://github.com/python-discord/bot-core/compare/v10.4.0...v10.5.0

Changelog
Sourced from pydis...

#
dependabot[bot]
[python-discord/sir-lancebot] New branch created: dependabot/pip/pre-commit-3.6.0
#
dependabot[bot]
[python-discord/sir-lancebot] Pull request opened: #1409 Bump pre-commit from 3.5.0 to 3.6.0

Bumps pre-commit from 3.5.0 to 3.6.0.

Release notes
Sourced from pre-commit's releases.

pre-commit v3.6.0
Features

Check minimum_pre_commit_version first when parsing configs.

#3092 PR by @​asottile.

Fixes

Fix deprecation warnings for importlib.resources.

#3043 PR by @​asottile.

Fix deprecation warnings for rmtree.

#3079 PR by @​edgarrmondragon.

Updating

Drop support for python<3.9.

#3042 PR by @​asottile.
#3093 PR by @​asottile....

odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
#
github-merge-queue[bot]
[python-discord/workers] New branch created: gh-readonly-queue/main/pr-207-9424f06dbbf422e4da506e8db33ff5e68afdab3c
odd spireBOT
regal archBOT
#
github-merge-queue[bot]
[python-discord/workers] branch deleted: gh-readonly-queue/main/pr-207-9424f06dbbf422e4da506e8db33ff5e68afdab3c
#
dependabot[bot]
[python-discord/workers] New comment on pull request #122: Bump undici from 5.23.0 to 5.26.3 in /github-filter

OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting @dependabot ignore this major version or @dependabot ignore this minor version. You can also ignore all major, minor, or patch releases for a dependency by adding an [ignore condition](https://docs.github.com/en/code-security/supply-chain-security/configuration-options-for-dependency-upda...

regal archBOT
#
dependabot[bot]
[python-discord/bot] Pull request opened: #2840 Bump feedparser from 6.0.10 to 6.0.11

Bumps feedparser from 6.0.10 to 6.0.11.

Changelog
Sourced from feedparser's changelog.

6.0.11 - 2023-12-10

Resolve cgi module deprecation warnings. (#330)

Commits

6cdc208 Merge pull request #417 from kurtmckee/hotfix-6.0.11
7da33c6 Update the README
1d89154 Update project metadata
13f7ef1 Replace a call to cgi.parse_header()
See full diff in compare view

[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/co...

#
dependabot[bot]
[python-discord/bot] New branch created: dependabot/pip/feedparser-6.0.11
#
dependabot[bot]
[python-discord/bot] New branch created: dependabot/pip/pre-commit-3.6.0
#
dependabot[bot]
[python-discord/bot] Pull request opened: #2841 Bump pre-commit from 3.5.0 to 3.6.0

Bumps pre-commit from 3.5.0 to 3.6.0.

Release notes
Sourced from pre-commit's releases.

pre-commit v3.6.0
Features

Check minimum_pre_commit_version first when parsing configs.

#3092 PR by @​asottile.

Fixes

Fix deprecation warnings for importlib.resources.

#3043 PR by @​asottile.

Fix deprecation warnings for rmtree.

#3079 PR by @​edgarrmondragon.

Updating

Drop support for python<3.9.

#3042 PR by @​asottile.
#3093 PR by @​asottile....

#
dependabot[bot]
[python-discord/bot] New branch created: dependabot/pip/pydis-core-10.5.0
#
dependabot[bot]
[python-discord/bot] Pull request opened: #2842 Bump pydis-core from 10.4.0 to 10.5.0

Bumps pydis-core from 10.4.0 to 10.5.0.

Release notes
Sourced from pydis-core's releases.

v10.5.0
What's Changed

Mark dependencies using tilde version specifiers. This is to allow users of pydis core to use newer versions of these libraries without us having to cut a new release. by @​ChrisLovering in python-discord/bot-core#197

Full Changelog: https://github.com/python-discord/bot-core/compare/v10.4.0...v10.5.0

Changelog
Sourced from pydis...

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/site] branch deleted: document-faq-python-discord-content
#
dependabot[bot]
[python-discord/metricity] New branch created: dependabot/pip/pydis-core-10.5.0
#
dependabot[bot]
[python-discord/metricity] Pull request opened: #107 Bump pydis-core from 10.4.0 to 10.5.0

Bumps pydis-core from 10.4.0 to 10.5.0.

Release notes
Sourced from pydis-core's releases.

v10.5.0
What's Changed

Mark dependencies using tilde version specifiers. This is to allow users of pydis core to use newer versions of these libraries without us having to cut a new release. by @​ChrisLovering in python-discord/bot-core#197

Full Changelog: https://github.com/python-discord/bot-core/compare/v10.4.0...v10.5.0

Changelog
Sourced from pydis...

#
dependabot[bot]
[python-discord/metricity] New branch created: dependabot/pip/pre-commit-3.6.0
#
dependabot[bot]
[python-discord/metricity] Pull request opened: #108 Bump pre-commit from 3.5.0 to 3.6.0

Bumps pre-commit from 3.5.0 to 3.6.0.

Release notes
Sourced from pre-commit's releases.

pre-commit v3.6.0
Features

Check minimum_pre_commit_version first when parsing configs.

#3092 PR by @​asottile.

Fixes

Fix deprecation warnings for importlib.resources.

#3043 PR by @​asottile.

Fix deprecation warnings for rmtree.

#3079 PR by @​edgarrmondragon.

Updating

Drop support for python<3.9.

#3042 PR by @​asottile.
#3093 PR by @​asottile....

regal archBOT
regal archBOT
#
jchristgit
[python-discord/site] New comment on pull request #1165: Implement editing of offensive message records

Surely if we fail to delete the message due to a Discord issue, we just shouldn't delete it from the DB. When the bot next checks for messages to delete, the message that failed will still be there, with a date in the past, so it will try to delete again.

I think the idea is more to have a longer, adequate retry period. As far as I remember from when we implemented this, this cleanup is more intended as a "later down the line" batch job rather than "this needs to be removed instantly", s...

#
ChrisLovering
[python-discord/site] branch deleted: reschedule-offensive-messages
#
jchristgit
[python-discord/site] New branch created: frozen-fields-unify
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/site] New branch created: Raise-sentry-warning-when-ratelimitted
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/site] branch deleted: frozen-fields-unify
#
ChrisLovering
[python-discord/site] branch deleted: Raise-sentry-warning-when-ratelimitted
odd spireBOT
regal archBOT
regal archBOT
#
janine9vn
[python-discord/bot] New comment on issue #1353: Implement a better system for keeping track of alternate accounts

I'm going to pick this up as a winter holidays project I think.

The approach I'm thinking of:

  • New infraction type "association" that operates kind of like a note.
    • I'll probably ask some folks for help in trying to determine how to structure this infraction type. It has some unique meta-information I'm a bit unsure how to handle wrt our current models setup.
  • New moderation bot command !association (with aliases: !alt, !alts, !assoc) that will look like the following:
    ...
regal archBOT
#
jchristgit
[python-discord/site] New branch created: user-alts
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
dependabot[bot]
[workers] Branch dependabot/npm_and_yarn/url-unfurler/development-dependencies-4f2fa657b6 was force-pushed to `00455de`

Compare changes

odd spireBOT
regal archBOT
#
dependabot[bot]
[workers] Branch dependabot/npm_and_yarn/serve-robots/development-dependencies-4f2fa657b6 was force-pushed to `415a8a8`

Compare changes

regal archBOT
odd spireBOT
regal archBOT
#
dependabot[bot]
[python-discord/patsy] New branch created: dependabot/pip/fastapi-0.105.0
#
dependabot[bot]
[python-discord/patsy] Pull request opened: #73 Bump fastapi from 0.103.1 to 0.105.0

Bumps fastapi from 0.103.1 to 0.105.0.

Release notes
Sourced from fastapi's releases.

0.105.0
Features

✨ Add support for multiple Annotated annotations, e.g. Annotated[str, Field(), Query()]. PR #10773 by @​tiangolo.

Refactors

🔥 Remove unused NoneType. PR #10774 by @​tiangolo.

Docs

📝 Tweak default suggested configs for generating clients. PR #10736 by @​tiangolo.

Internal

🔧 Update sponsors, add Scalar. PR #10728 by @​tiangolo.
🔧 Update sponsors,...

#
ChrisLovering
[python-discord/site] New branch created: update-sentry-config
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
jchristgit
[python-discord/site] branch deleted: update-sentry-config
regal archBOT
regal archBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
oak estuaryBOT
odd spireBOT
oak estuaryBOT
regal archBOT
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
jchristgit
[python-discord/site] New branch created: mailing-list-model
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/site] New review comment on pull request #1171: Add alternate accounts to the user model

I missed something from the original requirements when first reviewing this PR, only realised when starting to do the bot side.

We want an info/reason text field here too, which is required when marking two users as alts. This is so that mods can give proof as to why they this the two users are alts.

This does also add a bit more complexity...as we likely want a way to edit this reason too.

I'll start on the bot side impl of this anyway, since it's mostly going to be the same from ...

regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] New branch created: feat/1353/alt-users
#
ChrisLovering
[python-discord/bot] Pull request opened: #2845 Add a cog to manage tracking a user's alt accounts

Important note

This must be merged with https://github.com/python-discord/site/pull/1171

Summary

This allows moderators, in mod channels, to add, remove & list a user's alt accounts.

When searching for infractions by user id, the number of al accounts is also shown, if there are any.

There are two TODOs in the code, which are awaiting the site implementation to be updated.

Screenshot

Wording can be discussed...
![image](https://github.com/python-discord/bot/assets/97...

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

Connected!

oak estuaryBOT
regal archBOT
#
hedyhli
[python-discord/site] Pull request review submitted: #1138 Fixed truncated dropdown menu

Thank you! This was bothering me for a long time.

One problem though, this doesn't fix it on mobile:

Maybe it's out of scope, I could fix this case in another PR.

However, here's my idea:

We can add two lines to the max width media query (in the same file), so the dropdown menu can then be fixed on the right edge:

On line 12, something like this:

.dropdown-menu {
  left: unset;
  right: 0;
}

What do you think?

odd spireBOT
regal archBOT
regal archBOT
#
hedyhli
[python-discord/site] Pull request opened: #1176 Dark theme

I have pulled in @GDWR's work and new changes from main.

I have also went ahead and fixed all the conflicts.

This is intended to supplement the original PR #657, in case anyone working on #362 wishes to base the work on top of this one.

Unfortunately I did the pull from main in 2 stages, which created two commits of the same message. I couldn't find a way to rectify this without having to fix the conflicts all over again, which is rather laborious.

odd spireBOT
regal archBOT
regal archBOT
#
jchristgit
[python-discord/site] New review comment on pull request #1171: Add alternate accounts to the user model

I've implemented this, unfortunately we lose the symmetrical aspect of the ManyToManyField then:

https://docs.djangoproject.com/en/5.0/topics/db/models/#intermediary-manytomany

For a model which has a many-to-many relationship to itself through an intermediary model, two foreign keys to the same model are permitted, but they will be treated as the two (different) sides of the many-to-many relationship. If there are more than two foreign keys though, you must also specify through...

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
jchristgit
[python-discord/site] New comment on pull request #1176: Dark theme

Unfortunately I did the pull from main in 2 stages, which created two commits of the same message. I couldn't find a way to rectify this without having to fix the conflicts all over again, which is rather laborious.

You could try to squash the commits in an interactive rebase. Or use git rerere (reuse recorded resolution)

odd spireBOT
odd spireBOT
regal archBOT
#
hedyhli
[python-discord/site] New comment on pull request #1176: Dark theme

@jchristgit thanks for these ideas. Unfortunately interactive rebase didn't seem to work as they were merge commits, and AIUI rerere only takes effect after it is enabled. I have made it into a single commit and force pushed, so all good now. (Made use of git checkout --theirs <file> which sped things up.)

Lint check is also fixed.

odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
hedyhli
[python-discord/site] New comment on issue #362: Implement a dark theme

Hi @Ibrahim2750mi, since it's been over a year without a follow-up comment, I'm assuming you're no longer in interested in implementing this?

It has come to my attention that this issue may be up for grabs again, and we still don't have a dark theme 😢

I have recently pulled in GDWR's previous work and synced upstream changes (fixed conflicts). If anyone wishes to work on top of it, it should be very easy to do so.

If no one decides to pick this up in a week (starting today), can get...

regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot-core] New branch created: dont-read-response-for-204-codes
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
shtlrs
[python-discord/site] New branch created: bulk-create-deleted-messages
regal archBOT
#
jchristgit
[python-discord/site] branch deleted: bulk-create-deleted-messages
regal archBOT
#
shtlrs
[python-discord/site] New branch created: debug-exit-status-2
#
ChrisLovering
[python-discord/site] branch deleted: debug-exit-status-2
#
shtlrs
[python-discord/site] New branch created: useless-pr
#
shtlrs
[python-discord/bot] New branch created: useless-pr
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
#
shtlrs
[python-discord/site] New branch created: download-artifact-with-curl
#
ChrisLovering
[python-discord/site] branch deleted: download-artifact-with-curl
regal archBOT
regal archBOT
regal archBOT
#
jchristgit
[python-discord/site] New branch created: nplusone-infractions-expanded
odd spireBOT
regal archBOT
#
jchristgit
[python-discord/site] New branch created: nplusone-nomination-entries
odd spireBOT
regal archBOT
regal archBOT
#
jchristgit
[python-discord/site] New branch created: slightly-optimize-tag-updates
odd spireBOT
regal archBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

oak estuaryBOT
regal archBOT
#
shtlrs
[python-discord/bot] New branch created: new-mailing-list-schema
#
shtlrs
[python-discord/bot] Pull request opened: #2852 Use the new mailing lists API to track news

WARNING: This needs to be merged along with site#1174

In the effort of removing the bot-settings table that has both news & defcon settings, we've started migrating these to their own tables.

This updates the PythonNews Cog to use the new endpoint to fetch/update news that have been tracked/posted.

We used to do syncing of schemas as we relied on a "dynamic" JSON, but now the mailing lists are predefined so no syncing is requi...

#
shtlrs
[python-discord/site] branch deleted: nplusone-nomination-entries
#
jchristgit
[python-discord/site] branch deleted: slightly-optimize-tag-updates
#
jchristgit
[python-discord/site] branch deleted: nplusone-infractions-expanded
odd spireBOT
regal archBOT
regal archBOT
thin oysterBOT
#
Sir Robin

Connected!

thin oysterBOT
#
Sir Robin

Connected!

regal archBOT
odd spireBOT
regal archBOT
#
shtlrs
[python-discord/site] New review comment on pull request #1174: Migrate mailing lists to their own API endpoints

As I was making the bot side, I realised that the fact that we can't create mailing lists is an issue for the following reasons:

  1. We don't have any migrations that create theses mailing lists, so if you delete a container that used to have th old table structure, you'll never end up with the target mailing lists.
  2. We used to workaround this bot side thanks to the sync_mailing_lists method that creates the proper JSON that will be dumped in the db, this is no longer true
  3. The bot wi...
regal archBOT
#
ChrisLovering
[python-discord/bot] New branch created: bump-bot-core
oak estuaryBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
#
jchristgit
[python-discord/site] branch deleted: useless-pr
regal archBOT
regal archBOT
#
shtlrs
[python-discord/bot] branch deleted: useless-pr
#
shtlrs
[python-discord/bot] New branch created: download-artifacts-with-curl
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
regal archBOT
regal archBOT
#
jchristgit
[python-discord/bot] branch deleted: download-artifacts-with-curl
night lilyBOT
#
Sir Lancebot

Connected!

oak estuaryBOT
regal archBOT
#
MrHemlock
[python-discord/bot] Pull request opened: #2856 Voice Verify Button Instead of Voice Verify Command

Summary

Users have been having some issues knowing exactly what to do when trying to verify for voice permissions. I've gone ahead and converted the command (which people never quite seemed to understand where to use the command) into an easy to press button.

I've also reworked how message deletions are handled within the cog, as the delete_after argument on sent messages so that we aren't handling it solely inside the cog.

The only thing that would need to be done before we pull ...

odd spireBOT
regal archBOT
#
shtlrs
[python-discord/bot] New review comment on pull request #2856: Voice Verify Button Instead of Voice Verify Command

You could shorten this quite a bit

except ResponseCodeError as err:
    message = "We encountered an error while attempting to find data for your user. " \
              "Please try again and let us know if the problem persists."
    if err.response.status == 404:
        message = """
        We were unable to find user data for you.
        Please try again shortly.
        If this problem persits, please contact the server staff through ModMail.
        """
        log.inf...
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
oak estuaryBOT
regal archBOT
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
Robin5605
[python-discord/bot] New review comment on pull request #2856: Voice Verify Button Instead of Voice Verify Command

I think what we could do is if the user invoking this command omits the channel, then the default channel would be #voice-verify. If they specify a channel, the message would go there.

Same deal with the text, maybe we could set the fallback text value to what we already have, but it can be overridden if necessary.

@MrHemlock I think we'll also either need to delete the original message that invoked the command if it's sent in #voice-verify so the initial command message doesn't li...

regal archBOT
oak estuaryBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
devdanzin
[python-discord/site] Issue opened: #1193 Add Software Design by Example to resources

Software Design by Example is a great free intermediate level book on software design using Python. Here's a proposed description:

description: A tool-based introduction to Software Design with Python. This book teaches
  design by explaining examples of small versions of familiar tools to show how 
  experienced software designers think. It introduces some fundamental ideas in computer
  science that many self-taught programmers haven’t encountere...
odd spireBOT
regal archBOT
#
MrHemlock
[python-discord/bot] New review comment on pull request #2856: Voice Verify Button Instead of Voice Verify Command

I was thinking we would be doing it in a channel like mod-spam or admin-spam. And we'd have the ability to just delete the message itself. Not only that, not deleting it right away makes it easy to re-copy and fix if I (inevitably) type something incorrectly while putting up the rules and message. I feel like this command really doesn't need to have a lot of functionality tied up with it. It's only going to be used once in a blue moon, likely just by me.

And yes, because it's adding a ...

regal archBOT
regal archBOT
regal archBOT
#
MrHemlock
[python-discord/bot] New review comment on pull request #2856: Voice Verify Button Instead of Voice Verify Command

After looking this over multiple times, I just don't see the benefit of splitting it up. The methods I would be creating would mostly be one line returns. It feels like it ends up more.... I guess Java like, with gets that don't really benefit the code.

I do see where it could help readability by being able to just look at the general class structure and see the method names, but at the same time, splitting it up leads to a couple complications. For instance, for the try block, I cons...

odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
shtlrs
[python-discord/bot] New comment on issue #2861: Edit eval message instead of deleting and re-sending

Technically, there isn't anything that would prevent us from doing it.

However, when there's a high throughput in the channel, getting a "refresher" instead of having the old eval burried in a pile of newly sent messages can redirect attention to what's trying to be demonstrated by the code evaluation.

Unless someone is spamming the eval command, which can be remediated by a cooldown, I'm not entirely sure about editing the old output.

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

Connected!

odd spireBOT
regal archBOT
#
vivekashok1221
[python-discord/bot] New review comment on pull request #2839: Support editing of timeout durations.

If the user is not present in the server, the infraction duration is updated in the database and a confirmation message is sent- but the actual timeout duration is not changed Discord-side (basically the same undesirable behaviour we have for editing timeout duration right now) .

We should either:
a) notify that the user has left the server
b) apply the new timeout duration when the user rejoins the server (might have to bring back some of logic we had for on_member_join event from ...

odd spireBOT
regal archBOT
regal archBOT
#
ichard26
[python-discord/bot] New branch created: raw-reply
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
oak estuaryBOT
odd spireBOT
oak estuaryBOT
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
kwzrd
[python-discord/bot] Pull request opened: #2869 Branding: improve IO resilience

Closes GH-2808

This PR aims to improve the resiliency of the Branding manager cog.

Context

The Branding manager automatically discovers events once per day at midnight -- or when instructed to. During each such discovery, it makes multiple HTTP requests to the GitHub API. First, it fetches a list of all events in the Branding repository, and then it fetches those events one-by-one.

If the bot fails to fetch an event, it skips it. This behaviour made sense originally, as we didn'...

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

Connected!

oak estuaryBOT
regal archBOT
regal archBOT
regal archBOT
#
wookie184
[python-discord/site] Pull request review submitted: #1166 Add endpoint to fetch filters in JSON format

The response format here doesn't seem ideal. My understanding is that "filters" is the display name, and "valid_filters" is what goes in the URL(?), but the response doesn't link these together.

We could link them together in the structure, along the lines of this:

[
    {
      "name": "Difficulty",
      "url_value": "difficulty",
      "options": [
        {"name": "Beginner", "url_value": "beginner"},
        ...
      ],
      "icon": "fas fa-brain",
      "hidden": fal...
regal archBOT
#
wookie184
[python-discord/bot] New branch created: fix-incorrect-test
odd spireBOT
regal archBOT
#
wookie184
[python-discord/bot] New branch created: fix-pydantic-warning-in-test
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
wookie184
[bot:main] 6 new commits

4d1ba10 Capitalize "Discord" in header; Change print st... - swfarnsworth
96038fd Rewriting of non-code sections. - swfarnsworth
36f1aa2 New tag to explain why == True et al are wrong. - swfarnsworth
0ff5fad Revert 'Code goes here on a new line' to 'Hello... - swfarnsworth
0ccd4fb Merge branch 'main' into swfarnsworth/tag-updates - wookie184

#
wookie184
[python-discord/bot] branch deleted: swfarnsworth/tag-updates
odd spireBOT
oak estuaryBOT
odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
oak estuaryBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

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

Connected!

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

Connected!

odd spireBOT
regal archBOT
regal archBOT
#
wookie184
[python-discord/bot] New branch created: move-modlog-method-to-util
odd spireBOT
oak estuaryBOT
regal archBOT
#
wookie184
[python-discord/bot] New comment on issue #1010: ClientPayloadError: Response payload is not completed

This is an error we still get very occasionally (https://python-discord.sentry.io/issues/4083824164). It also seems to come up fairly often on the aiohttp issue tracker (e.g. https://github.com/aio-libs/aiohttp/issues/4581), so it seems that it's either an obscure aiohttp bug that nobody has managed to track down, or some unavioidable network error.

I also assume this could pop up anywhere, but since the sync cogs make relatively quite a lot of network traffic it's most likely to be there....

regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
MrHemlock
[python-discord/bot] branch deleted: voice_button
oak estuaryBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
wookie184
[python-discord/bot] Issue opened: #2878 Update "unambiguous" converters given Discord username changes

The current logic for detecting a username in the unambigous username converter checks if it contains a discriminator, but since discord removed discriminators it doesn't make sense any more:

https://github.com/python-discord/bot/blob/da6997aab25b57152eec5d8c706977c472563853/bot/converters.py#L425-L433

We can probably safely just remove support for passing a username, since the converters are mostly used for moderation commands where IDs are pretty much all that is used anyway.

regal archBOT
#
janine9vn
[python-discord/sir-robin] New branch created: completionist-2023
odd spireBOT
regal archBOT
regal archBOT
#
shtlrs
[python-discord/bot] New review comment on pull request #2682: Use `LinePaginator` from `bot-core`

@wookie184

Hey ! I'm trying to pick this up again and I was wondering what you meant exactly by

One simple way around this for now would be to just create a wrapper here that passes in the necessary constants and we can just use that instead. In the future we may want to think of other ways we can have configuration for bot-core.

Do you mean that we'd keep the pagination emojis as part of the parameters, and then make a decorator that will pass them for us ?
Something like

regal archBOT
#
ChrisLovering
[python-discord/sir-robin] branch deleted: completionist-2023
thin oysterBOT
#
Sir Robin

Connected!

regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
wookie184
[python-discord/bot] New review comment on pull request #2682: Use `LinePaginator` from `bot-core`

I was thinking we could replace bot/pagination.py with something like this

from pydis_core import LinePaginator as _LinePaginator
...

PaginationEmojis = PaginationEmojisModel(delete=Emojis.trashcan)

class LinePaginator(_LinePaginator):
    @classmethod
    async def paginate(cls, *args, **kwargs) -> discord.Message | None:
        return await super().paginate(PaginationEmojis, *args, **kwargs)

then we wouldn't need to update any imports either.

The main annoy...

regal archBOT
regal archBOT
#
shenanigansd
[python-discord/sir-lancebot] New branch created: shenanigansd-patch-2
#
shenanigansd
[python-discord/sir-lancebot] Pull request opened: #1423 Remove "What third-party library do you wish was in the Python standard library?" from topics

Relevant Issues

Raised by @nedbat on [Discord](#helpers message) (requires PyDis Helpers role)

Description

  • Removed "What third-party library do you wish was in the Python standard library?" from topics

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 discor...
odd spireBOT
regal archBOT
#
wookie184
[python-discord/sir-lancebot] branch deleted: shenanigansd-patch-2
#
wookie184
[python-discord/sir-lancebot] New branch created: fix-pypi-capitalisation
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
odd spireBOT
regal archBOT
#
shtlrs
[python-discord/bot] New branch created: use-botcore-paginator
#
wookie184
[python-discord/sir-lancebot] branch deleted: fix-pypi-capitalisation
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
wookie184
[python-discord/bot] New branch created: update-unambiguous-converters
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
wookie184
[python-discord/bot] branch deleted: update-unambiguous-converters
odd spireBOT
oak estuaryBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
brad90four
[python-discord/branding] New branch created: aoc_2023_completionist
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/branding] branch deleted: aoc_2023_completionist
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/snekbox] New branch created: bump-eval-deps
regal archBOT
regal archBOT
#
wookie184
[python-discord/snekbox] branch deleted: bump-eval-deps
regal archBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
Xithrius
[python-discord/bot] branch deleted: move-modlog-method-to-util
odd spireBOT
regal archBOT
#
Xithrius
[python-discord/bot] branch deleted: fix-pydantic-warning-in-test
odd spireBOT
oak estuaryBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT