#dev-log

1 messages · Page 98 of 1

regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
MarkKoz
[python-discord/snekbox] New branch created: pip\-tools
#
MarkKoz
[python-discord/snekbox] Pull request opened: #139 Use pip\-tools instead of Pipenv

Pipenv was limited to only "normal" and "dev" dependencies. pip-tools is more flexible, since each requirements file can be constrained by more than one other requirements file. This means CI can choose to only install coverage deps in the test job and linting deps in the lint job, but these dependencies will still be constrained by each other and by other requirements, so a developer will be able to install both sets simultaneously without conflicts.

Use a Makefile to make up for the loss...

#
HassanAbouelela
[python-discord/bot-core] Pull request opened: #79 Upgrade Docs

This PR upgrades our docs in two aspects:

  • Restores release-based changelogs (ed4b2fb). They were originally removed due to not wanting to maintain a list of versions, but we ended up doing that anyways. This is nicer for users than the changelog.md file we are maintaining
  • Adds sphinx-multiversion (4c9cad2). This is the big one. I still need to get the CI working for it, but that should be much easier now. I'll probably have it only run on main when deploying to prod, so we can keep th...
#
ChrisLovering
[python-discord/sir-lancebot] New review comment on pull request #1049: Bookmark enhancements

When you book mark a command, the link to that message is sent to you in DM. The best UX is to allow users to delete the message directly from the same place.

The UX issue described below is related to the help command, rather than this command directly.

We have two options to fix that, We could allow the help command to be ran in DMs, which leads to the screenshot I posted about below, or we allow the bookmark delete command to also be ran in server, so at least users can get the help ...

regal archBOT
#
CyberCitizen01
[python-discord/sir-lancebot] Pull request opened: #1060 Minor changes for consistency

Relevant Issues

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

Description

Until now I've done nothing much but made a little change and setup the bot. I plan on doing the following:

  • Handle random page number such that it is always less than 500.
  • Update log messages to include status code of the response and also handle the situation when response doesn't include status_code and/or status_message.

Did you:

...

odd spireBOT
regal archBOT
regal archBOT
#
wookie184
[python-discord/sir-lancebot] New comment on pull request #1049: Bookmark enhancements

IMO it's even more confusing if we allow it in DMs, since the Help command only shows the command directly, which makes users think they can run &delete directly

Looked into that bug and it seems to be because specifying root_aliases causes bookmark delete to be registered as a command rather than subcommand, which causes it to be listed even though subcommands usually aren't, and this isn't done correctly (this isn't specific to DMs).

Tentative patch that seems to fix it:

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

Connected!

odd spireBOT
regal archBOT
regal archBOT
#
wookie184
[python-discord/sir-lancebot] New comment on pull request #1053: replace image\_url with source\_image\_url in spookygif\.py

@Xithrius Sorry, I am unable to set up the project due to some issues. So I can't test it locally before committing. I will not be able to work on this.

Alright, i'll close this PR and set the original Issue as up for grabs again. If you're interested in trying to set up the project in the future feel free to join the discord server and we can help with any issues.

regal archBOT
#
MarkKoz
[python-discord/snekbox] branch deleted: pip\-tools
regal archBOT
#
ChrisLovering
[python-discord/bot-core] branch deleted: upgrade\-docs
regal archBOT
odd spireBOT
regal archBOT
#
MarkKoz
[python-discord/snekbox] New branch created: pyproject
regal archBOT
regal archBOT
#
ChrisLovering
[bot:main] 4 new commits

e08cd9c Updating and improving clarity in help document... - Autonymic
c3b3a42 Merge branch 'python-discord:main' into docs/cl... - Autonymic
597ad57 Update clean regex description with cleaner exa... - Autonymic
80a5b83 Merge pull request #2179 from Autonymic/docs/cl... - ChrisLovering

oak estuaryBOT
odd spireBOT
regal archBOT
oak estuaryBOT
regal archBOT
regal archBOT
regal archBOT
#
MarkKoz
[python-discord/snekbox] Pull request opened: #140 Add pyproject\.toml and versioning

Summary:

  • Add a pyproject.toml using setuptools as the PEP 517 build backend
  • Use setuptools's experimental PEP 621 support to define metadata in pyproject.toml
  • Version the package using a CalVer variant which is derived from the current commit's date
  • Use the version instead of the SHA to tag containers and the Sentry release
  • Make Gunicorn and Sentry SDK optional dependencies (installed via extras)

The addition of versioning is progress towards #138. Originally I planned...

regal archBOT
regal archBOT
#
wookie184
[python-discord/branding] New branch created: larger\-intersex\-flag
odd spireBOT
regal archBOT
#
minalike
[python-discord/branding] branch deleted: larger\-intersex\-flag
odd spireBOT
regal archBOT
regal archBOT
#
MarkKoz
[python-discord/snekbox] branch deleted: pyproject
regal archBOT
#
MarkKoz
[python-discord/snekbox] New branch created: bug/ci/version\-commit\-count
#
MarkKoz
[python-discord/snekbox] Pull request opened: #141 Fix version commit count

The commit count was off by 1 since the last newline was being stripped and thus not counted. In CI, the checkout action was not retrieving history, so the commit count was not able to get the right number of commits. I set it to fetch 100 commits, which should be more than reasonably safe; it would take over 100 commits on the release day to cause the version to be incorrect, and even then it's only really a problem if a second release happens on the same day.

#
MarkKoz
[python-discord/snekbox] branch deleted: bug/ci/version\-commit\-count
#
MarkKoz
[python-discord/snekbox] New branch created: bug/ci/sentry\-release\-tag
#
MarkKoz
[python-discord/snekbox] branch deleted: bug/ci/sentry\-release\-tag
regal archBOT
regal archBOT
#
MarkKoz
[python-discord/snekbox] New branch created: feat/tests/wsgi\-app\-from\-cfg
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
HassanAbouelela
[python-discord/snekbox] branch deleted: feat/tests/wsgi\-app\-from\-cfg
regal archBOT
#
Robin5605
[python-discord/sir-robin] Pull request opened: #45 Add AoC task functionality

This is meant to be a proof of concept, and get the basic functionality working. There are a few things that could be improved upon (better error messages, use embeds instead of regular messages for info messages, better info messages, etc), and some features such as restricting the command to certain users/roles (admins/owners) have been left out for the time being, as I do not have enough information to code that up. However, that should be relatively easy. I welcome any and all suggestions...

regal archBOT
#
MarkKoz
[python-discord/snekbox] New branch created: black
regal archBOT
regal archBOT
#
3ddP
[python-discord/sir-lancebot] Issue opened: #1062 improving the \.latex command

Description

When one uses the .latex command with lancebot, the bot replies with a rendered image of the tex commands. The requested features are:

  • when the original .latex message is edited, lancebot deletes its previous message with the rendered tex, and replies with the updated version
  • lancebot's replies can be interacted with by using specific reactions. For example, if one reacts with an x emoji, the message with the rendered tex is deleted.

Reasoning

The texit bot offe...

regal archBOT
#
HassanAbouelela
[python-discord/branding] branch deleted: allow\-more\-banners
odd spireBOT
regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
regal archBOT
#
MarkKoz
[python-discord/snekbox] branch deleted: black
#
MarkKoz
[python-discord/bot] New branch created: enhance/back/1929/allowed\-strings\-literal
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
Numerlor
[python-discord/bot] Pull request opened: #2187 Filter out all header links in signatures for docs

Replacing the paragraph sign on the embed string, which was used previously, missed some other symbols that were used for permalinks, like the # here

The replace was previously done on the description too, but from a quick look I haven't found anything that'd use that in the description, may have been left over from when other definitions were also sent in the description. If it pops up somewhere the same filtering can be added to the description tags

odd spireBOT
regal archBOT
night lilyBOT
#
Sir Lancebot

Connected!

regal archBOT
#
MarkKoz
[python-discord/snekbox] New branch created: feat/nsjail/104/customise\-output\-limit
regal archBOT
regal archBOT
regal archBOT
#
MarkKoz
[snekbox] Branch feat/nsjail/104/customise\-output\-limit was force-pushed to `9541d08`

Compare changes

#
MarkKoz
[snekbox] Branch feat/nsjail/104/customise\-output\-limit was force-pushed to `25dbf25`

Compare changes

regal archBOT
odd spireBOT
regal archBOT
regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[bot:main] 8 new commits

5da7bdf Unpin all messages when help channel goes dormant - dementati
043cc65 Merge branch 'main' into feature/1903/fix-pin-i... - dementati
2f9c96c Merge branch 'main' into feature/1903/fix-pin-i... - dementati
a9a9e7b Merge branch 'main' into feature/1903/fix-pin-i... - dementati
19e7bc3 HelpChannels: pass message directly to pin_wrapper - MarkKoz

#
ChrisLovering
[python-discord/bot] branch deleted: feature/1903/fix\-pin\-inconsistency
oak estuaryBOT
odd spireBOT
regal archBOT
#
MarkKoz
[snekbox] Branch feat/nsjail/104/customise\-output\-limit was force-pushed to `c9bfff8`

Compare changes

#
ChrisLovering
[python-discord/bot] branch deleted: improve\-pastebin\-error\-handling
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
#
MarkKoz
[python-discord/bot] branch deleted: enhance/back/1929/allowed\-strings\-literal
odd spireBOT
oak estuaryBOT
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
oak estuaryBOT
odd spireBOT
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

thin oysterBOT
#
Sir Robin

Connected!

oak estuaryBOT
odd spireBOT
regal archBOT
#
MarkKoz
[python-discord/bot-core] Issue opened: #85 Port modules from bots

Let's discuss what should be ported over. Here is what I have thought of so far:

  • Extension management
  • Ping command
  • Source command (maybe?)
  • Help command
  • Internal eval
    • Including paste service utilities
    • Base class for use with eval-like cogs (snekbox, internal eval, latex)
  • Error handlers
  • Logging utils
odd spireBOT
regal archBOT
regal archBOT
#
DMFriends
[python-discord/sir-lancebot] New comment on issue #985: Sudoku command

Quick update on this: I am still working on this, just haven't opened a PR yet - I don't think I'm ready to do that yet. I'm planning on getting it to a state where I will feel comfortable opening a PR sometime between now and the end of August. (Yes I know that's a large timeframe, but this turned out to be a lot harder than I initially thought but I like the challenge so I'm going to keep working on this for now.)

regal archBOT
#
mbaruh
[python-discord/bot] New branch created: clean\_improv
#
mbaruh
[python-discord/bot] Pull request opened: #2189 Clean improvments
  • Fixed a bug where clean between deleted the last message. Clarified in the appropriate command descriptions that the clean limits are exclusive.
  • Added a shortcut to a common use case of deleting all recent public messages of a user. Since the purge alias was never used for the clean group I repurposed it for this new command.
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
Numerlor
[python-discord/bot-core] Issue opened: #87 Provide command decorator for blocking duplicated command output

In certain situations, multiple users invoking the same command with static output is common, resulting in duplicate messages from the bot, for example linking the same doc page.

The bot currently handles a per channel cooldown on tags, but commands always run.
I propose adding a decorator that can be applied to static output commands (e.g. doc, rules etc.) which will cause a cooldown to be applied depending on the command arguments, or maybe some output method that returns equal object...

regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
MarkKoz
[python-discord/bot] New comment on pull request #2189: Clean improvments

This argument can be None, so it should be annotated as such https://github.com/python-discord/bot/blob/0bbcde6af81405380bbae1eca3556338331b03fa/bot/exts/moderation/clean.py#L244=

There is an issue with the annotation, but it's actually different. after is the one which isn't optional, while before is. This is enforced by _validate_input

Technically both are optional since they're just passed to history(), and that function allows both to be None.

#
mbaruh
[python-discord/bot] New comment on pull request #2189: Clean improvments

If I write a function which takes a number and adds 1 to it, I could assign a default value of 0 to the number, but I have no reason to. That's not how I want the function to behave. If I don't supply it a number I want it to yell at me, not because of what the function can do in practice if I allow it, but by my choice.

regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
mbaruh
[python-discord/bot] branch deleted: clean\_improv
oak estuaryBOT
odd spireBOT
odd spireBOT
regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] New branch created: auto\-bump\-nomination\-threads
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
mbaruh
[python-discord/bot] New branch created: fix\_invite\_autoban
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
sentry-io[bot]
[python-discord/forms-backend] Issue opened: #175 TypeError: backend\.models\.form\.Form\(\) argument after \*\* must be a mapping, not NoneType

Sentry Issue: FORMS-BACKEND-H

TypeError: backend.models.form.Form() argument after ** must be a mapping, not NoneType
(9 additional frame(s) were not displayed)
...
  File "starlette/routing.py", line 266, in handle
    await self.app(scope, receive, send)
  File "starlette/endpoints.py", line 42, in dispatch
    response = await handler(request)
  File "spectree/spec.py", line 166, in async_...
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] branch deleted: auto\-bump\-nomination\-threads
#
ChrisLovering
[python-discord/bot] branch deleted: fix\_invite\_autoban
oak estuaryBOT
odd spireBOT
regal archBOT
#
swfarnsworth
[python-discord/site] New branch created: swfarnsworth/resources
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] New branch created: allow\-char\-info\-in\-all\-channels\-bar\-py\-gen
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] New branch created: fix/bot\#2194
#
ChrisLovering
[python-discord/bot] branch deleted: allow\-char\-info\-in\-all\-channels\-bar\-py\-gen
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] New branch created: always\-prepend\-guild\-name\-to\-guild\-invite\-filters
odd spireBOT
regal archBOT
#
Numerlor
[python-discord/bot-core] Pull request opened: #88 Add a decorator that stops successive duplicate invocations of commands

Ports over update_wrapper_globals/command_wraps and adds a decorator that stops successive duplicate invocations of a command with the same arguments in the same channel.

I've extended the source lookup func for linkcode to also look into if statements for the if TYPE_CHECKING use and added a guard to prevent if from erroring on cases that aren't taken care of. It's still missing on some different cases where assignments could occur but that can be addressed in a different PR specifi...

odd spireBOT
regal archBOT
#
MarkKoz
[python-discord/snekbox] branch deleted: feat/nsjail/104/customise\-output\-limit
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
Xithrius
[python-discord/sir-lancebot] New branch created: realpython\-homepage
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
Xithrius
[python-discord/sir-lancebot] branch deleted: realpython\-homepage
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
#
mbaruh
[python-discord/bot] New branch created: fix\_return\_button
#
mbaruh
[python-discord/bot] Pull request opened: #2197 Fix help embed views

Due to various breaking changes in the discord.py library the views of the help embed stopped working properly.

  • Use add_item in CommandView. This was done in GroupView but was apparently missed in CommandView, and the former (hacky) method no longer works.
  • Instead of editing the help message normally (which required to then defer the response), the message edit is made to be the response itself. It seems like the callback would automatically defer the response if none was made, ...
odd spireBOT
regal archBOT
regal archBOT
#
HassanAbouelela
[python-discord/forms-frontend] branch deleted: backend\-command
#
ChrisLovering
[python-discord/forms-backend] Issue opened: #176 Prompt for OAuth2 consent on every auth flow

Description

This would prompt the user to re-consent to sharing info vvery time a they go through the OAuth2 flow.

This also gives them a chance to swap the user that they are logging into Discord with.

Justification

Currently Discord will cache the user you have logged into for forms, meaning if you log out by deleting your token in local storage, the next time you login Discord will automatically login with the same account.

This causes an issue, especially for admins, if y...

regal archBOT
#
ChrisLovering
[python-discord/forms-frontend] Issue opened: #439 Prompt for OAuth2 consent on every auth flow

Description

This would prompt the user to re-consent to sharing info vvery time a they go through the OAuth2 flow.

This also gives them a chance to swap the user that they are logging into Discord with.

Justification

Currently Discord will cache the user you have logged into for forms, meaning if you log out by deleting your token in local storage, the next time you login Discord will automatically login with the same account.

This causes an issue, especially for admins, if y...

#
ChrisLovering
[python-discord/forms-frontend] New branch created: oauth2\-prompt\-consent
regal archBOT
#
HassanAbouelela
[python-discord/forms-frontend] branch deleted: oauth2\-prompt\-consent
#
HassanAbouelela
[python-discord/forms-backend] New branch created: enable\-login\-in\-prod
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
HassanAbouelela
[python-discord/forms-backend] branch deleted: enable\-login\-in\-prod
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
HassanAbouelela
[python-discord/forms-backend] New comment on pull request #173: Replace BaseHTTPMiddleware with pure ASGI middleware

The largest issue right now is that they aren't merged, so no one will know where to look for them. This is only a temporary problem of course, but if you plan on making this same migration on projects where they need to edit the middleware more frequently, that can be a blocker.

More relevantly, there are a few minor things here and there. In no particular order:

  • The Reusing Starlette components section feels lacking. It explains how to go about creating a request and response object...
regal archBOT
#
mbaruh
[python-discord/site] New branch created: cj9\_info
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
mbaruh
[python-discord/site] branch deleted: cj9\_info
odd spireBOT
regal archBOT
#
HassanAbouelela
[python-discord/forms-frontend] New branch created: bump\-dependencies
#
HassanAbouelela
[python-discord/forms-frontend] Pull request opened: #454 Bump Dependencies

I ran into an issue upgrading dependencies from dependabot, since we had a lot of upgrades that would require upgrading multiple libraries at once, even if their version specifiers don't indicate so. This was particularly true for webpack dependencies. On top of that, many things need to be bumped and rewritten in bulk, such as the react dependencies. I went through and made bumps for all those, each "group" in a commit.

Most commits have just been version changes with no code changes, but...

regal archBOT
regal archBOT
#
kosayoda
[python-discord/site] New branch created: events\-css
odd spireBOT
regal archBOT
#
kosayoda
[python-discord/site] branch deleted: events\-css
odd spireBOT
regal archBOT
#
minalike
[python-discord/site] New comment on issue #725: Fix events page inconsistency\.

The two spots I noticed:

I found the code jam page itself difficult to find (an easy to miss link on the events page): https://www.pythondi...

regal archBOT
#
kosayoda
[python-discord/site] New branch created: frontpage\-banners
odd spireBOT
regal archBOT
#
Xithrius
[python-discord/sir-lancebot] branch deleted: fix\-issue\-863
regal archBOT
regal archBOT
#
Numerlor
[python-discord/bot] Pull request opened: #2199 filter out header links in descriptions

Continuation of #2187, spotted the paragraph in a description of not in
didn't think of symbols without signatures when doing the last PR.

Adds the filtering to the description tags.
I've also noticed some things were being filtered with a regex that I glossed over last time, I moved over the source filtering to go through html like headerlinks do; not sure what the backslashes were filtering.

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
ChrisLovering
[site:main] 5 new commits

9d3a1ad Add frontpage banner assets for Summer CJ 2022. - kosayoda
8cf3113 Add Summer CJ 2022 banner to frontpage. - kosayoda
deb7e65 Show shadow on hover for CJ frontpage banner. - kosayoda
1364a1a Add hover transition to match welcome video. - kosayoda
f49d400 Merge pull request #726 from python-discord/fro... - ChrisLovering

#
ChrisLovering
[python-discord/site] branch deleted: frontpage\-banners
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
D0rs4n
[python-discord/sir-robin] Issue opened: #46 Code Jam: Generate output for the team arrangment from a CSV file

The mentioned output should be something that we can easily share with the participants, to let them know which team they are on.
I was planning to make use of Discord UI Views, and create an embed with a button (permanent view), that would send an ephemeral message with an embed detailing the information regarding the team. But, if anyone has any kind of suggestion, please share.

regal archBOT
regal archBOT
#
MarkKoz
[python-discord/snekbox] New branch created: feat/nsjail/101/create\-all\-cgroups
night lilyBOT
#
Sir Lancebot

Connected!

oak estuaryBOT
odd spireBOT
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
#
mbaruh
[python-discord/site] New branch created: more\_cj9\_info
odd spireBOT
regal archBOT
#
RohanJnr
[python-discord/sir-lancebot] New branch created: help\_command\_fix\_863
#
RohanJnr
[python-discord/sir-lancebot] Pull request opened: #1064 Help command fix, normalize suggestions for unknown commands

Relevant Issues

issue: #863

Refer to https://github.com/python-discord/sir-lancebot/pull/884

Fixes

  • The help command used to raise an error when provided with an invalid command name (fixed)
  • There used to be different command suggestions from the help command and the error handler when an unknown command was provided.
    • This has been fixed by using the same method for finding similar commands.

Changes

  • Providing an invalid subcommand but valid parent command (for...
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
HassanAbouelela
[python-discord/site] Issue opened: #728 Deploy Preview Failures

We get occasional CI failures from the netlify preview deploys due to GitHub rate limits. Example:

Traceback (most recent call last):
  File "static-builds/netlify_build.py", line 122, in 
    download_artifact(*get_build_artifact())
  File "static-builds/netlify_build.py", line 36, in get_build_artifact
    pull_request.raise_for_status()
  File "/opt/buildhome/python3.8/lib/python3.8/site-packages/ht...
#
HassanAbouelela
[python-discord/site] New comment on issue #728: Deploy Preview Failures

GitHub App

This seemed promising at first, since it has a much more robust permission system than PATs, and can be scoped per repo. However, accessing the API through an app is only meant to be performed by a secure backend server. You need to use a JWT generated within 10 minutes of the request. Generating the token requires access to the app's private key, which keeping in-line with limitation 2 makes this a no-go.

We can get around this issue by setting up a simple server that perfo...

#
HassanAbouelela
[python-discord/site] New comment on issue #728: Deploy Preview Failures

Netlify CLI

Not possible owing to limitation 1. You need a secret to deploy using the CLI. You also lose out on a lot of the nice features, since you can't manually deploy previews. You can only deploy actual deployments. This is not great for security. On top of that, we'd have to manually add a comment to the PR linking the build, which I believe requires a GitHub app running on a server, (see the previous comment for why that's not desirable). If we're running stuff on our server eithe...

#
HassanAbouelela
[python-discord/site] New comment on issue #728: Deploy Preview Failures

Intermediary Build Server

Instead of asking the GitHub API for the data, we could have the action upload the result to a server we control, and expose those results without authentication or rate limiting issues. @ChrisLovering suggested using our ansible servers with rsync for this purpose.

The following are not limitations, but considerations:

  1. We can't have secrets here, owing to limitation 1, meaning the rsync server has to be unprotected
    2....
regal archBOT
regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/.github] New branch created: events\-speedrun
#
ChrisLovering
[python-discord/.github] Pull request opened: #12 Allow admins/event lead to speedrun event content changes

By moving events inside the first OR block, this allows PRs with only event content changes to be merged with just an admins/event lead approval.

However, by moving events inside that block, it would also mean that the coredev/staff AND block could also satisfy a event content PR approval. To mitigate this, I also added events into that block too, meaning that AND block would also require amdins/event lead approval to satisfy a PR, but would be skipped when paths do not match.

regal archBOT
regal archBOT
#
mbaruh
[python-discord/site] branch deleted: more\_cj9\_info
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
oak estuaryBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot] branch deleted: fix\_return\_button
odd spireBOT
oak estuaryBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
#
HassanAbouelela
[python-discord/forms-frontend] branch deleted: bump\-dependencies
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
minalike
[python-discord/site] New branch created: update\-cj\-frequency
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
kosayoda
[site:main] 4 new commits

cd6e2b5 Update mentions of Code Jam frequency from twic... - minalike
87c07de Make Code Jam information page more discoverable - minalike
2ed2504 Update pydis_site/templates/events/pages/code-j... - minalike
da5108b Merge pull request #730 from python-discord/upd... - kosayoda

#
kosayoda
[python-discord/site] branch deleted: update\-cj\-frequency
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
HassanAbouelela
[python-discord/forms-backend] New branch created: add\-setup
#
HassanAbouelela
[python-discord/forms-backend] Pull request opened: #179 Allow Setup For Unittests

Adds the setup method as a special method for unittest question types. Additionally, a couple QoL fixes are included. I changed unit to self to clarify what it actually is, and I changed an lstrip to a removeprefix to avoid removing unintentional portions of the name (for instance something -> omething)

This will require migrating a few database entries by hand, so hold off on merging till that is ready.

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
ichard26
[python-discord/sir-lancebot] New branch created: ichard26/remove\-redundant\-space\-in\-alias\-help
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
wookie184
[python-discord/sir-lancebot] branch deleted: ichard26/remove\-redundant\-space\-in\-alias\-help
night lilyBOT
#
Sir Lancebot

Connected!

odd spireBOT
regal archBOT
regal archBOT
#
HassanAbouelela
[python-discord/forms-frontend] New comment on issue #392: Better preview embeds for forms in Discord

This is something I looked into a while back. It's an interesting idea, but is not very easy, as that data needs to be present at request time, since discord does not run any JS on the requested resource. We can only easily add this data after the request has been created. Contact Joe for more info on this.

I'll mark this issue as stale for now in case someone comes up with a good solution, but I think this is just not feasible right now.

#
HassanAbouelela
[python-discord/bot-core] Pull request opened: #91 Miscellaneous Bug Fixes

This PR fixes a few miscellaneous bugs and issues that didn't deserve their own issues. Most of this is fairly self-explanatory, such as missing arguments or incorrect docstrings. Many of the code changes are not issues in production either due to being edge cases or being overridden by end projects.

These can be released as an independent bug-fix release or bundled with the next release.

regal archBOT
regal archBOT
regal archBOT
#
Numerlor
[python-discord/bot-core] New review comment on pull request #91: Miscellaneous Bug Fixes

That happens because the type is documented in its Returns:... but trying it out when removed, sphinx-autodoc-typehints fails in resolving it as it sees it as _asyncio.Task, and even with that fixed we wouldn't get the return type because 3.9 doesn't seem to store the annotation on Task for some reason (or at least the 3.9 I have as I can't find anything about it in the changelogs)

For now we can do asyncio.Task[TASK_RETURN] in the docstring

regal archBOT
regal archBOT
#
kosayoda
[python-discord/site] New branch created: fontawesome\-fix
#
kosayoda
[python-discord/site] Pull request opened: #731 Fix interactive timeline icons\.

Details

fa-duck and fa-alien-monster (renamed fa-alien-8bit in the latest
FontAwesome) are part of the Pro plan, which we no longer have.

I replaced fa-alien-monster with fa-discord for a nice "python discord" in the center, and fa-duck with fa-terminal to match fa-code on the other side.

Icon size and alignment are also adjusted for better visual clarity.

Thanks to @LemonPi314 for pointing out the issue.

Current

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

odd spireBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ks129
[python-discord/sir-robin] branch deleted: feat/code\-jams
thin oysterBOT
#
Sir Robin

Connected!

regal archBOT
regal archBOT
#
ChrisLovering
[python-discord/bot-core] New branch created: bump\-d\.py
#
ChrisLovering
[python-discord/bot-core] New tag created: v7\.2\.0\-alpha
#
ChrisLovering
[python-discord/bot] New branch created: ignore\-auto\-mod\-notificaitons
#
ChrisLovering
[python-discord/bot] Pull request opened: #2200 Ignore auto mod notificaitons

When a message triggers an auto_moderation_action notification, Discord re-writes the author field for the system message to look as if it's from the original author.

This means those messages counted towards our anti-spam filter. This both triggered the filter too early, and also resulted in those auto mod messages being deleted when the filter was hit.

TO ignore these messages, d.py had to be bumped to thee version where it added support for auto mod.

odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/bot-core] branch deleted: bump\-d\.py
#
ChrisLovering
[python-discord/bot-core] tag deleted: v7\.2\.0\-alpha
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
#
HassanAbouelela
[bot:main] 5 new commits

2b5b03b Bump botcore to get latest d.py changes - ChrisLovering
8fc03c3 Ignore auto_moderation_action messages when app... - ChrisLovering
ac5bb14 Use new application format for message data in ... - ChrisLovering
7835934 Pin fakeredis to 1.7.5 due to breaking aioredis... - ChrisLovering
405c0f5 Merge pull request #2200 from python-discord/ig... - HassanAbouelela

#
HassanAbouelela
[python-discord/bot] branch deleted: ignore\-auto\-mod\-notificaitons
oak estuaryBOT
odd spireBOT
odd spireBOT
odd spireBOT
regal archBOT
#
D0rs4n
[python-discord/sir-robin] New branch created: pr/code\-jam\-mgmt
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
#
mbaruh
[python-discord/branding] New branch created: cj9\_banner
#
mbaruh
[python-discord/branding] branch deleted: cj9\_banner
odd spireBOT
regal archBOT
#
kosayoda
[python-discord/branding] New branch created: summer\-code\-jam\-2022
odd spireBOT
regal archBOT
#
mbaruh
[python-discord/branding] branch deleted: summer\-code\-jam\-2022
regal archBOT
odd spireBOT
regal archBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/sir-lancebot] New branch created: bot\-core\-migration
regal archBOT
#
Bluenix2
[python-discord/site] New branch created: cj9\-frameworks\-reorder
#
Bluenix2
[python-discord/site] Pull request opened: #734 Rework and reorder CJ9 framework descriptions

This PR, simply put, reorders the CJ9 frameworks to be in a more familiar order based on recommendation:

  • FastAPI and Starlette are both ASGI frameworks many should have heard of
  • websockets is the only client library on the list
  • Django channels is kept on the list but I did not purposefully give it a different position
  • wsproto should be last, only for those who wish to use it

Additionally, the descriptions have been reworded to shine more light on the differences between the fra...

odd spireBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
regal archBOT
odd spireBOT
regal archBOT
regal archBOT
#
HassanAbouelela
[python-discord/forms-frontend] New branch created: fix\-dropdown\-overlap
#
mbaruh
[python-discord/forms-frontend] branch deleted: fix\-dropdown\-overlap
regal archBOT
#
ChrisLovering
[python-discord/bot] New branch created: fix\-trigger\_typing
regal archBOT
odd spireBOT
odd spireBOT
regal archBOT
#
ChrisLovering
[python-discord/sir-robin] branch deleted: pr/code\-jam\-mgmt
thin oysterBOT
#
Sir Robin

Connected!

regal archBOT
#
HassanAbouelela
[python-discord/forms-frontend] New branch created: make\-code\-selectable