#dev-log

1 messages ยท Page 17 of 1

regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
VirdanTheBurden
[python-discord/bot] Pull request opened: #2773 feat: context menu for !raw

Closes Issue #2760.

The !raw command has been restricted to only prefix command usage since inception, and has not been touched since. With the introduction of new interfaces for executing commands (i.e. application context menus) and the inconvenience of copying message links to check their raw content, this pull request hopes to simplify the process by adding a context menu button for users to click.

odd spireBOT
regal archBOT
#
Xithrius
[python-discord/site] branch deleted: remove-the-redirect-filter-list
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
Robin5605
[python-discord/bot] New review comment on pull request #2773: feat: context menu for !raw

I'll be moving the message sending responsibilities to the... overlying(?) methods.
That sounds like a good idea, now that we have two different methods that send the message in two different ways (normal !raw command and context-menu command. Former needs to use ctx.send while the latter needs to use interaction.response.send_message). It would make sense to practice principle of least responsibility here and have one function that's just responsible for fetching the raw content.

odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] New branch created: remove-cooldown-role
odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/snekbox] New branch created: python-3.12
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
oak estuaryBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
#
MarkKoz
[python-discord/snekbox] branch deleted: python-3.12
regal archBOT
odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] New branch created: update-references-of-3.11-eval-to-3.12
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/snekbox] New comment on issue #158: Allow Evaluation Using Multiple Python Versions

The snekbox Dockerfile now installs both 3.12.0 and 3.13-dev to /lang/python/<version>.
There is also /lang/python/default which is a symlink to /lang/python/3.12/ which is what snekbox currently uses for eval.

Now we need to update snekbox's API to support multiple versions. The following is my opinion on how to do that.

A new endpoint GET /versions is added, which returns all versions snekbox supports. The response body would include the name, a human friendly name, and wheth...

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
#
sarzz2
[python-discord/site] New comment on issue #1037: Truncated Dropdown Menu Items in the Sub-Articles Section

I have identified the issue, it's due to the whitespace property of a.dropdown-item

Simple fix:-

  • Remove the whitespace:no-wrap property however that would cause the menu item to look like the ss below

If the options needs to be in a single line then multiple CSS properties would need to be changed I imagine as I could not get it to work by playing around with some of the element properties only

<img width="1355" alt="image" src="https://github.com/python-discord/site/assets/45...

regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
MarkKoz
[python-discord/snekbox] New comment on issue #158: Allow Evaluation Using Multiple Python Versions

@ChrisLovering This was essentially the approach taken in #175, which has some discussion.

I think it is better to accept an arbitrary path to a binary to execute. I'm hesitant to make design decisions that pigeon-hole snekbox into just a Python evaluator or just a code evaluator in general. Technically there is not much difference between that and sandboxing arbitrary binaries.

On the other hand, that is the common use case, so we can compromise here. I don't see a strong need for expo...

regal archBOT
odd spireBOT
regal archBOT
#
Xithrius
[python-discord/bot] branch deleted: update-references-of-3.11-eval-to-3.12
oak estuaryBOT
regal archBOT
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

oak estuaryBOT
odd spireBOT
regal archBOT
regal archBOT
oak estuaryBOT
#

Doc item doc_item.symbol_id='enum.EnumType.__getattr__' present in loaded documentation inventories not found on site, inventories may need to be refreshed.

regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
#
sarzz2
[python-discord/site] New comment on issue #1015: Discord Button Shifts Off-Page at 1024-1040px Screen Width

Hello @jchristgit So if we try to implement the first solution "Collapse to using a hamburger menu a bit sooner." would be a bigger/more difficult change since the site is using Bulma to decide when to use a hamburger menu.
We would need to override various existing Bulma CSS classes, I have tried and haven't been able to do so successfully yet.

My proposal:-
Right now the discord button on the top right(which is causing the issue) has the following properties(in base.css):-

regal archBOT
regal archBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
oak estuaryBOT
regal archBOT
regal archBOT
#
kosayoda
[python-discord/site] Pull request review submitted: #1121 removing margin-left property from base.css to fix the overflow

I tested the preview deployment, and unfortunately the issue is not fixed. The issue is still reproducible on:

  • Stock chromium

  • Firefox, with persistent scrollbars General > Browsing > Always show scrollbars:

Given how finnicky the issue is, I'm inclined to suggest the removal of both padding-left and padding-right directly, which fixes the issue in all my test cases. It is a visual change, but I don't think it hurts the look:

regal archBOT
#
sarzz2
[python-discord/site] New comment on pull request #1121: removing margin-left property from base.css to fix the overflow

I tested the preview deployment, and unfortunately the issue is not fixed. The issue is still reproducible on:

  • Stock chromium

<img alt="image" width="721" src="https://user-images.githubusercontent.com/41782385/273406615-74fe62a2-afae-4ed9-950e-38a64588b889.png">

  • Firefox, with persistent scrollbars General > Browsing > Always show scrollbars:

<img alt="image" width="647" src="https://user-images.githubusercontent.com/41782385/273406685-6b7147e2-a075-48b8-91e3-d...

regal archBOT
#
shenanigansd
[python-discord/sir-lancebot] New branch created: shenanigansd-patch-1
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
#
kosayoda
[python-discord/site] branch deleted: fix-CJ-mobile-banner
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
oak estuaryBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
odd spireBOT
oak estuaryBOT
regal archBOT
#
ChrisLovering
[python-discord/snekbox] New branch created: multi-version-api
#
ChrisLovering
[python-discord/snekbox] Pull request opened: #196 Multi version api

This allows setting a binary_path key in the request body to /eval to specify which binary to run under.

If not supplied, it defaults to "/lang/python/default/bin/python"
if supplied, snekbox ensures that the path is within /lang (to avoid executing arbitrary binaries on the fs) and that the file exists.

This PR also comes with two unrelated changes

  1. https://github.com/python-discord/snekbox/commit/84598033acb13e1300d2e4ffc2b556fce906edd8 was needed as previous we were dro...
regal archBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

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

Connected!

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

Connected!

odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
oak estuaryBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
MarkKoz
[python-discord/snekbox] New review comment on pull request #196: Multiple binary support api

Oh I missed this in your description

if supplied, snekbox ensures that the path is within /lang (to avoid executing arbitrary binaries on the fs) and that the file exists.

It is fairly trivial to bypass this restriction by using the execve syscall. We cannot block this syscall either (at least not through nsjail's seccomp filters). That is due to a quirk in which the filter is applied before nsjail's own execve call, causing nsjail to commit suicide.

Granted, there may be use c...

odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
oak estuaryBOT
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/workers] branch deleted: bubble-up-upstream-discord-errors
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
thin oysterBOT
#
Sir Robin

Connected!

regal archBOT
oak estuaryBOT
regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/bot-core] branch deleted: Update-deps
regal archBOT
regal archBOT
#
Robin5605
[python-discord/bot] Pull request opened: #2795 Fix `!src` command resulting in 404 for tags

Closes #2770

The !src would give a GitHub link that's invalid for tags. For instance, this is the link given by !src enumerate compared to what it should've been:

  • https://github.com/python-discord/bot/blob/main/resources/tags/enumerate.md
  • https://github.com/python-discord/bot/blob/main/bot/resources/tags/enumerate.md

Note the difference. I think this is because of the path being relative to bot:
https://github.com/python-discord/bot/blob/66c8414396b28a7cac8f05602ee7f2b...

odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
Xithrius
[python-discord/bot] branch deleted: remove-cooldown-role
odd spireBOT
oak estuaryBOT
thin oysterBOT
#
Sir Robin

Connected!

odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] branch deleted: swfarnsworth/title-command-for-op
odd spireBOT
odd spireBOT
regal archBOT
#
3ddP
[python-discord/sir-lancebot] Issue opened: #1391 addition of .hsp command to lancebot

Description

The .hsp [text] command would take the provided text, split it into 3 roughly equal sized chunks, and append each of the chunks to []aha []tinky []oop in the places indicated by the brackets. Then, the resulting text would be posted on pydis.

Examples:
.hsp RFC (expected output: Raha Ftinky Coop)
.hsp Edd (expected output: Eaha Dtinky Doop (or Eaha Dinky Doop; see additional details))
.hsp intoxicant (expected output: intaha oxitinky cantoop)

Reasoning

We often...

regal archBOT
#
janine9vn
[python-discord/site] New branch created: cj10-finalize
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
janine9vn
[python-discord/sir-robin] Issue opened: #97 AoC: Convert commands to hybrid slash commands (ephemral where reasonable)

The following commands should be converting to hybrid slash commands:

  • daystar
  • leaderboard
  • global leaderboard
  • stats
  • countdown

The following commands should be converted to slash-only and be ephemeral:

  • connect/link
  • disconnect/unlink

The following commands should only be able to be run in advent-of-code-bot-commands, sir-lancebot-playground, or bot-commands. If they are tried to run elsewhere, they should redirect to advent-of-code-bot-commands and let the user...

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

Connected!

odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
#
Robin5605
[python-discord/sir-robin] New comment on issue #97: AoC: Convert commands to hybrid slash commands (ephemeral where reasonable)

@n0Oo0Oo0b would you like to split this issue up into tasks? There seems to be a lot of commands / tasks involved and I'd be willing to take up some of that work if you don't mind.

Sure, how should I split the tasks?

@n0Oo0Oo0b I think splitting it up the same way Kat's done it should work. We could make PRs for each of them, so they can be reviewed more effectively. For instance, a PR titled Convert &daystar command into a hybrid command. This way we can get multiple people w...

regal archBOT
#
sarzz2
[python-discord/site] Pull request opened: #1138 Fixed truncated dropdown menu

fixes #1037

Summary

In the "Sub-Articles" section of the "Contributing" page, when the dropdown menu is expanded, some section names are cut off at the right edge of the page making user scroll horizontally.

Description of changes

removed nowrap property and adjusted CSS of the subarticle to make it fit on the page

I confirm I have:

odd spireBOT
regal archBOT
regal archBOT
#
MarkKoz
[snekbox:main] 8 new commits

5637636 Refactor modules into subpackages - MarkKoz
3909703 Move iter_lstrip to separate utils module - MarkKoz
c85bc73 Refactor creation of NsJail args list into a se... - MarkKoz
64ca95c Refactor writing of input files into a separate... - MarkKoz
2c58889 Refactor attachment parsing into a separate fun... - MarkKoz

#
MarkKoz
[python-discord/snekbox] branch deleted: refactor-2
regal archBOT
regal archBOT
oak estuaryBOT
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
kwzrd
[python-discord/bot] New comment on issue #2099: Add branding assets to seasonal announcements in #changelog

I have explored the proposed approaches.

The easiest option is to include the icon in the embed thumbnail:

image

Another option is to use the author field. However, the author also needs a name, in addition to the icon. This opens a nice opportunity to move the event's name there, but the problem is that we currently include the event name in the body of the description:

![image](https:/...

regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
oak estuaryBOT
regal archBOT
#
sentry-io[bot]
[python-discord/bot] Issue opened: #2808 Branding cog - RuntimeError: Failed to fetch file due to status: 503

Sentry Issue: BOT-3MY

RuntimeError: Failed to fetch file due to status: 503
  File "bot/exts/backend/branding/_repository.py", line 204, in get_events
    instance = await self.construct_event(event_directory)
  File "bot/exts/backend/branding/_repository.py", line 179, in construct_event
    meta_bytes = await self.fetch_file(contents["meta.md"].download_url)
  File "bot/exts/backend/branding/_repository....
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
oak estuaryBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
oak estuaryBOT
regal archBOT
#
ChrisLovering
[python-discord/metricity] New branch created: bump-deps
odd spireBOT
oak estuaryBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
oak estuaryBOT
regal archBOT
odd spireBOT
oak estuaryBOT
odd spireBOT
odd spireBOT
regal archBOT
#
swfarnsworth
[python-discord/bot] New branch created: swfarnsworth/tag-updates
#
swfarnsworth
[python-discord/bot] Pull request opened: #2817 Tag updates

This PR updates two tags and introduces a new one about comparisons to True and False.

Regarding the new tag, I'm open to suggestions (and destructive criticism) about using vertical space more sparingly in the code section. I don't like having my_func(user_input) four times, but I'm worried that having ... on the same line as the if statements will be confusing for beginners.

odd spireBOT
regal archBOT
#
swfarnsworth
[python-discord/bot] New comment on pull request #2797: New Tag: "Avoid System Python for Development"

Thank you for writing this and opening this PR.

It's important that tags be as terse as possible, so as to drive a very specific point before losing the reader's attention. It's also important that the instructions are easy to follow for the target audience.

If we go forward with this tag, I'd like to first condense it significantly, to something along these lines:

Operating systems often come with a system Python interpreter that is used internally. While it may be suitable for ex...

regal archBOT
#
BruDriguezz
[python-discord/bot] New comment on pull request #2797: New Tag: "Avoid System Python for Development"

Thank you for writing this and opening this PR.

It's important that tags be as terse as possible, so as to drive a very specific point before losing the reader's attention. It's also important that the instructions are easy to follow for the target audience.

If we go forward with this tag, I'd like to first condense it significantly, to something along these lines:

Operating systems often come with a system Python interpreter that is used internally. While it may be s...

odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
jchristgit
[python-discord/site] branch deleted: cj10-finalize
regal archBOT
oak estuaryBOT
night lilyBOT
#
Sir Lancebot

Connected!

thin oysterBOT
#
Sir Robin

Connected!

night lilyBOT
#
Sir Lancebot

Connected!

thin oysterBOT
#
Sir Robin

Connected!

oak estuaryBOT
odd spireBOT
regal archBOT
regal archBOT
#
janine9vn
[python-discord/sir-robin] Pull request review submitted: #101 Turn AoC commands into hybrid commands

So an issue with the current approach, is that &aoc link still works as a regular command and is not ephemeral only. I would like to convert this completely over to a slash-only is possible. If it's not the most feasible then that's fine.

Additionally, with the hybrid group, I would prefer the prefix to be /aoc instead of the wordy /adventofcode

For the channel restrictions, that's not working because we're missing code that wraps the extensions with the whitelist command.
Sir-Lancebo...

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

Connected!

odd spireBOT
oak estuaryBOT
odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
oak estuaryBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
oak estuaryBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
oak estuaryBOT
regal archBOT
#
ChrisLovering
[python-discord/sir-lancebot] New branch created: bump-deps
#
ChrisLovering
[python-discord/sir-lancebot] Pull request opened: #1400 Bump deps

Description

This bumps deps to latest, and uses the new logging util from bot-core. Copied the same thing as what I did in https://github.com/python-discord/bot/pull/2766

Did you:

odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/sir-lancebot] branch deleted: bump-deps
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
#
ChrisLovering
[python-discord/sir-lancebot] branch deleted: upgrade-pydantic-2
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
#
ChrisLovering
[python-discord/sir-lancebot] New branch created: remove-manual-dotenv-call
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
#
ChrisLovering
[python-discord/sir-lancebot] branch deleted: remove-manual-dotenv-call
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
thin oysterBOT
#
Sir Robin

Connected!

regal archBOT
#
ChrisLovering
[python-discord/sir-robin] branch deleted: migrate-constants-to-pydantic-settings
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/sir-robin] New branch created: bump-deps
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/sir-robin] branch deleted: bump-deps
thin oysterBOT
#
Sir Robin

Connected!

regal archBOT
regal archBOT
regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/bot-core] New branch created: bump-deps
odd spireBOT
thin oysterBOT
#
Sir Robin

Connected!

regal archBOT
#
ChrisLovering
[python-discord/sir-robin] New branch created: only-debug-log-if-debug-is-enabled
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/sir-robin] branch deleted: only-debug-log-if-debug-is-enabled
thin oysterBOT
#
Sir Robin

Connected!

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
thin oysterBOT
#
Sir Robin

Connected!

regal archBOT
#
janine9vn
[python-discord/sir-robin] New branch created: sir-robin-touchups
#
janine9vn
[python-discord/sir-robin] Pull request opened: #105 Sir Robin Error Handling & Help Command updates

This PR updates and touches up the &help command output and some of the error handling. In addition, it also fixes how the channel whitelist works.

  • Help command cog is shamelessly stolen from Sir Lancebot
  • Error handling is updated to catch some of the new errors being thrown
  • This unifies how we do the channel whitelisting and simplifies it as well
  • Some general reorganization and cleanup of unused items
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
thin oysterBOT
#
Sir Robin

Connected!

thin oysterBOT
#
Sir Robin

Connected!

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/sir-robin] New branch created: more-check-failure-handling
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/sir-robin] branch deleted: sir-robin-touchups
#
ChrisLovering
[python-discord/sir-robin] branch deleted: more-check-failure-handling
#
janine9vn
[python-discord/sir-robin] New branch created: sir-robin-touchups
thin oysterBOT
#
Sir Robin

Connected!

odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
thin oysterBOT
#
Sir Robin

Connected!

regal archBOT
odd spireBOT
regal archBOT
#
n0Oo0Oo0b
[python-discord/sir-robin] Issue opened: #107 AoC: put specified user within the top 10 leaderboard if user is in the top 10

Currently, when someone in the top 10 runs &aoc lb, their score gets moved to the top of the leaderboard:

image

I think it would make more sense for it to show up in the respective location within the top 10 instead.

It may also be worth considering displaying users outside the top 10 at the bottom, along with the users placed immediately before and after the user, like so:

...
thin oysterBOT
#
Sir Robin

Connected!

thin oysterBOT
#
Sir Robin

Connected!

regal archBOT
#
janine9vn
[python-discord/sir-robin] New branch created: aoc-c-everywhere
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
Robin5605
[python-discord/sir-robin] New comment on pull request #101: Turn AoC commands into hybrid commands

Pushed commit https://github.com/python-discord/sir-robin/pull/101/commits/619a1db4de522892e6b2c6d421f25846c50d601d that makes link/unlink commands slash-command only as requested by Kat.

Would like some feedback here on the technical side - I've used the app_command property of discord.ext.commands.HybridGroup, in order to make a slash-only command under the existing hybrid group.
Ho...

odd spireBOT
regal archBOT
#
janine9vn
[python-discord/sir-robin] New branch created: aoc-lb-fix
odd spireBOT
regal archBOT
#
Xithrius
[python-discord/sir-robin] branch deleted: aoc-lb-fix
#
Xithrius
[python-discord/sir-robin] branch deleted: aoc-c-everywhere
thin oysterBOT
#
Sir Robin

Connected!

regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/sir-robin] New branch created: use-new-pasteservice
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
oak estuaryBOT
regal archBOT
oak estuaryBOT
regal archBOT
#
n0Oo0Oo0b
[python-discord/sir-lancebot] New comment on issue #985: Sudoku command

@n0Oo0Oo0b Hello. Just wondering if there has been any progress on this feature. I'd like to see it get implemented. Thanks!

I did have some progreess on the board generation but this has been on my backburner for a while. I'll be able to work on it soon once my exams are over and I'll open a draft PR once the board generation is done.

regal archBOT
#
shenanigansd
[python-discord/sir-lancebot] New comment on issue #1403: Festive reactions on links

Here is the code in question: https://github.com/python-discord/sir-lancebot/blob/54d618da59e63936d6d9c6ab8fed5b325112289a/bot/exts/holidays/holidayreact.py#L64

It's matching on christams|tree, so it will match on every tree, regardless of it's festivity. It'll even match on the Grinch's trees! This is obviously extremely worrisome, we don't want the Grinch anywhere near our festivities!! This is a MUST FIX issue!!!

<details>

<summary>regex101 screenshots</summary>

![image...

#
shenanigansd
[python-discord/sir-lancebot] New branch created: shenanigansd-patch-1
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
Vxnku
[python-discord/bot] Issue opened: #2832 Discord.gateway warning "Shard ID None heartbeat blocked for more than 10 seconds."

I don't understand python and discord bots. This code was written for me, a person who can no longer maintain contact with anyone at all. The error itself:

Loop thread traceback (most recent call last):
  File "C:\.HgBots\HG_bot_roles-main/main.py", line 877, in 
    client.run(discord_token)
  File "C:\Users\User\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.11_qbz5n2kfra8p0\LocalCache\local-packages\Python311\site-packages\discord\client.py", line 860, in run
    asynci...
regal archBOT
regal archBOT
#
VirdanTheBurden
[python-discord/sir-lancebot] New comment on pull request #1404: Add a space before "tree" in the "christmas" tree holiday reaction

Are we certain we don't mind most trees? A lot of trees can be stated that have no relation to the season itself. "Grinch's tree", sure, but then what about "skeleton tree"? That will still react with a tree; the regex matches, but that's for halloween.

If we're going to activate it for Christmas, I'd rather you match for christmas/xmas then lookahead for tree.

regal archBOT
#
sentry-io[bot]
[python-discord/bot] Issue opened: #2833 HTTPException: 400 Bad Request (error code: 50035): Invalid Form Body

Sentry Issue: BOT-3P5

The snekbox cog does not account for Discord's message length restriction when it generates the list of blocked files to append to the message.

HTTPException: 400 Bad Request (error code: 50035): Invalid Form Body
In content: Must be 2000 or fewer in length.
(4 additional frame(s) were not displayed)
...
  File "bot/decorators.py", line 133, in inner
  File "bot/exts/utils/snekbox/_c...
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

oak estuaryBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/metricity] branch deleted: bump-deps
regal archBOT
#
ChrisLovering
[python-discord/quackstack] New branch created: bump-deps
#
ChrisLovering
[python-discord/quackstack] branch deleted: bump-deps
odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

oak estuaryBOT
regal archBOT
odd spireBOT
regal archBOT
#
shtlrs
[python-discord/bot-core] Issue opened: #198 Centralize error handling

Problem

Currently, all of the error handling is usually done through an ErrorHandler Cog.

This cog will only handle errors raised from Text Commands and will not catch errors raised from App Commands (Slash Commands and Context Menu Commands)

Solution

The way we can centralize handling for App Commands would ideally be by subclassing the CommandTree class and override the on_error method, then upon instantiating the bot, will pass this class as an argument to...

regal archBOT
regal archBOT
regal archBOT
odd spireBOT
oak estuaryBOT
regal archBOT
regal archBOT
#
jchristgit
[python-discord/site] New branch created: lazy-load-footer-images
odd spireBOT
regal archBOT
#
jchristgit
[python-discord/site] New branch created: fix-broken-netcup-icon-in-footer
odd spireBOT
regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/site] branch deleted: lazy-load-footer-images
thin oysterBOT
#
Sir Robin

Connected!

regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT