#dev-log
1 messages ยท Page 29 of 1
GitHub Actions run 15812849295 succeeded.
Connected!
[python-discord/bot] New branch created: jb3/infraction-message-tidy-up
8bf7453 Delete automated infraction messages after a pe... - jb3
This PR automatically deletes messages generated by the bot as part of dealing out automated infractions (e.g. spam filters). These have a habit of filling up channels which are frequently abused but have low traffic otherwise (some channels appear as just a wall of automated infractions).
As of now, the timer is set to 8 hours, though we can increase this if we feel we need the longer persistence (though obviously the full record and log is still preserved in the #mod-alerts channel).
...
GitHub Actions run 15814795284 succeeded.
Can we avoid the message fetch call? Can we delete the message if we just know its ID (and channel/guild IDs)?
Should we also delete it from Redis in this case? On one hand, it's nice to be able to try later if there is a transient issue with fetching the channel. On the other hand, it's wasteful to keep trying if the channel has been deleted.
Consider setting a TTL in Redis for these records. That way if a channel truly is gone, stale records won't pile up.
How does this interact with threads that get closed? Can the bot still delete messages?
Bot should always have permissions to delete messages anywhere in the server. I will handle discord.Forbidden though and ensure we log it out (but still clear the record, this should all be best effort).
Without hooking into the internals of discord.py, there isn't a way without fetching the message (you could in theory either call the raw HTTP message or try coerce a discord.Object into being a discord.Message). I couldn't find a particularly clean way though so settled for just fetching the message.
No worries, it's not a major concern. We can revisit this if it proves to be a performance problem.
8bef06b Improve error handling with scheduled deletion ... - jb3
GitHub Actions run 15816336755 failed.
GitHub Actions run 15816348138 succeeded.
GitHub Actions run 15816367620 succeeded.
Connected!
GitHub Actions run 15816357725 succeeded.
GitHub Actions run 15816539391 succeeded.
Unsure if this is a warning, since trying to delete messages in closed threads will also raise Forbidden iirc
Ah I hadn't realised that manage messages does not override this. I'll drop it to an info log.
4b04acb Drop log verbosity for forbidden infraction mes... - jb3
GitHub Actions run 15829029328 succeeded.
Connected!
GitHub Actions run 15829095257 succeeded.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.31.0
GitHub Actions run 15860491862 succeeded.
One minor comment, though if others disagree then let me know, it is more a nit based on me not wanting to add dependence onto intricacies of Redis and hoping multiple keys exist referring to the same slowmode delay etc.
Given these are both structured and known values, I would maybe consider serializing them together into one store?
This is a nit though and others may disagree, it just appears that we use them more or less in the same place so whilst if we were just working in dict-land this would be preferred when working in a place with an underlying store I prefer it being the same.
[python-discord/bot] New branch created: jb3/mod-changes-refactor
Refactors some items from #3339 and #3338.
- Use partial objects to avoid unnecessary API calls when tidying up infraction messages
- I hadn't noticed the partial functions when I went on my hunt, I was working under the assumption I'd have to hook into the HTTP stuff or create my own partials, but this is much cleaner and cheaper.
- Don't tidy-up infractions outside mod channels (ensures that infractions issued by the bot automatically in mod-alerts stay as part of the context)
- Exc...
GitHub Actions run 15863677057 succeeded.
expiration_time = await self.slowmode_expiration_cache.get(channel.id, None)
if expiration_time is not None:
if slowmode_delay > SLOWMODE_MAX_DELAY:
f"The slowmode delay for {channel.mention} is {humanized_delay} and expires {expiration_timestamp}."
The "in" should be already included in a relative timestamp if I'm not mistaken
Also, should this specify what happens when the slowmode expires? e.g "and will revert to {humanized_original_delay} {expiration_timestamp}"
This section is then simplified with the usage of Duration.
f"{humanized_delay} which expires {humanized_duration}."
Same here, can specify what it's going to revert to.
Slowmode changes are usually silent. Unless the mods agreed that this is what they want I would send this message to #mods
GitHub Actions run 15868554580 succeeded.
Connected!
GitHub Actions run 15874535487 succeeded.
Suspect Issues
This pull request was deployed and Sentry observed the following issues:
- โผ๏ธ HTTPException: 400 Bad Request (error code: 50083)... in
production
<sub>Did you find this useful? React with a ๐ or ๐</sub>
Connected!
GitHub Actions run 15883781921 succeeded.
GitHub Actions run 15911850508 succeeded.
GitHub Actions run 15918804134 failed.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.31.0
GitHub Actions run 15924966471 succeeded.
GitHub Actions run 15926809244 succeeded.
GitHub Actions run 15946839433 succeeded.
Looks like mod-log will already pick up when the slowmode delay is modified and sends it as an embed. Should I also send the above to that channel?
And if i send it to mod-log, should get_or_fetch_channel be used here also? Should the mod-log channel be defined as an instance attribute inside init or is it fine to define it inside of the command provided that get_or_fetch_channel is used?
GitHub Actions run 15965556666 succeeded.
No if it's automatically picked up then great. I'd just send this to #mods
GitHub Actions run 15984150757 succeeded.
GitHub Actions run 15984868561 succeeded.
GitHub Actions run 15987063798 succeeded.
GitHub Actions run 15998819612 succeeded.
GitHub Actions run 16042471778 failed.
GitHub Actions run 16066313120 failed.
WTF
GitHub Actions run 16071498382 failed.
i mean what the fuck
GitHub Actions run 16078015762 succeeded.
@ocean rock p0 resolved
GitHub Actions run 16078086769 succeeded.
Team, please prepare for the completely unrelated and untested bug that is introduced in DRF as a result of jc's upgrade here. Thanks, Banks.
GitHub Actions run 16078130670 succeeded.
GitHub Actions run 16078112852 succeeded.
GitHub Actions run 16078152095 succeeded.
[site] Branch dependabot/pip/ruff-0.12.2 was force-pushed to `7fa0144`
GitHub Actions run 16078179658 succeeded.
GitHub Actions run 16196272678 succeeded.
merging this completely untested
[python-discord/king-arthur] branch deleted: jb3/github-management
Suspect Issues
This pull request was deployed and Sentry observed the following issues:
- โผ๏ธ ExtensionFailed: Extension 'arthur.exts.github.man... in
production
<sub>Did you find this useful? React with a ๐ or ๐</sub>
GitHub Actions run 16196405237 succeeded.
GitHub Actions run 16196556394 succeeded.
I'm so sorry
GitHub Actions run 16196669958 failed.
[python-discord/king-arthur] New branch created: jb3-patch-1
4b139fd Update init.py in GitHub API with new excep... - jb3
GitHub Actions run 16196877153 succeeded.
GitHub Actions run 16197143715 succeeded.
GitHub Actions run 16241349275 succeeded.
GitHub Actions run 16241338548 succeeded.
GitHub Actions run 16251198980 succeeded.
[python-discord/bot-core] New branch created: discord_invite_update
7a0f529 Update discord invite regex - janine9vn
Allows for more than one \ that occurs before the invite code, which is apparently valid.
Tested against a few known discord invites we see in regex101. Can provide the link to the regex101 testing if needed.
[python-discord/bot-core] branch deleted: discord_invite_update
e3c2cbd Deploying to docs from @ python-discord/bot-cor... - jb3
[python-discord/bot-core] New tag created: v11.6.1
8d6175d Deploying to docs from @ python-discord/bot-cor... - jb3
[python-discord/site] New branch created: resources-category-update
abf8ffc Add new category and rename existing one - janine9vn
As a part of going through our current open issues for resources, I've identified a new category we could benefit from (Conference Talks) and also renaming another one (Project Ideas --> Practice).
I adjusted the sole resources that was using the category that was renamed as well.
GitHub Actions run 16253338405 succeeded.
GitHub Actions run 16253387249 succeeded.
GitHub Actions run 16253476257 succeeded.
GitHub Actions run 16253503027 succeeded.
Connected!
GitHub Actions run 16253535623 succeeded.
Connected!
GitHub Actions run 16266792686 was cancelled.
Connected!
GitHub Actions run 16266833187 succeeded.
Connected!
GitHub Actions run 16288974159 succeeded.
GitHub Actions run 16289973684 succeeded.
GitHub Actions run 16292709554 succeeded.
GitHub Actions run 16292837487 succeeded.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.32.0
GitHub Actions run 16296744201 succeeded.
[python-discord/bot] New branch created: jb3/poetry-to-uv
- Migrate from using poetry to uv
- Update CI/CD to use uv over poetry
Once more projects have migrated we will need to update Contributing guidance on site to reflect this.
GitHub Actions run 16304795005 failed.
GitHub Actions run 16304879216 failed.
GitHub Actions run 16304907729 failed.
GitHub Actions run 16304923693 failed.
GitHub Actions run 16304938940 failed.
GitHub Actions run 16304950562 failed.
GitHub Actions run 16304971851 failed.
GitHub Actions run 16305014436 failed.
GitHub Actions run 16305035313 succeeded.
GitHub Actions run 16305041493 succeeded.
GitHub Actions run 16305068069 succeeded.
[python-discord/sir-lancebot] New branch created: jb3/poetry-to-uv
We're doing bind mounts to avoid copying the files into the image, which would affect the build cache?
Why not use stdlib venv? Does uv rely on virtualenv?
Relevant Issues
Not enough time left! Need faster dependency management! https://thebulletin.org/doomsday-clock/
Description
Poetry migrated to uv.
Did you:
- [ ] Join the Python Discord Community? No I haven't yet joined. I don't really like Discord.
- [ ] Read all the comments in this template? No I can't read only program. I'm washed.
- [ ] Ensure there is an issue open, or link relevant discord discussions? **There is a wi...
GitHub Actions run 16305375324 failed.
GitHub Actions run 16305400358 succeeded.
GitHub Actions run 16305536292 succeeded.
GitHub Actions run 16305542204 succeeded.
<img width="298" height="169" alt="image" src="https://github.com/user-attachments/assets/585fbe07-322a-4d69-8022-b1621a476514" />
[python-discord/bot] branch deleted: jb3/poetry-to-uv
Connected!
GitHub Actions run 16305619753 succeeded.
[python-discord/sir-lancebot] branch deleted: jb3/poetry-to-uv
Connected!
GitHub Actions run 16305945688 succeeded.
GitHub Actions run 16306015074 failed.
GitHub Actions run 16306024624 failed.
GitHub Actions run 16306029770 failed.
Connected!
GitHub Actions run 16306037223 succeeded.
Connected!
GitHub Actions run 16306124792 succeeded.
@umbral grove ALL IN ONE COMMIT!!??!???!
OF COURSE!!!
YOU DONT NEED GIT HISTORY!!!
XITHRIUS!
you win this round.
GitHub Actions run 16311361879 succeeded.
GitHub Actions run 16315222171 succeeded.
GitHub Actions run 16315400152 succeeded.
GitHub Actions run 16315502396 succeeded.
GitHub Actions run 16315627019 succeeded.
GitHub Actions run 16315965743 succeeded.
description in title, don't add this to your project and gl in your future works :)
GitHub Actions run 16318355342 failed.
GitHub Actions run 16326609619 succeeded.
Connected!
GitHub Actions run 16326632916 succeeded.
GitHub Actions run 16350975104 succeeded.
[python-discord/sir-robin] New branch created: jb3/poetry-to-uv
- Replace Poetry with uv
- Update Docker to use uv
- Update lint workflow to use uv for dependencies
GitHub Actions run 16354124543 failed.
GitHub Actions run 16354153528 succeeded.
GitHub Actions run 16354222132 succeeded.
8cbc8e8 Migrate lint-test from setup-python to uv - Xithrius
Docker, CI, and pyproject.toml have poetry replaced with uv. All guides have poetry replaced with uv. PyCharm portion of installing project dependencies is (temporarily?) removed since I'm not aware of a uv equivalent of the jetbrains poetry plugin.
GitHub Actions run 16359510993 was cancelled.
GitHub Actions run 16359512955 failed.
GitHub Actions run 16359542514 failed.
GitHub Actions run 16359629598 failed.
GitHub Actions run 16359739112 failed.
GitHub Actions run 16359804501 failed.
GitHub Actions run 16360099800 failed.
ratelimited 
GitHub Actions run 16362900535 failed.
This PR adds the assets for Summer Code Jam 2025.
- Discord banner.
- Alternative banner without extra text.
- Server icon.
- Illustrator .ai source file.
- SVG versions of the graphics.
GitHub Actions run 16372400061 succeeded.
GitHub Actions run 16372727342 succeeded.
GitHub Actions run 16373548369 succeeded.
GitHub Actions run 16373591269 succeeded.
GitHub Actions run 16374604940 failed.
GitHub Actions run 16374693723 failed.
GitHub Actions run 16374744032 failed.
GitHub Actions run 16374846672 succeeded.
GitHub Actions run 16374964870 succeeded.
GitHub Actions run 16362900535 succeeded.
GitHub Actions run 16375112961 succeeded.
GitHub Actions run 16375107033 succeeded.
GitHub Actions run 16375263866 succeeded.
Connected!
Connected!
GitHub Actions run 16378565130 succeeded.
Connected!
GitHub Actions run 16378574216 succeeded.
Connected!
GitHub Actions run 16378594229 succeeded.
GitHub Actions run 16378581486 succeeded.
/dev/null is webscale
97a9d85 Dependencies/docker to uv - Xithrius
b34e714 Project installation guides from poetry to uv - Xithrius
91c70d7 Installation of project dependencies guide migr... - Xithrius
c91c5e3 Other references/usages of poetry migrated to uv - Xithrius
6457abf Migrate lint-test from setup-python to uv - Xithrius
[python-discord/site] branch deleted: uv-migration
When I run this locally, I get the following error:
web-1 | Using CPython 3.11.13 interpreter at: /usr/local/bin/python3
web-1 | Creating virtual environment at: .venv
web-1 | error: failed to create directory `/app/.venv`: Read-only file system (os error 30)
Adding ENV UV_PROJECT_ENVIRONMENT="/build/.venv" to the dockerfile fixes it.
GitHub Actions run 16379321576 succeeded.
GitHub Actions run 16379325406 succeeded.
GitHub Actions run 16379310304 failed.
GitHub Actions run 16379695605 succeeded.
GitHub Actions run 16379834990 succeeded.
GitHub Actions run 16379889679 succeeded.
GitHub Actions run 16379985096 failed.
GitHub Actions run 16380057150 succeeded.
Connected!
GitHub Actions run 16386747851 succeeded.
GitHub Actions run 16391969972 failed.
7b7c451 Bump pre-commit from 4.1.0 to 4.2.0 - dependabot[bot]
1fca2c1 Merge pull request #1489 from python-discord/de... - jchristgit
2b4c791 Bump python-dotenv from 1.0.1 to 1.1.0 - dependabot[bot]
b4df5dd Merge pull request #1496 from python-discord/de... - jchristgit
bccdcfa Bump django from 5.1.7 to 5.1.8 - dependabot[bot]
[python-discord/site] New branch created: forms-admin
GitHub Actions run 16392035031 failed.
GitHub Actions run 16392061585 failed.
[python-discord/site] New review comment on pull request #1497: Set up forms app with authentication
We can definitely get away with the role information in the API app.
[python-discord/site] New review comment on pull request #1497: Set up forms app with authentication
We don't need:
- Nearly as much role information as we are currently storing. We use this almost solely for calculating scopes and can probably get away with name and ID (if even that, most of this data is validated on the fly)
- We don't need member information, just user information and whether they are in the guild or not.
As a sample of a filled response object:
{
"_id" : "7d880da8-0f06-4f4e-bc1d-9987eefa0d51",
"user" : {
"username" : "...",
...
[python-discord/site] New review comment on pull request #1497: Set up forms app with authentication
https://discord.com/developers/docs/reference#user-agent
We should probably add our user agent in here.
[python-discord/site] New review comment on pull request #1497: Set up forms app with authentication
Can we get rid of this file if it's empty anyway?
[python-discord/site] New review comment on pull request #1497: Set up forms app with authentication
Is it expensive to create a new client for each request?
[python-discord/site] New review comment on pull request #1497: Set up forms app with authentication
Is this supposed to match the module name? Can we avoid hardcoding the name? Maybe something with importlib can help?
[python-discord/site] New review comment on pull request #1497: Set up forms app with authentication
Should these be single backticks?
[python-discord/site] New review comment on pull request #1497: Set up forms app with authentication
Get a tuple of all roles from the cache, or Discord API if not available.
[python-discord/site] New review comment on pull request #1497: Set up forms app with authentication
This argument does not exist for this function.
[python-discord/site] New review comment on pull request #1497: Set up forms app with authentication
Do we need error handling for this call or is it fine to let the exception bubble up?
Can you remove the .ai file? That type of file doesn't do well when git-tracked and the SVG should be enough.
If we want we can post it in a discord channel for archival reasons or store it in our events folder in GDrive.
Otherwise, fantastic work! I love it.
GitHub Actions run 16406589611 was cancelled.
GitHub Actions run 16406656499 succeeded.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.33.0
GitHub Actions run 16417989270 succeeded.
GitHub Actions run 16424149087 failed.
GitHub Actions run 16424219402 failed.
GitHub Actions run 16425140438 succeeded.
:white_check_mark: Successfully fast-forwarded commits from python-discord:fix-typo onto main
[python-discord/king-arthur] branch deleted: fix-typo
GitHub Actions run 16425376510 succeeded.
[python-discord/site] New branch created: Matiiss-add-job-jobs-keywords-to-rule-command
5d18599 Add job, jobs as additional keywords for in... - Matiiss
GitHub Actions run 16429784571 succeeded.
[python-discord/site] branch deleted: Matiiss-add-job-jobs-keywords-to-rule-command
GitHub Actions run 16429804463 succeeded.
Connected!
GitHub Actions run 16430506875 succeeded.
GitHub Actions run 16430564388 succeeded.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.33.1
GitHub Actions run 16443785139 succeeded.
[python-discord/site] New branch created: cj12-initial-update
- upload banner and logo
- new cj12 pages
- update home page
GitHub Actions run 16448277148 succeeded.
GitHub Actions run 16448412142 succeeded.
Minor one, otherwise all looks good to me on the static preview.
This should be as easy as possible, which means you should consider using dependency managers like <code>uv</code> or <code>poetry</code>.
With our recent move to uv and our existing use of poetry, there aren't any situations where I'd recommend pipenv now. For us as reviewers, this is going to make things so much easier when initialising environments.
ed151b4 Add some font weights back where they make sense - jb3
[python-discord/site] branch deleted: cj12-initial-update
GitHub Actions run 16450261897 was cancelled.
GitHub Actions run 16450261897 succeeded.
Hi @Snipy7374, just checking whether this is still a feature you'd like to implement?
You may need to pull in some updates from the main branch, if you no longer have time that's also completely okay, just let us know!
If you need any help or guidance feel free to drop a message in the #dev-contrib channel.
Hi @AbdelbassitAb, just checking whether this is still a feature you'd like to implement?
You may need to pull in some updates from the main branch, if you no longer have time that's also completely okay, just let us know!
If you need any help or guidance feel free to drop a message in the #dev-contrib channel.
If this is something you're able to update to the latest changes from main, I'm happy to merge this! Sorry for the delayed review!
GitHub Actions run 16451746035 succeeded.
GitHub Actions run 16451839150 succeeded.
Connected!
GitHub Actions run 16451813679 succeeded.
Connected!
GitHub Actions run 16451933676 succeeded.
GitHub Actions run 16452020940 succeeded.
GitHub Actions run 16452064950 succeeded.
Connected!
GitHub Actions run 16452481154 succeeded.
Closing due to inactivity for now, please comment if this is something you'd like re-opening to work on in future!
Closing due to inactivity for now, please comment if this is something you'd like re-opening to work on in future!
Closing due to inactivity for now, please comment if this is something you'd like re-opening to work on in future!
GitHub Actions run 16452628099 succeeded.
Closing due to inactivity for now, please comment if this is something you'd like re-opening to work on in future!
Closing due to inactivity for now, please comment if this is something you'd like re-opening to work on in future!
Closing due to inactivity for now, please comment if this is something you'd like re-opening to work on in future!
Connected!
GitHub Actions run 16452711680 succeeded.
GitHub Actions run 16452888833 failed.
No time for now, closing.
This is mostly done, I think the resource approval process has been tested enough we know how to get things through.
Chris has solved this in the linked commit, any further surfacing we can add when we get time or people raise further ideas.
With current state of the repository we do not need this.
With the current state of contirbution to our repositories this is not super important.
If people ask in #dev-contrib we can point them to the section of bot where we have already documented this.
After speaking to the Python Discord delegates in Madagascar we have concluded that this is no longer an issue for people on most connections. Even when using slower connection throttles. We can work on smaller more actionable issues as we identify performance issues throughout site.
cc @janine9vn as events lead, is this still something we think we need with the frequency that we run events?
Any solution here is a pain and we can probably just accept the annoyance.
If we do revisit we should just add an endpoint called by a Kubernetes cronjob.
But the image is no longer available, and so only the PyCharm logo is showing. We should maybe also remove the wikimedia link there.
It could be replaced by https://upload.wikimedia.org/wikipedia/en/0/08/JetBrains_beam_logo.svg.
[python-discord/bot] New comment on issue #240: Moderation undo reaction, infraction void subcommand
We are closing this issue as the frequency of contributions to our repositories at the moment and the reduced need to revoke infractions/reduced rates of false applications.
DevOps are still able to void infractions when necessary.
Due to the reduced number of infractions we are distributing and reduced contribution to our repos this is no longer something we feel we need. Moderators are able to issue timeouts and as such as can replicate bans and mutes even when the bot is offline.
Closed in 2025 review of open issues. We do not feel this still poses enough of an inconvenience for this to be a neceesary feature.
Closed in 2025 review of open issues. We do not feel there is a large need for this feature.
Solved by jump_urls being attached to infractions.
Closed in 2025 review of open issues. We do not feel there is a large need for this feature.
Closed in 2025 review of open issues. We do not feel there is a large need for this feature.
Closed in 2025 review of open issues. We do not feel there is a large need for this feature.
Closed in 2025 review of open issues. We do not feel there is a large need for this feature.
Closed in 2025 review of open issues. We do not feel there is a large need for this feature.
Closed in 2025 review of open issues. We do not feel there is a large need for this feature.
95fc4cf Add dynamic module loading fix for dataclasses - janine9vn
[python-discord/forms-backend] New branch created: dataclass-futures-module-loading-fix
The code has the details commented for future reference.
Weird issue I would not have expected.
I tested locally and all seemed well, but this should be redeployed when either joe or myself are around to validate the fix works as expected in prod.
GitHub Actions run 16454020134 failed.
111c063 KUTIEKAT ALWAYS LINTS BEFORE PUSHING - jb3
[python-discord/forms-backend] branch deleted: dataclass-futures-module-loading-fix
GitHub Actions run 16454179132 succeeded.
GitHub Actions run 16454181797 succeeded.
GitHub Actions run 16454204436 succeeded.
GitHub Actions run 16454206374 succeeded.
GitHub Actions run 16454208675 succeeded.
GitHub Actions run 16454211056 succeeded.
GitHub Actions run 16454213474 succeeded.
GitHub Actions run 16454215330 succeeded.
GitHub Actions run 16454217405 failed.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.33.2
GitHub Actions run 16454220875 succeeded.
GitHub Actions run 16454222832 succeeded.
GitHub Actions run 16454225564 succeeded.
Closing this as we no longer actively seem to want this project.
Closed in 2025 review of open issues. We do not feel there is a large need for this feature.
Client-side type checking has got better nowadays. I agree we could potentially look at mypy but with tools like pyright being integrated into editors and having support for LSP I suggest we just bolster our recommendation of client-side tooling.
Interested in thoughts of our Core Devs now looking at this from a 2025 perspective?
Given this feature is being used less I don't know if the added complexity is worth it. We still offer users the link to delete it and they can resend if they need to fundamentally change what they linked to.
We're a little more on top of this now, so I'm not sure we need this. Even if we notify admins there are votes that will genuinely take a lot longer than 2 weeks (particularly when a candidate does not have a lot of activity).
!help tag
Query "tag" not found.
I suspect that turning tag into a hybrid command (if this is possible) should fix this?
I think this is niche and we are not seeing abuse of this, if people use this to get around filters we can implement a fix.
It wasn't impleneted as part of #1530 but it now very easy to do so by adding another formatting option to https://github.com/python-discord/bot/blob/c36a5df84f8f017a90d465928566fc8885ea7bcc/bot/exts/filtering/_settings_types/actions/infraction_and_notification.py#L173-L176
Connected!
GitHub Actions run 16458704907 succeeded.
Mautovoice (moderated auto-voice) would let new users bypass the voice gate by granting temporary permissions to un-gated users when a supervising moderator is present in a public voice channel. This will hopefully cut down on spamming and provide a better experience for people wanting to chat on the server.
How it should work:
- System activates when a
moderator-role member (not mod-team) joins a public voice channel and is not deafened. - Bot automatically grants temporary speaking permi...
My concerns around missing or outdated type stubs still remain. We could enable it on repositories on a case by case basis.
Hi @Snipy7374, just checking whether this is still a feature you'd like to implement?
You may need to pull in some updates from the
mainbranch, if you no longer have time that's also completely okay, just let us know!If you need any help or guidance feel free to drop a message in the
#dev-contribchannel.
if the feature request is still valid yeah, why not
Hi @Snipy7374, just checking whether this is still a feature you'd like to implement?
You may need to pull in some updates from the
mainbranch, if you no longer have time that's also completely okay, just let us know!If you need any help or guidance feel free to drop a message in the
#dev-contribchannel.if the feature request is still valid yeah, why not
Reopened now, as mentioned let us know in #dev-contrib if you need any help with the review comments or ...
GitHub Actions run 16467033874 succeeded.
Reopened now, as mentioned let us know in #dev-contrib if you need any help with the review comments or getting the branch up to date.
Thank you, I'll work on this tonight, hopefully there aren't too many conflicts.
GitHub Actions run 16477484460 succeeded.
You can go party every other day and stay up until 2 AM but when the Python Discord brotherhood needs you, it's "No time for now". Curious.
GitHub Actions run 16506418162 failed.
GitHub Actions run 16520848979 succeeded.
You can go party every other day and stay up until 2 AM but when the Python Discord brotherhood needs you, it's "No time for now". Curious.
Feel free to take it, my time constraint primarily revolves around the fact I don't know Django or DRF at all.
5 Years of Python Discord Django Addicts Anonymous, "Joe". 5 Years.
5 Years of Python Discord Django Addicts Anonymous, "Joe". 5 Years.
???????????????????
Closing for now as this looks to be for a concluded school project, and I'm assuming this won't be updated.
Moving forward with the migration to bot core would be nice so we can use this in Polonium.
[python-discord/bot] New branch created: jb3/3.13-upgrade
Upgrades Dockerfile & pyproject.toml to use 3.13.
Tested locally, bot starts without issue.
GitHub Actions run 16551529808 succeeded.
[python-discord/site] New branch created: jb3/3.13-upgrade
Upgrades tooling and contributing guides to Python 3.13.
GitHub Actions run 16551620322 failed.
GitHub Actions run 16551659408 succeeded.
target-version in the linting section of pyproject needs updating too, but looks good
target-version in the linting section of pyproject needs updating too, but looks good
GitHub Actions run 16551757002 succeeded.
GitHub Actions run 16551766680 succeeded.
There are some new warnings in the test CI
tests/bot/exts/test_cogs.py::CommandNameTests::test_names_dont_shadow
/home/runner/work/bot/bot/bot/exts/utils/extensions.py:29: FutureWarning: functools.partial will be a method descriptor in future Python versions; wrap it in enum.member() if you want to preserve the old behavior
RELOAD = functools.partial(Bot.reload_extension)
The fix should be the same as for lancebot https://github.com/python-discord/sir-lancebot/pull/1670/commits/...
[python-discord/site] branch deleted: jb3/3.13-upgrade
1546ba6 Update extensions functools.partial to enum.member - jb3
GitHub Actions run 16579933747 succeeded.
GitHub Actions run 16579881155 succeeded.
Connected!
GitHub Actions run 16582533042 succeeded.
Hi, this PR is practically deleting all files. Please state your intentions.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.33.2
GitHub Actions run 16603787999 succeeded.
I'm trying to extend the bot by adding a trade execution method (take_trade()) to the loop that fetches the latest candles. However, whenever I include these methods, the bot freezes after fetching one or two candles โ no errors are shown, and it stops printing or responding beyond that point.
โ
When I remove the take_trade() calls, the loop works perfectly.
But as soon as I add them (even with simple print statements inside), the bot freezes silently after the second loop.
I tried adding d...
what?????
I am not sure you have opened this issue on the correct GitHub repository. Nothing you have described is functionality in this project.
This is the community Discord bot for the Python Discord community.
I am not sure you have opened this issue on the correct GitHub repository. Nothing you have described is functionality in this project.
This is the community Discord bot for the Python Discord community.
bro iam unexpreased bignear so i dont now any rules i find lots of ways to solve my problem so then chat gpt sagest me this that hear maive i can get any help
This is not the correct place for this issue, locking it.
GitHub Actions run 16609855105 failed.
GitHub Actions run 16614325017 succeeded.
Site is very unhappy with this, check the deploy preview.
86cc15d Upgrade pyenv to 2.6.5 for access to latest Pyt... - ChrisLovering
abe9d24 Upgrade Python within snadbox to latest 3.13 & ... - ChrisLovering
35943c0 Add comment to explain why 3.13t can't be bumpe... - ChrisLovering
d0a5dd3 Close & unlick SharedMemory object in tests - ChrisLovering
10a80ee Update eval deps to latest versions - ChrisLovering
[python-discord/snekbox] New branch created: bump-to-latest
Why did this become necessary?
Or rather, was this because of the Python version upgrade?
Looks related to the new version yeah. It's from the resource_tracker, which the docs say (see track kwarg docs) was added in 3.13.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.34.0
GitHub Actions run 16621655868 succeeded.
006b79f Install fishhook now that it supports 3.13 - ChrisLovering
[python-discord/snekbox] branch deleted: bump-to-latest
86cc15d Upgrade pyenv to 2.6.5 for access to latest Pyt... - ChrisLovering
abe9d24 Upgrade Python within snadbox to latest 3.13 & ... - ChrisLovering
35943c0 Add comment to explain why 3.13t can't be bumpe... - ChrisLovering
d0a5dd3 Close & unlick SharedMemory object in tests - ChrisLovering
10a80ee Update eval deps to latest versions - ChrisLovering
GitHub Actions run 16623627166 succeeded.
snadbox? @mint goblet??
mark, I apologise
Please force push to main to rectify this error immediately. Thanks.
Lol, I also just noticed Close and unlick
GitHub Actions run 16637473743 succeeded.
GitHub Actions run 16649974986 succeeded.
For invocations of !pip or !d a negative result is still a useful result. Deleting the bot response and the invocation is undesired in these cases (the author still has the option to delete anyway).
This kinda also applies to the code running commands where if you accidentally run in the wrong channel your invocation is nuked and the code you typed is lost, but that might need to be handled differently by posting the code in the channel where the execution results goes.
Hey @n0Oo0Oo0b! Have you worked on this issue at all since your last comment? If so (or even if you haven't) and you're still interested in working on it, I'm free for the next few weeks until I start college and would love to work on it together if that's cool with you. Let me know. Thanks!
I'd go for something like "Random word"/"Choose for me"/"Auto-suggest" rather than "add word".
I plan to use the already existing hangman_words.txt file
Don't you need the word list to be categorized into types (like nouns, adjectives, etc)?
I'd go for something like "Random word"/"Choose for me"/"Auto-suggest" rather than "add word".
I'll go with "Choose for me". ๐
I plan to use the already existing hangman_words.txt file
Don't you need the word list to be categorized into types (like nouns, adjectives, etc)?
Technically yes, although I think it might be a bit difficult to do that because some of the template use very sp...
It just doesn't make sense to me for just the "React with ๐ to receive your own bookmark..." message to auto-delete when you run .bm with DM's off.
It's probably because there's a timeout on how long we (used to) listen for reactions (we use buttons now).
Just add delete_after=60 as a parameter to the ctx.send() function call here. Although I'm not sure if that's going to work, corr...
Relevant Issues
Link to discussion: #dev-contrib message
Description
Changed one line of the Madlibs code which was causing a "ValueError: zip() argument 2 is shorter than argument 1" and, as a result, not displaying the final Madlib. All I had to do is change the "strict" parameter in the zip() function call to False and that fixed the issue.
Did you:
- [x] Join the [Python Discord Community...
GitHub Actions run 16706059059 succeeded.
Motivation:
- The current .joke command supports chuck, neutral, and all. Adding dad jokes increases the variety without disrupting existing functions.
- This serves as the first open-source contribution
Proposal:
- Integrate icanhazdadjoke API, a free API for developers
- Extend .joke command with a new option: .joke dad
- No refactor or dependency required, no change to current logic
@DMFriends Sure, I'd be happy to work on it together! I only have a bit of uncommitted code for generating the sudoku board. How could we collaborate on this together?
I will invite you as a collaborator to my fork.
Connected!
GitHub Actions run 16712086636 succeeded.
@vivekashok1221 I thought a bit more about how to implement the random word button: do you think it would be acceptable if I asked ChatGPT to generate a list of 5-10 random words for each part of speech used across all of the Madlibs templates (I would attach the madlibs_templates.json file)? Then I would just create a new JSON file which would include each of the generated lists, which I would then load in the main program and then, when the button is clicked, choose a random word from the...
Personally I'm neutral on this issue since I don't think the embed sticking around is that big of a deal
Sure, it's certainly not a big deal -- I just feel like it makes sense to have some sort of auto-delete, especially if similar error messages in other cogs get auto-deleted, which may or may not be the case -- I'm not familiar with most of the other cogs in Lancebot.
That being said, it sounds like this isn't a necessary change, so I will close this issue.
[python-discord/sir-robin] New branch created: cj12-updates
These updates are based off of our feedback/improvement list from last year.
GitHub Actions run 16755264398 succeeded.
Connected!
GitHub Actions run 16761198656 succeeded.
I think adding multiplayer functionality similar to the duck game could also work pretty well. Something like +1 point for filling in a digit correctly / -1 point for an incorrect guess; once the whole grid is filled, the bot would display each user's total score.
Interesting idea. Let's see what the Core Devs think. @wookie184 thoughts?
Also can I be assigned to the issue again since I'm now working on this together with n0ob?
- Change PyPI link to GitHub
GitHub Actions run 16766517876 succeeded.
GitHub Actions run 16816299298 failed.
My cash machine is red
A JSON for the words seems fine, you can can come up with them however you like. No objections to the end game button
GitHub Actions run 16841736935 failed.
GitHub Actions run 16841768146 succeeded.
JavaScript...
GitHub Actions run 16843371533 succeeded.
GitHub Actions run 16843372825 succeeded.
GitHub Actions run 16843374645 succeeded.
GitHub Actions run 16843377534 succeeded.
GitHub Actions run 16843376282 succeeded.
GitHub Actions run 16843378736 succeeded.
GitHub Actions run 16843379790 failed.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.34.1
GitHub Actions run 16843381049 succeeded.
GitHub Actions run 16843382830 succeeded.
GitHub Actions run 16843384406 succeeded.
Relevant Issues
Closes #1000
Description
- Added an "End Game" button to end the game in case a user decides not to play after starting a game.
- Added a "Choose for me" button that randomly chooses a word from the
madlibs_word_bank.jsonfile and adds it to the Madlib in case the user can't think of a good word to use for the given part of speech. - As I was writing the code for the "Choose for me" button, I realized that it wouldn't make sense not to add a cooldown for th...
GitHub Actions run 16845333861 failed.
GitHub Actions run 16845469631 failed.
Relevant Issues
Closes #1000
Description
- Added an "End Game" button to end the game in case a user decides not to play after starting a game.
- Added a "Choose for me" button that randomly chooses a word from the madlibs_word_bank.json file and adds it to the Madlib in case the user can't think of a good word to use for the given part of speech.
- As I was writing the code for the "Choose for me" button, I realized that it wouldn't make sense not to add a cooldown for the bu...
GitHub Actions run 16852674038 succeeded.
GitHub Actions run 16852936150 succeeded.
GitHub Actions run 16853518740 succeeded.
Sorry it took me so long to respond, solid PR overall. I made a small improvement to make the code a bit cleaner (which fixed a small bug as a side effect)
Sorry it took me so long to respond, solid PR overall. I made a small improvement to make the code a bit cleaner (which fixed a small bug as a side effect)
Looks much nicer, thanks! Just out of curiosity, what was the bug?
you used .partition in the get function which returns three values, but unpacked into only two variables. In the other functions you used .split and [0] so there were no issues there
13975b4 Implemented optional duration parameter in slow... - b0nes1
db5ee77 WIP: Added tests for slowmode command with dura... - b0nes1
dd444fe Fix issue with unawaited coroutine in test - b0nes1
875c6d2 Fix bug in test causing channel edit to not be ... - b0nes1
a15262d Refactor slowmode command to use Duration conve... - b0nes1
GitHub Actions run 16853752855 succeeded.
Connected!
GitHub Actions run 16853756392 succeeded.
mkdir whatsapp-bot
cd whatsapp-bot
npm init -y
npm install whatsapp-web.js qrcode-terminal
Instructions unclear, Chris has been uploaded to the WhatsApp mainframe. He cannot escape.
[python-discord/bot-core] New branch created: chris-forgot-2fa-lol
e5394de extend regex for new discord shenanigans - fisher60
Chris is traveling abroad and is unable to access. Therefore, I have become his proxy. An 'apprentice' if you will. Shame because I have years more software engineering experience compared to him.
- r"(https?:\/\/)?(discord:\/{0,2})?" # Optional protocols
+ r"(https?:\/\/)?(discord:\/*)?" # Optional protocols
9b5e8e0 fix comment styling in comments - fisher60
[python-discord/bot-core] branch deleted: chris-forgot-2fa-lol
6b17286 Deploying to docs from @ python-discord/bot-cor... - ChrisLovering
yeah
[python-discord/bot-core] New tag created: v11.7.0
723ffff Deploying to docs from @ python-discord/bot-cor... - jb3
Connected!
GitHub Actions run 16866673023 succeeded.
Motivation:
- The current .joke command supports chuck, neutral, and all. Adding dad jokes increases the variety without disrupting existing functions.
- This serves as the first open-source contribution
Proposal:
- Integrate icanhazdadjoke API, a free API for developers
- Extend .joke command with a new option: .joke dad
- No refactor or dependency required, no change to current logic
GitHub Actions run 16894346517 failed.
GitHub Actions run 16895784999 succeeded.
[king-arthur] Branch dependabot/github_actions/actions/checkout-5 was force-pushed to `a9c88ea`
GitHub Actions run 16895836461 succeeded.
GitHub Actions run 16906861952 succeeded.
GitHub Actions run 16911722397 succeeded.
GitHub Actions run 16914743558 succeeded.
GitHub Actions run 16919030095 succeeded.
[python-discord/king-arthur] Checks Failed on PR: #330 Bump kubernetes-asyncio from 32.3.0 to 33.3.0
GitHub Actions run 16919656368 failed.
Relevant Issues
Description
Changes made:
- Extend the existing 'joke' command with 'dad' category using icanhazdadjoke API
- Keep the existing joke categories (chuck, neutral, all)
How it's implemented: - When dad joke is called, an asynchronous HTTP GET request is made to https://icanhazdadjoke.com with aiohttp
- Implement...
GitHub Actions run 16926132843 succeeded.
Connected!
GitHub Actions run 16940003365 succeeded.
[king-arthur] Branch dependabot/github_actions/actions/checkout-5 was force-pushed to `2bcef91`
GitHub Actions run 16942496077 succeeded.
GitHub Actions run 16942615652 succeeded.
Relevant Issues
Closes #985
Description
- Grid size: 6x6
.sudoku [difficulty]to start a game, send coordinate and digit to enter guesses (e.g.A1 1)- Difficulties: easy (13 digits filled), normal (11 digits filled), hard (9 digits filled)
- Each guess message will receive a reaction of either โ if correct or โ if incorrect. Correct guesses are automatically added to the board and digits are marked in green on the board.
- Number of correct/incorrect guess...
GitHub Actions run 16971744058 failed.
[python-discord/bot] New branch created: mbaruh/fix-modpings-schedule
- Fixes the modpings scheduler. Various bugs aside, key idea is that both caches are necessary to determine whether or not the moderator should currently have the role. Therefore almost all the checks, role changes, and scheduling are now under a central function
handle_moderator_state. - Adds the sync command in case there's some problem, and the status command to view current status, when pings come back, and what the current schedule is.
GitHub Actions run 16972656651 succeeded.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.34.1
GitHub Actions run 16978028802 succeeded.
Sentry Issue: BOT-436
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xf5 in position 14: invalid start byte
File "bot/exts/filtering/filtering.py", line 237, in on_message
await _extract_text_file_content(a)
File "bot/exts/filtering/filtering.py", line 73, in _extract_text_file_content
file_lines = file_content_bytes.decode(file_encoding).splitlines()
Unhandled exception in on_message.
`...
@swfarnsworth your input would be appreciated
@mbaruh I think this is the first time this has happened?
We could just add errors='ignore'.
Sentry Issue: BOT-3ZN
AttributeError: 'NoneType' object has no attribute 'channel_id'
(2 additional frame(s) were not displayed)
...
File "bot/exts/filtering/filtering.py", line 971, in _resolve_action
list_actions, list_message, list_triggers = await filter_list.actions_for(ctx)
File "bot/exts/filtering/_filter_lists/antispam.py", line 73, in actions_for
triggers = await sublist.filter_list_re...
GitHub Actions run 17067608960 succeeded.
Connected!
GitHub Actions run 17140034056 failed.
GitHub Actions run 17146366621 succeeded.
GitHub Actions run 17151754688 succeeded.
GitHub Actions run 17162396065 succeeded.
759e7a3 Bump matplotlib to minimum 3.10.5 in eval-deps - ChrisLovering
c23e8cd Bump the ci-dependencies group across 1 directo... - dependabot[bot]
Connected!
Connected!
GitHub Actions run 17241671934 succeeded.
[python-discord/king-arthur] Issue opened: #331 Allow 'partial matching' when searching for pod logs
Currently, when fetching logs of a pod, you have to submit the entire pod name as it is in the cluster
For example
M-x pods logs sir-lancebot-fuamfjzo7493n bots
This can be further simplified because we mostly have one pod running per service
So something like
M-x pods logs sir-lancebot bots
would be nice
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.35.0
GitHub Actions run 17243546653 succeeded.
We can already do this in a slightly different way. By using the deployment name instead you don't need ot know the exact pod name
M-x pods logs deploy/sir-lancebot bots
GitHub Actions run 17277077076 succeeded.
GitHub Actions run 17309163658 failed.
Doc item doc_item.symbol_id='discord.MediaGalleryItem' present in loaded documentation inventories not found on site, inventories may need to be refreshed.
For some commands such as !eval if executed in a non-snekbox category we redirect the output of the command and delete the invocation message.
This can be annoying for users when the original invocation cannot be recovered and to respond in #bot-commands with a new invocation is effortful.
Instead, we should amend the decorator that redirects the output of commands to #bot-commands to also upload the full invocation message to the paste service (using the util from pydis_core).
https://g...
Chris, this is some of your greatest work yet. Keep up the good work! :shipit:
I think we used a commit instead of a branch to minimise the amount of data we pull from the remote.
I can take a look @jb3 at this just gimme a day or two.
Wouldn't doing -b master--single-branch clone the entire master branch and then after that we checkout to a older commit.
Whereas the -b 3.4 --depth 1 only clones the single commit that is tagged with that name?
happy to change it back if I've misunderstood that
Sorry, I had it backwards. We originally had it as you wrote it now. It was changed to a commit because we wanted a specific unreleased version.
See the commit below and its associated PR description. https://github.com/python-discord/snekbox/commit/b1bcd3b04fb2a83fd7bd4fe930582ccf0b6bf627
When we update nsjail we should regenerate the protobuf code for its config. I believe our README explains this process.
When we update nsjail we should regenerate the protobuf code for its config. I believe our README explains this process.
Ah cool, didn't know about this, I'm assuming it's ./scripts/protoc.py 3.4 can look tomorrow unless joe gets it first
GitHub Actions run 17356123642 succeeded.
Fix for #3376
Added:
- Whenever a user sends a !eval command in a non-snekbox channel the redirect will include the original invocation command.
GitHub Actions run 17363140799 succeeded.
GitHub Actions run 17381585358 succeeded.
GitHub Actions run 17381834527 succeeded.
GitHub Actions run 17383229466 succeeded.
[python-discord/forms-backend] Checks Successful on PR: #296 Update dependency sentry-sdk to v2.35.1
GitHub Actions run 17389403507 succeeded.
GitHub Actions run 17389614992 succeeded.
It looks like these lines are indented an extra level.
This one line should be indented one level less.
This could be one line using a ternary condition.
@jb3, am I correct in my understanding that this copies messages from the server to our pastebin, possibly unexpectedly?
Would this potentially send an empty message?
Why are we changing the order and putting the mention first?
@jb3, am I correct in my understanding that this copies messages from the server to our pastebin, possibly unexpectedly?
Yes, but with it being a bot command I'm less worried about that. Previously we've worried about that when things such as automatically uploading unformatted code have been mentioned, or when we are talking about uploading messages which are not direct interactions with the bot.
In this case, it's reasonable to temporarily upload after a user has directly used our featu...
Perhaps we can send the user the link to delete the paste? You can largely copy the code here: https://github.com/python-discord/bot/blob/cb4401b45978d90bb53cdbe4c03d7b89747562b2/bot/exts/utils/attachment_pastebin_uploader.py#L130-L133
I wish the Gay/MLM flag to be added to the list of Pride emojis available in the server.
Example:
This would fit in with the rest of the other sexuality flags, such as the pan flag and lesbian flag.
Perhaps we can send the user the link to delete the paste? You can largely copy the code here:
I mean if they did a command in #python-discussion and then tried to find it later it would be hard, as that channel get lots of messages.
GitHub Actions run 17414251228 succeeded.
GitHub Actions run 17414305549 succeeded.
Do you want it changed? It can be done.
I don't have a strong preference, but I figure we should leave it alone if there is no compelling reason to change it.
Yeah this was probably something I changed inadvertedly when I wrote something on mobile, it can be changed back.
Should I change and commit then? Or should we just leave it as is?
Change it back, there's no reason to alter it from what already exists.
GitHub Actions run 17417866055 succeeded.
Resolved with the ternary changes.
Change log:
- Added alt-text for the Welcome Video in the homepage
- Added alt-texts of "" for images that does not affect people with screen-reader in the events page as well as the timeline page
- Changes alt-texts in the events page form "AoC" to "Advent of Code" so, people who don't have knowledge about it can still know what it is.
Fixes
- #1382
- #1381
- #1406
GitHub Actions run 17418014767 succeeded.
GitHub Actions run 17418314167 succeeded.
Perhaps we can send the user the link to delete the paste? You can largely copy the code here:
Added.
GitHub Actions run 17419911880 failed.
When a string is too long, the preferred format is like this
content = (
"Lorem ipsum dolor sit amet, consectetur "
f"adipiscing {elit}, sed do eiusmod tempor "
"incididunt ut labore et dolore magna aliqua."
)
This leverages implicit string concatenation. Note that segments using f-string need to start with an f. The trailing spaces at the end of each line are literal single spaces that are not line breaks.
GitHub Actions run 17420006927 succeeded.
Yeah, Just saw the lint errors, pushed a commit.
Formatting and wording.
# Send a DM to the user about the redirect and paste removal
await ctx.author.send(
f"Your command output was redirected to <#{Channels.bot_commands}>."
f" [Click here](<{paste_response.removal}>) to delete the pasted"
" copy of your original command."
)
Is this issue still up for grabs? Would it be a valid approach to split the cases and match Christmas ๐ and Tree ๐ฒ? If so, I would like to take on this one.
GitHub Actions run 17428262548 succeeded.
GitHub Actions run 17431128102 succeeded.
This line can be removed now.
This should be wrapped in a try/except so that we still run the command even if we can't DM the user (user has blocked the bot, etc.)