#dev-log

Workflow Run

Forms Backend #123

Commit

c5ab400

regal archBOT Dec 16, 2020, 3:27 PM

#

SebastiaanZ

[forms-frontend:sebastiaan/ci/status\-embed] 1 new commit

f3c5d98 Add enhanced status embed workflow - SebastiaanZ

#

SebastiaanZ

[python-discord/forms-frontend] New branch created: sebastiaan/ci/status\-embed

#

SebastiaanZ

[forms-frontend:sebastiaan/ci/status\-embed] 1 new commit

9e96b65 Upload artifact with pull request payload - SebastiaanZ

#

SebastiaanZ

[python-discord/forms-frontend] Pull request opened: #43 Use custom status embed to signal workflow status to Discord

I've added a workflow-run triggered workflow to send an enhanced status embed to our Discord webhook whenever the "Test & Lint" workflow finishes. If the Test & Lint workflow was triggered for a pull_request, it will now upload a build artifact to communicate details about the PR to the status embed workflow.

#

[forms-frontend] Netlify checks success on pull request #43

#

[forms-frontend] GitHub Actions checks success on pull request #43

#

[forms-frontend] GitHub Code Scanning checks success on pull request #43

#

[forms-frontend] GitHub Actions checks success on pull request #43

#

ks129

[bot:master] 1 new commit

ab0785b Add codeowner entries for ks129 - ks129

oak estuaryBOT Dec 16, 2020, 3:45 PM

#

Python Bot

Connected!

odd spireBOT Dec 16, 2020, 3:45 PM

#

[python-discord/bot] Workflow Successful

GitHub Actions run 426012261 succeeded.

Actor

Workflow Run

Deploy #57

Commit

ab0785b

regal archBOT Dec 16, 2020, 3:47 PM

#

ks129

[site:master] 1 new commit

9a203c5 Add codeowner entires for ks129 - ks129

#

ks129

[sir-lancebot:master] 1 new commit

c641324 Add codeowner entries for ks129 - ks129

odd spireBOT Dec 16, 2020, 3:51 PM

#

[python-discord/site] Workflow Successful

GitHub Actions run 426024086 succeeded.

Actor

Workflow Run

Deploy #14

Commit

9a203c5

night lilyBOT Dec 16, 2020, 3:52 PM

#

Sir Lancebot

Connected!

odd spireBOT Dec 16, 2020, 3:53 PM

#

[python-discord/sir-lancebot] Workflow Successful

GitHub Actions run 426025000 succeeded.

Actor

Workflow Run

Build #81

Commit

c641324

regal archBOT Dec 16, 2020, 3:54 PM

#

ks129

[forms-frontend:main] 1 new commit

1d4679d Add codeowner entries for ks129 - ks129

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

ks129

[forms-frontend:main] 1 new commit

4b0435b Add jb3 codeowner request for CI and API - ks129

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

ks129

[python-discord/forms-frontend] Pull request review submitted: #43 Use custom status embed to signal workflow status to Discord

#

ks129

[python-discord/bot] Pull request review submitted: #1283 Remove selenium from the element list

#

ks129

[bot:remove\-selenium] 222 new commits

db5cf7d Add append subcommand for infraction group - PureFunctor
f9971be Add get_latest_infraction utility function - PureFunctor
1ab0084 Refactor routine for obtaining latest infraction - PureFunctor
4ce59c6 Fix unawaited coroutine - PureFunctor
abbb62a Add "a" alias for append - PureFunctor

odd spireBOT Dec 16, 2020, 4:04 PM

#

[python-discord/bot] Checks Successful on PR: #1283 Remove selenium from the element list

GitHub Actions run 426049914 succeeded.

PR Author

Akarys42

Workflow Run

Lint & Test #156

Source Branch

remove-selenium

regal archBOT Dec 16, 2020, 4:05 PM

#

coveralls

[python-discord/bot] New comment on pull request #1283: Remove selenium from the element list

Coverage remained the same at 56.57% when pulling 5beeb824411df9d72676fba3cd07f1f83a3c4ad9 on remove-selenium into ab0785b06157e9628c02bdb5aaecef0d5d8c5cb4 on master.

#

ks129

[python-discord/bot] Pull request closed: #1283 Remove selenium from the element list

#

ks129

[bot:master] 3 new commits

51c4ecd Remove selenium from the element list - Akarys42
5beeb82 Merge branch 'master' into remove-selenium - ks129
52d7f09 Merge pull request #1283 from python-discord/re... - ks129

#

ks129

[python-discord/bot] branch deleted: remove\-selenium

oak estuaryBOT Dec 16, 2020, 4:10 PM

#

Python Bot

Connected!

odd spireBOT Dec 16, 2020, 4:10 PM

#

[python-discord/bot] Workflow Successful

GitHub Actions run 426066411 succeeded.

Actor

Workflow Run

Deploy #58

Commit

52d7f09

regal archBOT Dec 16, 2020, 4:18 PM

#

MarkKoz

[python-discord/forms-frontend] Pull request review submitted: #43 Use custom status embed to signal workflow status to Discord

#

MarkKoz

[python-discord/forms-frontend] New review comment on pull request #43: Use custom status embed to signal workflow status to Discord

          wget --quiet --header="Authorization: token $GITHUB_TOKEN" -O pull_request_payload.zip "$DOWNLOAD_URL" || exit 2

#

MarkKoz

[python-discord/forms-frontend] New review comment on pull request #43: Use custom status embed to signal workflow status to Discord

The cat is unnecessary. jq accepts a filename argument.

          DOWNLOAD_URL=$(jq -r '.artifacts[] | select(.name == "pull-request-payload") | .archive_download_url' artifacts.json)

#

MarkKoz

[python-discord/forms-frontend] New review comment on pull request #43: Use custom status embed to signal workflow status to Discord

This makes the script safer. e and pipefail I consider essentially for any non-trivial script to get some sane error handling behaviour.

        run: |
          set -euo pipefail
          curl -s -H "Authorization: token $GITHUB_TOKEN" ${{ github.event.workflow_run.artifacts_url }} > artifacts.json

Also consider setting x, which will output each command as its executed for debugging purposes.

The || exit ... will no longer be necessary after commands unl...

#

MarkKoz

[python-discord/forms-frontend] Pull request review submitted: #43 Use custom status embed to signal workflow status to Discord

#

MarkKoz

[python-discord/forms-frontend] New review comment on pull request #43: Use custom status embed to signal workflow status to Discord

Likely a good idea to add these. -S shows errors even when silent. -L follows HTTP 3xx redirects.

          curl -sSL -H "Authorization: token $GITHUB_TOKEN" ${{ github.event.workflow_run.artifacts_url }} > artifacts.json

#

MarkKoz

[python-discord/forms-frontend] New review comment on pull request #43: Use custom status embed to signal workflow status to Discord

This makes the script safer. e and pipefail I consider essentially for any non-trivial script to get some sane error handling behaviour.

        run: |
          set -euo pipefail

Also consider setting x, which will output each command as its executed for debugging purposes.

The || exit ... will no longer be necessary after commands unless you want a custom exit code.

#

SebastiaanZ

[python-discord/forms-frontend] Pull request review submitted: #43 Use custom status embed to signal workflow status to Discord

#

SebastiaanZ

[python-discord/forms-frontend] New review comment on pull request #43: Use custom status embed to signal workflow status to Discord

I had some inconsistent results with curl while downloading the zip-file that I couldn't quite explain. I'll see if I can dig them up. wget worked reliable every time.

#

SebastiaanZ

[python-discord/forms-frontend] Pull request review submitted: #43 Use custom status embed to signal workflow status to Discord

#

SebastiaanZ

[python-discord/forms-frontend] New review comment on pull request #43: Use custom status embed to signal workflow status to Discord

No, if the artifact was not uploaded, it will send an embed for a non-PR workflow. This is just a safeguard as the action currently only supports success/failure/cancelled as workflow conclusions. It's not really important here, but I added it as a safe guard against a future CI redesign suddenly breaking this action unexpectedly.

regal archBOT Dec 16, 2020, 5:05 PM

#

HassanAbouelela

[forms-frontend:add\-cross\-env] 20 new commits

53c6217 Update dependency @sentry/react to v5.29.0 - renovate-bot
861e3ab Update dependency @swc/core to v1.2.40 - renovate-bot
ee893db Update dependency @types/react-router-dom to v5... - renovate-bot
3cb3a9a Update dependency webpack-cli to v4.2.0 - renovate-bot
eaf3116 Merge pull request #40 from python-discord/reno... - jb3

#

[forms-frontend] GitHub Actions checks success on pull request #34

#

[forms-frontend] GitHub Code Scanning checks success on pull request #34

#

[forms-frontend] GitHub Actions checks success on pull request #34

#

ks129

[forms-backend:ks123/response\-delete] 1 new commit

97879ae Add DELETE method to response route for form re... - ks129

#

ks129

[python-discord/forms-backend] New branch created: ks123/response\-delete

#

[forms-frontend] Netlify checks success on pull request #34

#

ks129

[python-discord/forms-backend] Pull request opened: #36 Add DELETE method to response route for form response deleting

Admin-only route that deletes form response by form ID and response ID in path parameters.

odd spireBOT Dec 16, 2020, 5:11 PM

#

[python-discord/forms-backend] Checks Successful on PR: #36 Add DELETE method to response route for form response deleting

GitHub Actions run 426194485 succeeded.

PR Author

Workflow Run

Forms Backend #124

Source Branch

ks123/response-delete

regal archBOT Dec 16, 2020, 5:27 PM

#

jarbasmedeiros

[python-discord/bot] Fork created: jarbasmedeiros/bot

#

SebastiaanZ

[python-discord/sir-lancebot] Pull request review submitted: #535 Refactor Advent of Code background tasks to account for deseasonification

#

SebastiaanZ

[python-discord/sir-lancebot] New review comment on pull request #535: Refactor Advent of Code background tasks to account for deseasonification

I don't think it's necessary at the moment to factor it out at the moment.

#

SebastiaanZ

[sir-lancebot:sebastiaan/advent\-of\-code/refactor\-background\-tasks] 2 new commits

c641324 Add codeowner entries for ks129 - ks129
487e554 Merge branch 'master' into sebastiaan/advent-of... - SebastiaanZ

odd spireBOT Dec 16, 2020, 5:37 PM

#

[python-discord/sir-lancebot] Checks Successful on PR: #535 Refactor Advent of Code background tasks to account for deseasonification

GitHub Actions run 426245812 succeeded.

PR Author

SebastiaanZ

Workflow Run

Lint #138

Source Branch

sebastiaan/advent-of-code/refactor-ba...

regal archBOT Dec 16, 2020, 5:40 PM

#

janine9vn

[python-discord/sir-lancebot] New comment on issue #321: AoC: Leaderboard per star

@Skaft Hey, are you still working on this? It's a great idea and I'd love to have it for AoC this year.

#

SebastiaanZ

[python-discord/sir-lancebot] Pull request closed: #535 Refactor Advent of Code background tasks to account for deseasonification

#

SebastiaanZ

[sir-lancebot:master] 10 new commits

82a6007 Add function that allows tasks to sleep until AoC - SebastiaanZ
cb41813 Let status update task sleep until the AoC starts - SebastiaanZ
575fedc Ensure status countdown waits for bot start-up - SebastiaanZ
3e5cff4 Let puzzle notification sleep until AoC starts - SebastiaanZ
d9e98c3 Clarify time_left_until_est_midnight function - SebastiaanZ

#

SebastiaanZ

[python-discord/sir-lancebot] branch deleted: sebastiaan/advent\-of\-code/refactor\-background\-tasks

#

ks129

[python-discord/sir-lancebot] Pull request review submitted: #509 Spook Name Rate Game

Actually, some more comments.

#

ks129

[python-discord/sir-lancebot] New review comment on pull request #509: Spook Name Rate Game

At beginning of this listener this should catch bots instantly, return and then remove check for this from other lines.

#

ks129

[python-discord/sir-lancebot] New review comment on pull request #509: Spook Name Rate Game

Here is no need to make this call 3 lines. Also 2 * 60 * 60 mentioning in comment is also not required, better should be # sleep for two hours.

#

ks129

[python-discord/sir-lancebot] New review comment on pull request #509: Spook Name Rate Game

This should use environment variable

night lilyBOT Dec 16, 2020, 5:48 PM

#

Sir Lancebot

Connected!

odd spireBOT Dec 16, 2020, 5:49 PM

#

[python-discord/sir-lancebot] Workflow Successful

GitHub Actions run 426266334 succeeded.

Actor

SebastiaanZ

Workflow Run

Build #82

Commit

20288cc

regal archBOT Dec 16, 2020, 6:13 PM

#

swfarnsworth

[python-discord/forms-backend] New comment on issue #9: Add support for unittest evaluation using snekbox

Would this be as simple as concatenating the unit tests onto the end of the user's submission and evaluating it in snekbox as one script?

#

MarkKoz

[python-discord/forms-backend] New comment on issue #9: Add support for unittest evaluation using snekbox

From this end, possibly yes. I have concerns about namespace conflicts but it should be possible to write a test suite in a manner that minimally pollutes the namespace. From the snekbox end, the endpoint may need to be adjusted to allow disabling/changing time and memory limits.

#

jb3

[forms-frontend:bugfix/webpack\-config] 1 new commit

779f59c Replace broken webpack mode config with NODE_ENV - jb3

#

jb3

[python-discord/forms-frontend] New branch created: bugfix/webpack\-config

#

jb3

[python-discord/forms-frontend] Pull request opened: #44 Replace broken webpack mode config with NODE\_ENV

Replaces the config in webpack.config.js which sets the webpack mode from using a weird env function parameter to fetching from the NODE_ENV environment variable, which will always be one of "development" or "production", or alternative will default to "none".

#

[forms-frontend] GitHub Actions checks success on pull request #44

#

[forms-frontend] Netlify checks success on pull request #44

#

[forms-frontend] GitHub Code Scanning checks success on pull request #44

#

[forms-frontend] GitHub Actions checks success on pull request #44

#

swfarnsworth

[python-discord/forms-backend] New comment on issue #9: Add support for unittest evaluation using snekbox

@MarkKoz we could start every name specific to the test suite with something like __PyDisTest_ (I believe PyCharm does something like this for its debugger), while also checking if the set of names defined in the user's submission is a subset of the ones we expect to run tests with. I don't know off the top of my head what changing those configurations for snekbox would entail but I'll of course wait to see what Akarys says before doing any work towards this.

#

HassanAbouelela

[python-discord/forms-frontend] Pull request review submitted: #44 Replace broken webpack mode config with NODE\_ENV

#

HassanAbouelela

[python-discord/forms-frontend] New review comment on pull request #44: Replace broken webpack mode config with NODE\_ENV

This works, but can be simplified to

    mode: process.env.NODE_ENV,

as undefined will fallback to "none".

#

jb3

[forms-frontend:bugfix/webpack\-config] 1 new commit

ebb997d Update webpack.config.js - jb3

#

[forms-frontend] GitHub Actions checks success on pull request #44

#

[forms-frontend] Netlify checks success on pull request #44

#

[forms-frontend] GitHub Code Scanning checks success on pull request #44

#

[forms-frontend] GitHub Actions checks success on pull request #44

regal archBOT Dec 16, 2020, 7:17 PM

#

jb3

[python-discord/forms-frontend] Pull request closed: #44 Replace broken webpack mode config with NODE\_ENV

#

jb3

[forms-frontend:main] 3 new commits

779f59c Replace broken webpack mode config with NODE_ENV - jb3
ebb997d Update webpack.config.js - jb3
0451091 Merge pull request #44 from python-discord/bugf... - jb3

#

jb3

[python-discord/forms-frontend] branch deleted: bugfix/webpack\-config

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

HassanAbouelela

[forms-frontend:add\-cross\-env] 4 new commits

779f59c Replace broken webpack mode config with NODE_ENV - jb3
ebb997d Update webpack.config.js - jb3
0451091 Merge pull request #44 from python-discord/bugf... - jb3
5b122ac Merge branch 'main' into add-cross-env - HassanAbouelela

#

[forms-frontend] GitHub Actions checks success on pull request #34

#

[forms-frontend] Netlify checks success on pull request #34

#

[forms-frontend] GitHub Code Scanning checks neutral on pull request #34

#

[forms-frontend] GitHub Actions checks success on pull request #34

regal archBOT Dec 16, 2020, 7:48 PM

#

ks129

[forms-frontend:main] 1 new commit

fa64b6f Fetch forms only once in discovery - ks129

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks failure on main

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

jb3

[python-discord/forms-frontend] Pull request review submitted: #34 Fix Environment Modifying Scripts on Windows

Tested locally, works fine on macOS and I presume you tested it on Windows :smile:

#

jb3

[python-discord/forms-frontend] Pull request closed: #34 Fix Environment Modifying Scripts on Windows

#

jb3

[forms-frontend:main] 4 new commits

0b30260 Adds Cross-Env - HassanAbouelela
44b58ac Merge remote-tracking branch 'origin/main' into... - HassanAbouelela
5b122ac Merge branch 'main' into add-cross-env - HassanAbouelela
6b4bd58 Merge pull request #34 from python-discord/add-... - jb3

#

jb3

[python-discord/forms-frontend] branch deleted: add\-cross\-env

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

jb3

[python-discord/forms-backend] Pull request review submitted: #36 Add DELETE method to response route for form response deleting

#

jb3

[python-discord/forms-backend] Pull request closed: #36 Add DELETE method to response route for form response deleting

#

jb3

[forms-backend:main] 2 new commits

97879ae Add DELETE method to response route for form re... - ks129
20d49e2 Merge pull request #36 from python-discord/ks12... - jb3

#

jb3

[python-discord/forms-backend] branch deleted: ks123/response\-delete

odd spireBOT Dec 16, 2020, 8:12 PM

#

[python-discord/forms-backend] Workflow Successful

GitHub Actions run 426521328 succeeded.

Actor

Workflow Run

Forms Backend #125

Commit

20d49e2

regal archBOT Dec 16, 2020, 10:34 PM

#

GitHubEmploy

[python-discord/sir-lancebot] Pull request opened: #546 Update tox\.ini

Relevant Issues

Description

Reasoning

Screenshots

Additional Details

Did you:

[ ] Join the Python Discord Community?
[ ] If dependencies have been added or updated, run pipenv lock?
[ ] Lint your code (pipenv run lint)?
[ ] Set the PR to allow edits from contributors?

#

python-discord-bot

[python-discord/sir-lancebot] New comment on pull request #546: Update tox\.ini

Thank you for contributing to Python Discord!

Please check out the following documents:

Join the Discord server! - If you have not already, join the Discord server and introduce yourself in our #dev-contrib channel.
Python Discord Contributor Wiki - A set of resources about setting up our services
Python Discord Code of Conduct - Code of conduct for our commu...

odd spireBOT Dec 16, 2020, 10:35 PM

#

[python-discord/sir-lancebot] Checks Successful on PR: #546 Update tox.ini

GitHub Actions run 426769854 succeeded.

PR Author

GitHubEmploy

Workflow Run

Lint #140

Source Branch

GitHubEmploy:patch-1

regal archBOT Dec 16, 2020, 11:27 PM

#

jb3

[forms-backend:docs/api\-spec] 9 new commits

d0dd289 Add Spectree package - jb3
9932f04 Add validation utilities - jb3
6fb231a Make model changes - jb3
a9006d8 Document forms routes - jb3
00fae00 Document auth route - jb3

#

jb3

[python-discord/forms-backend] New branch created: docs/api\-spec

#

jb3

[python-discord/forms-backend] Pull request opened: #37 Provide HTML documentation of endpoints

odd spireBOT Dec 16, 2020, 11:31 PM

#

[python-discord/forms-backend] Checks Successful on PR: #37 Provide HTML documentation of endpoints

GitHub Actions run 426856898 succeeded.

PR Author

Workflow Run

Forms Backend #126

Source Branch

docs/api-spec

regal archBOT Dec 16, 2020, 11:31 PM

#

jb3

[python-discord/forms-backend] New comment on pull request #37: Provide HTML documentation of endpoints

regal archBOT Dec 16, 2020, 11:50 PM

#

jb3

[bot:master] 1 new commit

14e7160 Update verification.py - jb3

oak estuaryBOT Dec 16, 2020, 11:54 PM

#

Python Bot

Connected!

odd spireBOT Dec 16, 2020, 11:54 PM

#

[python-discord/bot] Workflow Successful

GitHub Actions run 426892362 succeeded.

Actor

Workflow Run

Deploy #59

Commit

14e7160

regal archBOT Dec 17, 2020, 12:21 AM

#

jb3

[python-discord/forms-backend] New comment on issue #9: Add support for unittest evaluation using snekbox

We have migrated Python Discord Form issue tracking to Notion.

This specific ticket can be found here.

This discussion will be locked for now, all future conversation should occur in the Notion documents.

If you are an external collaborator who is interested in contributing to forms please shoot me a DM on Discord, joe#6000.

This GitHub discussion has bee...

#

jb3

[python-discord/forms-backend] New comment on issue #15: Deliver a payload to a configured webhook on form submission

We have migrated Python Discord Form issue tracking to Notion.

This specific ticket can be found here.

This discussion will be locked for now, all future conversation should occur in the Notion documents.

If you are an external collaborator who is interested in contributing to forms please shoot me a DM on Discord, joe#6000.

This GitHub discuss...

#

jb3

[python-discord/forms-backend] New comment on issue #10: Validate form responses using Python code

We have migrated Python Discord Form issue tracking to Notion.

This specific ticket can be found here.

This discussion will be locked for now, all future conversation should occur in the Notion documents.

If you are an external collaborator who is interested in contributing to forms please shoot me a DM on Discord, joe#6000.

This GitHub discussion has been transf...

#

jb3

[forms-backend:main] 1 new commit

0d95ec4 Create config.yml - jb3

odd spireBOT Dec 17, 2020, 12:27 AM

#

[python-discord/forms-backend] Workflow Successful

GitHub Actions run 426938161 succeeded.

Actor

Workflow Run

Forms Backend #127

Commit

0d95ec4

regal archBOT Dec 17, 2020, 12:27 AM

#

jb3

[forms-frontend:main] 1 new commit

cc0ca54 Create config.yml - jb3

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

regal archBOT Dec 17, 2020, 4:50 AM

#

HassanAbouelela

[forms-backend:submission\-timestamp] 1 new commit

f3369d8 Adds Timestamp Field - HassanAbouelela

#

HassanAbouelela

[python-discord/forms-backend] New branch created: submission\-timestamp

#

HassanAbouelela

[python-discord/forms-backend] Pull request opened: #38 Adds Timestamp Field

Closes this notion ticket.

Adds timestamp field to user form responses on model generation.

The model saves an ISO formatted time string to avoid serialization errors. The field is set by the program, even if a user passes the timestamp field.

odd spireBOT Dec 17, 2020, 4:54 AM

#

[python-discord/forms-backend] Checks Successful on PR: #38 Adds Timestamp Field

GitHub Actions run 427297972 succeeded.

PR Author

https://github.com/python-discord/forms-backend/blob/main/backend/routes/forms/response.py#L27

Workflow Run

Forms Backend #128

Source Branch

submission-timestamp

regal archBOT Dec 17, 2020, 5:59 AM

#

ks129

[python-discord/forms-backend] Pull request review submitted: #38 Adds Timestamp Field

Current way doesn't work, because this overwrite datetime every time when model instance is created. Instance is also created when document is fetched:

Datetime must be provided by submit route. Parser should be @validator('timestamp', pre=True) and this should take datetime and return str.

regal archBOT Dec 17, 2020, 6:38 AM

#

HassanAbouelela

[forms-backend:submission\-timestamp] 1 new commit

d2427c7 Fixes DB Timestamp on Return - HassanAbouelela

odd spireBOT Dec 17, 2020, 6:39 AM

#

[python-discord/forms-backend] Checks Successful on PR: #38 Adds Timestamp Field

GitHub Actions run 427444737 succeeded.

PR Author

Workflow Run

Forms Backend #129

Source Branch

submission-timestamp

regal archBOT Dec 17, 2020, 6:41 AM

#

ks129

[python-discord/forms-backend] Pull request review submitted: #38 Adds Timestamp Field

#

ks129

[python-discord/forms-backend] New review comment on pull request #38: Adds Timestamp Field

    def set_timestamp(cls, iso_string: t.Optional[str]) -> t.Optional[str]:

You do None check, so iso_string can be None. When this raise ValueError, then no string is returned and this is also None.

#

HassanAbouelela

[forms-backend:submission\-timestamp] 1 new commit

1d28213 Update backend/models/form_response.py - HassanAbouelela

#

HassanAbouelela

[python-discord/forms-backend] Pull request review submitted: #38 Adds Timestamp Field

#

HassanAbouelela

[python-discord/forms-backend] New review comment on pull request #38: Adds Timestamp Field

Good catch, thanks.

odd spireBOT Dec 17, 2020, 6:45 AM

#

[python-discord/forms-backend] Checks Successful on PR: #38 Adds Timestamp Field

GitHub Actions run 427453119 succeeded.

PR Author

Workflow Run

Forms Backend #130

Source Branch

submission-timestamp

regal archBOT Dec 17, 2020, 7:02 AM

#

ks129

[python-discord/forms-backend] Pull request review submitted: #38 Adds Timestamp Field

Tested locally, works.

#

ks129

[python-discord/forms-backend] Pull request closed: #38 Adds Timestamp Field

#

ks129

[forms-backend:main] 4 new commits

f3369d8 Adds Timestamp Field - HassanAbouelela
d2427c7 Fixes DB Timestamp on Return - HassanAbouelela
1d28213 Update backend/models/form_response.py - HassanAbouelela
30af30b Merge pull request #38 from python-discord/subm... - ks129

#

ks129

[python-discord/forms-backend] branch deleted: submission\-timestamp

odd spireBOT Dec 17, 2020, 7:05 AM

#

[python-discord/forms-backend] Workflow Successful

GitHub Actions run 427482994 succeeded.

Actor

Workflow Run

Forms Backend #131

Commit

30af30b

regal archBOT Dec 17, 2020, 7:24 AM

#

ks129

[python-discord/forms-backend] Pull request review submitted: #37 Provide HTML documentation of endpoints

Wow, really nice. Tested.

#

ks129

[forms-backend:docs/api\-spec] 6 new commits

0d95ec4 Create config.yml - jb3
f3369d8 Adds Timestamp Field - HassanAbouelela
d2427c7 Fixes DB Timestamp on Return - HassanAbouelela
1d28213 Update backend/models/form_response.py - HassanAbouelela
30af30b Merge pull request #38 from python-discord/subm... - ks129

odd spireBOT Dec 17, 2020, 7:26 AM

#

[python-discord/forms-backend] Checks Successful on PR: #37 Provide HTML documentation of endpoints

GitHub Actions run 427517706 succeeded.

PR Author

Workflow Run

Forms Backend #132

Source Branch

docs/api-spec

regal archBOT Dec 17, 2020, 7:30 AM

#

ks129

[python-discord/forms-backend] Pull request closed: #37 Provide HTML documentation of endpoints

#

ks129

[forms-backend:main] 11 new commits

d0dd289 Add Spectree package - jb3
9932f04 Add validation utilities - jb3
6fb231a Make model changes - jb3
a9006d8 Document forms routes - jb3
00fae00 Document auth route - jb3

#

ks129

[python-discord/forms-backend] branch deleted: docs/api\-spec

odd spireBOT Dec 17, 2020, 7:35 AM

#

[python-discord/forms-backend] Workflow Successful

GitHub Actions run 427526638 succeeded.

Actor

Workflow Run

Forms Backend #133

Commit

3d4038b

regal archBOT Dec 17, 2020, 9:57 AM

#

HassanAbouelela

[forms-backend:discord\-webhook] 6 new commits

b80b329 Adds Webhook Option - HassanAbouelela
b795197 Merge branch 'main' into discord-webhook - HassanAbouelela
595c19e Adds Frontend URL to Env - HassanAbouelela
20c72c4 Makes Webhooks Optional - HassanAbouelela
bbbfa15 Adds Webhook Sending Functionality - HassanAbouelela

#

HassanAbouelela

[python-discord/forms-backend] New branch created: discord\-webhook

regal archBOT Dec 17, 2020, 1:33 PM

#

SebastiaanZ

[async-rediscache:release/0\.1\.3] 2 new commits

bf8795a Reverse direction of RedisQueue for consistency - SebastiaanZ
6508cc8 Add deprecation warning to NamespaceLock - SebastiaanZ

#

SebastiaanZ

[python-discord/async-rediscache] New branch created: release/0\.1\.3

regal archBOT Dec 17, 2020, 2:24 PM

#

SebastiaanZ

[async-rediscache] Branch release/0\.1\.3 was force-pushed to `710825d`

Compare changes

odd spireBOT Dec 17, 2020, 2:36 PM

#

[python-discord/bot] Checks Successful on PR: #1016 Some bug fixes

GitHub Actions run 428301175 succeeded.

PR Author

Workflow Run

Lint & Test #159

Source Branch

ks129:bug-fixes

regal archBOT Dec 17, 2020, 2:42 PM

#

SebastiaanZ

[async-rediscache:release/0\.1\.3] 4 new commits

e350b74 Make pop method atomic from a Redis perspective - SebastiaanZ
6826d8a Install fakeredis with Lua support for unittests - SebastiaanZ
94bef00 Add init.py to make importlib.resources work - SebastiaanZ
91d4cf8 Stop RedisQueue from issuing deprecation warnings - SebastiaanZ

#

Akarys42

[python-discord/sir-lancebot] New comment on pull request #546: Update tox\.ini

Hello @GitHubEmploy,

Since this change doesn't seem to have any linked issues, neither any reasoning about why this change has been made, I'm going to close this PR.
Feel free to add a comment explaining why do you think we should ignore this flake8 error.

Thanks for your interest in our project!

#

Akarys42

[python-discord/sir-lancebot] Pull request closed: #546 Update tox\.ini

regal archBOT Dec 17, 2020, 3:54 PM

#

ks129

[python-discord/bot] Pull request review submitted: #1016 Some bug fixes

#

ks129

[python-discord/bot] New review comment on pull request #1016: Some bug fixes

Changed this log from error to info.

odd spireBOT Dec 17, 2020, 3:55 PM

#

[python-discord/bot] Checks Successful on PR: #1016 Some bug fixes

GitHub Actions run 428465789 succeeded.

PR Author

Workflow Run

Lint & Test #160

Source Branch

ks129:bug-fixes

#

[python-discord/bot] Checks Failed on PR: #1289 Add Branding Manager, add GitHub token

GitHub Actions run 428476434 failed.

PR Author

Workflow Run

Lint & Test #161

Source Branch

ks129:branding-manager

#

[python-discord/bot] Checks Successful on PR: #1289 Add Branding Manager, add GitHub token

GitHub Actions run 428502483 succeeded.

PR Author

Workflow Run

Lint & Test #162

Source Branch

ks129:branding-manager

regal archBOT Dec 17, 2020, 4:14 PM

#

coveralls

[python-discord/bot] New comment on pull request #1289: Add Branding Manager, add GitHub token

Coverage decreased (-0.4%) to 56.122% when pulling 4aa7429cc967c43d2797c9bf7a24968c97855822 on ks129:branding-manager into 14e71609e8e40be0832b66df7a0c309ba262659a on python-discord:master.

#

renovate[bot]

[forms-frontend:renovate/webpack\-5\.x] 1 new commit

adcd196 Update dependency webpack to v5.11.0 - renovate-bot

#

renovate[bot]

[python-discord/forms-frontend] New branch created: renovate/webpack\-5\.x

#

renovate[bot]

[python-discord/forms-frontend] Pull request opened: #45 Update dependency webpack to v5\.11\.0

This PR contains the following updates:

Package	Change	Age	Adoption	Passing	Confidence
webpack	`5.10.3` -> `5.11.0`		[![adopt...

#

[forms-frontend] GitHub Actions checks success on pull request #45

#

[forms-frontend] GitHub Code Scanning checks success on pull request #45

#

[forms-frontend] GitHub Actions checks success on pull request #45

#

[forms-frontend] Netlify checks success on pull request #45

regal archBOT Dec 17, 2020, 6:36 PM

#

GitHubEmploy

[python-discord/sir-lancebot] New comment on pull request #546: Update tox\.ini

Hello @GitHubEmploy,

Since this change doesn't seem to have any linked issues, neither any reasoning about why this change has been made, I'm going to close this PR.
Feel free to add a comment explaining why do you think we should ignore this flake8 error.

Thanks for your interest in our project!

If you don't change this, the program will become liable for crashing randomly. After this, It will stop, IDK why, but for me when I changed this it stopped. You guys can resear...

regal archBOT Dec 17, 2020, 8:13 PM

#

SebastiaanZ

[async-rediscache:release/0\.1\.3] 14 new commits

e0cd542 Add fakeredis with Lua support to setup.py extras - SebastiaanZ
51a0f0c State that Python 3.9 is supported in setup.py - SebastiaanZ
c6fe0ee Add test for TypeError raised by increment method - SebastiaanZ
5b7a839 Add rescheduling scripts for atomic rescheduling - SebastiaanZ
588eee8 Add version compatibility for Lua unpack function - SebastiaanZ

#

SebastiaanZ

[python-discord/async-rediscache] branch deleted: release/0\.1\.3

#

SebastiaanZ

[async-rediscache:release/0\.2\.0] 23 new commits

bf8795a Reverse direction of RedisQueue for consistency - SebastiaanZ
b90d525 Add deprecation warning to NamespaceLock - SebastiaanZ
e5dfc7a Add optional value converter that supports errors - SebastiaanZ
b1b6882 Allow loading/registering Lua scripts in Redis - SebastiaanZ
710825d Make increment atomic from a redis perspective - SebastiaanZ

#

SebastiaanZ

[python-discord/async-rediscache] New branch created: release/0\.2\.0

#

SebastiaanZ

[async-rediscache:release/0\.1\.3] 2 new commits

557ceea Unify namespace_lock usage across all data types - SebastiaanZ
9ab03ed Add .coverage results file to .gitignore - SebastiaanZ

#

SebastiaanZ

[python-discord/async-rediscache] New branch created: release/0\.1\.3

#

jb3

[python-discord/forms-frontend] Pull request review submitted: #45 Update dependency webpack to v5\.11\.0

#

jb3

[python-discord/forms-frontend] Pull request closed: #45 Update dependency webpack to v5\.11\.0

#

jb3

[forms-frontend:main] 2 new commits

adcd196 Update dependency webpack to v5.11.0 - renovate-bot
d54e653 Merge pull request #45 from python-discord/reno... - jb3

#

jb3

[python-discord/forms-frontend] branch deleted: renovate/webpack\-5\.x

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

SebastiaanZ

[python-discord/async-rediscache] Pull request opened: #9 Prepare the release of version 0\.2\.0 of Async RedisCache

It's high time for a new release of Async RedisCache. This new version will bring a number of new features to this package and add a deprecation warning to the usage of NamespaceLock.

New features

Atomic Compound Operations

All compound operations consisting of multiple Redis commands should now be atomic from a Redis perspective due to the usage of a Lua Redis script that adds the missing functionality.

RedisQueue Task Iteration

You can now iterate over a RedisQueue w...

#

[forms-frontend] GitHub Actions checks success on main

#

[async-rediscache] GitHub Actions checks cancelled on pull request #9

#

SebastiaanZ

[python-discord/async-rediscache] New tag created: v0\.2\.0\-rc2

#

[async-rediscache] GitHub Actions checks failure on v0\.2\.0\-rc2

#

renovate[bot]

[forms-frontend:renovate/sentry\-monorepo] 1 new commit

58f4d00 Update dependency @sentry/react to v5.29.2 - renovate-bot

#

renovate[bot]

[python-discord/forms-frontend] New branch created: renovate/sentry\-monorepo

#

renovate[bot]

[python-discord/forms-frontend] Pull request opened: #46 Update dependency @sentry/react to v5\.29\.2

This PR contains the following updates:

Package	Change	Age	Adoption	Passing	Confidence
@sentry/react	`5.29.1` -> `5.29.2`	[](https://docs.renovatebot...

#

[forms-frontend] GitHub Actions checks success on pull request #46

#

[forms-frontend] GitHub Code Scanning checks success on pull request #46

#

[forms-frontend] GitHub Actions checks success on pull request #46

#

[forms-frontend] Netlify checks success on pull request #46

#

SebastiaanZ

[async-rediscache:release/0\.2\.0] 3 new commits

557ceea Unify namespace_lock usage across all data types - SebastiaanZ
9ab03ed Add .coverage results file to .gitignore - SebastiaanZ
e29b5d7 Bump version to v0.2.0 in setup.py - SebastiaanZ

#

[async-rediscache] GitHub Actions checks failure on pull request #9

#

SebastiaanZ

[python-discord/async-rediscache] New tag created: v0\.2\.0\-rc3

#

[async-rediscache] GitHub Actions checks success on v0\.2\.0\-rc3

#

SebastiaanZ

[async-rediscache:release/0\.2\.0] 1 new commit

606ace3 Update Pipfile.lock - SebastiaanZ

#

[async-rediscache] GitHub Actions checks failure on pull request #9

#

SebastiaanZ

[async-rediscache:release/0\.2\.0] 1 new commit

234ba48 Install coverage.py with toml for pyproject.toml - SebastiaanZ

#

[async-rediscache] GitHub Actions checks success on pull request #9

#

SebastiaanZ

[python-discord/async-rediscache] branch deleted: release/0\.1\.3

regal archBOT Dec 17, 2020, 10:02 PM

#

SebastiaanZ

[async-rediscache:release/0\.2\.0] 2 new commits

4f43da0 Add time-machine as a testing dependency - SebastiaanZ
47f0b32 Add namespace set_expiry and set_expiry_at options - SebastiaanZ

#

[async-rediscache] GitHub Actions checks success on pull request #9

#

SebastiaanZ

[async-rediscache:release/0\.2\.0] 1 new commit

6b05bae Add test for set_expiry_at with datetime instance - SebastiaanZ

#

[async-rediscache] GitHub Actions checks success on pull request #9

#

SebastiaanZ

[python-discord/async-rediscache] New tag created: v0\.2\.0\-rc4

#

[async-rediscache] GitHub Actions checks failure on v0\.2\.0\-rc4

#

SebastiaanZ

[python-discord/async-rediscache] Pull request closed: #9 Prepare the release of version 0\.2\.0 of Async RedisCache

#

SebastiaanZ

[async-rediscache:master] 32 new commits

bf8795a Reverse direction of RedisQueue for consistency - SebastiaanZ
b90d525 Add deprecation warning to NamespaceLock - SebastiaanZ
e5dfc7a Add optional value converter that supports errors - SebastiaanZ
b1b6882 Allow loading/registering Lua scripts in Redis - SebastiaanZ
710825d Make increment atomic from a redis perspective - SebastiaanZ

#

[async-rediscache] GitHub Actions checks success on master

#

jb3

[python-discord/forms-frontend] Pull request review submitted: #46 Update dependency @sentry/react to v5\.29\.2

#

jb3

[python-discord/forms-frontend] Pull request closed: #46 Update dependency @sentry/react to v5\.29\.2

#

jb3

[forms-frontend:main] 2 new commits

58f4d00 Update dependency @sentry/react to v5.29.2 - renovate-bot
07ca917 Merge pull request #46 from python-discord/reno... - jb3

#

jb3

[python-discord/forms-frontend] branch deleted: renovate/sentry\-monorepo

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

SebastiaanZ

[python-discord/async-rediscache] New tag created: v0\.2\.0

#

[async-rediscache] GitHub Actions checks success on v0\.2\.0

regal archBOT Dec 17, 2020, 11:51 PM

#

HassanAbouelela

[forms-backend:discord\-webhook] 1 new commit

cbd38ce Sends Embed Asynchronously - HassanAbouelela

#

jb3

[forms-backend:main] 1 new commit

78288e7 Update submit.py - jb3

odd spireBOT Dec 17, 2020, 11:55 PM

#

[python-discord/forms-backend] Workflow Successful

GitHub Actions run 429311439 succeeded.

Actor

Workflow Run

Forms Backend #134

Commit

78288e7

regal archBOT Dec 18, 2020, 12:04 AM

#

HassanAbouelela

[forms-backend:discord\-webhook] 2 new commits

78288e7 Update submit.py - jb3
746df7a Merge branch 'main' into discord-webhook - HassanAbouelela

#

HassanAbouelela

[python-discord/forms-backend] Pull request opened: #39 Adds Discord Embed on Form Submission

Closes this notion ticket.

Important to do note

The webhook links to a frontend URL that displays the form response, which is currently not supported. The frontend URL was added to environment variables in preparation, but the actual viewing URL will have to be added in place of the placeholder once available.

Description

Implements discord webhoo...

odd spireBOT Dec 18, 2020, 12:06 AM

#

[python-discord/forms-backend] Checks Successful on PR: #39 Adds Discord Embed on Form Submission

GitHub Actions run 429329945 succeeded.

PR Author

Workflow Run

Forms Backend #135

Source Branch

discord-webhook

regal archBOT Dec 18, 2020, 12:16 AM

#

jb3

[python-discord/forms-backend] New review comment on pull request #39: Adds Discord Embed on Form Submission

I think that we could use something prettier than _user_mention_ here. How about {user}?

#

jb3

[python-discord/forms-backend] New review comment on pull request #39: Adds Discord Embed on Form Submission

Related to other comment.

#

jb3

[python-discord/forms-backend] Pull request review submitted: #39 Adds Discord Embed on Form Submission

Initial review pass. Functionality all looks good and is what I was expecting, nice work!

#

jb3

[python-discord/forms-backend] New review comment on pull request #39: Adds Discord Embed on Form Submission

Not sure how I feel about synchronous validation here. This has the potential to block the worker from handling any requests.

If we can't use asyncio in the validator maybe we should write our own utility for validating and then call that before even creating the webhook model.

#

HassanAbouelela

[python-discord/forms-backend] New review comment on pull request #39: Adds Discord Embed on Form Submission

I've considered making the string formattable and using it as an f-string, but my fear here is that means we allow anyone to run code inside the message. I don't know who will be allowed to create forms yet, so I didn't want to leave an easy exploit in.

#

HassanAbouelela

[python-discord/forms-backend] Pull request review submitted: #39 Adds Discord Embed on Form Submission

#

jb3

[python-discord/forms-backend] Pull request review submitted: #39 Adds Discord Embed on Form Submission

#

jb3

[python-discord/forms-backend] New review comment on pull request #39: Adds Discord Embed on Form Submission

Yeah, that's a fair concern. We could look at using string.Formatter to implement a safe formatter which can access a limited subset of keys, docs here.

Our Modmail bot has a nice implementation of a safe formatter found here

regal archBOT Dec 18, 2020, 1:51 AM

#

HassanAbouelela

[forms-backend:discord\-webhook] 1 new commit

df0a8a1 Change Embed Description - HassanAbouelela

odd spireBOT Dec 18, 2020, 1:52 AM

#

[python-discord/forms-backend] Checks Successful on PR: #39 Adds Discord Embed on Form Submission

GitHub Actions run 429478194 succeeded.

PR Author

Workflow Run

Forms Backend #136

Source Branch

discord-webhook

regal archBOT Dec 18, 2020, 3:27 AM

#

swfarnsworth

[bot:swfarnsworth/developer\_command] 2 new commits

fc1f7ac User gets the bot DM when verified via !verify. - swfarnsworth
2b09d73 Responses from the bot mention the user. - swfarnsworth

#

swfarnsworth

[python-discord/bot] New branch created: swfarnsworth/developer\_command

regal archBOT Dec 18, 2020, 4:11 AM

#

swfarnsworth

[python-discord/bot] Pull request opened: #1328 Manual verification with \`\!verify\` sends the user the confirmation DM\.

Previously, users wouldn't get the DM from the bot that one usually gets when they verify if a moderator verifies them manually. This PR solves this.

Per @Den4200's request, the response from the bot also mentions the user rather than stating their account name.

odd spireBOT Dec 18, 2020, 4:13 AM

#

[python-discord/bot] Checks Successful on PR: #1328 Manual verification with `!verify` sends the user the confirmation DM.

GitHub Actions run 429660584 succeeded.

PR Author

Workflow Run

Lint & Test #163

Source Branch

swfarnsworth/developer_command

regal archBOT Dec 18, 2020, 4:14 AM

#

coveralls

[python-discord/bot] New comment on pull request #1328: Manual verification with \`\!verify\` sends the user the confirmation DM\.

Coverage decreased (-0.004%) to 56.567% when pulling 2b09d739074f6d1ae259e234ea2ab787711d839d on swfarnsworth/developer_command into 14e71609e8e40be0832b66df7a0c309ba262659a on master.

regal archBOT Dec 18, 2020, 8:25 AM

#

HassanAbouelela

[forms-backend:discord\-webhook] 1 new commit

34c94ba Uses Builtin User Class - HassanAbouelela

odd spireBOT Dec 18, 2020, 8:26 AM

#

[python-discord/forms-backend] Checks Successful on PR: #39 Adds Discord Embed on Form Submission

GitHub Actions run 430023371 succeeded.

PR Author

Workflow Run

Forms Backend #137

Source Branch

discord-webhook

regal archBOT Dec 18, 2020, 1:53 PM

#

ks129

[forms-backend:ks123/form\-patch] 1 new commit

30de873 Add PATCH method to form endpoint - ks129

#

ks129

[python-discord/forms-backend] New branch created: ks123/form\-patch

#

ks129

[python-discord/forms-backend] Pull request opened: #40 Add PATCH method to form endpoint

To allow updating form after creation, add PATCH method to form endpoint. Does partial update.

odd spireBOT Dec 18, 2020, 1:58 PM

#

[python-discord/forms-backend] Checks Successful on PR: #40 Add PATCH method to form endpoint

GitHub Actions run 430595851 succeeded.

PR Author

Workflow Run

Forms Backend #138

Source Branch

ks123/form-patch

regal archBOT Dec 18, 2020, 2:09 PM

#

ks129

[forms-backend:ks123/dns\-check\-remove] 3 new commits

6d3b7d9 Remove DNS blacklist handling from submit route - ks129
de0a76c Remove DNS blacklist field from antispam model - ks129
9a54873 Remove pydnsbl dependency - ks129

#

ks129

[python-discord/forms-backend] New branch created: ks123/dns\-check\-remove

#

ks129

[forms-backend:ks123/form\-patch] 2 new commits

78288e7 Update submit.py - jb3
3ab9ef5 Merge branch 'main' into ks123/form-patch - ks129

odd spireBOT Dec 18, 2020, 2:10 PM

#

[python-discord/forms-backend] Checks Successful on PR: #40 Add PATCH method to form endpoint

GitHub Actions run 430618358 succeeded.

PR Author

Workflow Run

Forms Backend #139

Source Branch

ks123/form-patch

regal archBOT Dec 18, 2020, 2:13 PM

#

ks129

[python-discord/forms-backend] Pull request opened: #41 Remove DNS blacklist check from form submission

Removed validation from route.
Removed field from model.
Removed dependency.

odd spireBOT Dec 18, 2020, 2:15 PM

#

[python-discord/forms-backend] Checks Successful on PR: #41 Remove DNS blacklist check from form submission

GitHub Actions run 430625765 succeeded.

PR Author

Workflow Run

Forms Backend #140

Source Branch

ks123/dns-check-remove

regal archBOT Dec 18, 2020, 4:10 PM

#

Vthechamp22

[python-discord/sir-lancebot] Pull request review submitted: #509 Spook Name Rate Game

#

Vthechamp22

[python-discord/sir-lancebot] New review comment on pull request #509: Spook Name Rate Game

Oops I think I had formatted it with black by mistake once, so maybe thats why it spans across 3 lines

odd spireBOT Dec 18, 2020, 4:21 PM

#

[python-discord/sir-lancebot] Checks Successful on PR: #509 Spook Name Rate Game

GitHub Actions run 430862827 succeeded.

PR Author

Vthechamp22

Workflow Run

Lint #141

Source Branch

Vthechamp22:master

regal archBOT Dec 18, 2020, 5:07 PM

#

jb3

[python-discord/forms-backend] Pull request review submitted: #41 Remove DNS blacklist check from form submission

#

jb3

[forms-backend:ks123/dns\-check\-remove] 2 new commits

78288e7 Update submit.py - jb3
d2f610a Merge branch 'main' into ks123/dns-check-remove - jb3

odd spireBOT Dec 18, 2020, 5:08 PM

#

[python-discord/forms-backend] Checks Successful on PR: #41 Remove DNS blacklist check from form submission

GitHub Actions run 430952576 succeeded.

PR Author

Workflow Run

Forms Backend #141

Source Branch

ks123/dns-check-remove

regal archBOT Dec 18, 2020, 5:10 PM

#

jb3

[python-discord/forms-backend] Pull request closed: #41 Remove DNS blacklist check from form submission

#

jb3

[forms-backend:main] 5 new commits

6d3b7d9 Remove DNS blacklist handling from submit route - ks129
de0a76c Remove DNS blacklist field from antispam model - ks129
9a54873 Remove pydnsbl dependency - ks129
d2f610a Merge branch 'main' into ks123/dns-check-remove - jb3
d78075f Merge pull request #41 from python-discord/ks12... - jb3

#

jb3

[python-discord/forms-backend] branch deleted: ks123/dns\-check\-remove

#

ks129

[forms-backend:ks123/form\-patch] 6 new commits

6d3b7d9 Remove DNS blacklist handling from submit route - ks129
de0a76c Remove DNS blacklist field from antispam model - ks129
9a54873 Remove pydnsbl dependency - ks129
d2f610a Merge branch 'main' into ks123/dns-check-remove - jb3
d78075f Merge pull request #41 from python-discord/ks12... - jb3

odd spireBOT Dec 18, 2020, 5:13 PM

#

[python-discord/forms-backend] Checks Successful on PR: #40 Add PATCH method to form endpoint

GitHub Actions run 430960033 succeeded.

PR Author

Workflow Run

Forms Backend #143

Source Branch

ks123/form-patch

#

[python-discord/forms-backend] Workflow Successful

GitHub Actions run 430957201 succeeded.

Actor

Workflow Run

Forms Backend #142

Commit

d78075f

regal archBOT Dec 18, 2020, 5:18 PM

#

jb3

[python-discord/forms-backend] Pull request review submitted: #40 Add PATCH method to form endpoint

Needs code to actually write back to the database, but validation looks good.

#

jb3

[python-discord/forms-backend] New review comment on pull request #40: Add PATCH method to form endpoint

Our new policy for Pydantic validation errors is HTTP 422, since this will automatically pick up the response structure for pydantic validations.

                return JSONResponse(e.errors(), status_code=422)

#

ks129

[forms-backend:ks123/form\-patch] 1 new commit

473f61d Actually update form in DB in PATCH method - ks129

#

ks129

[forms-backend:ks123/form\-patch] 1 new commit

1b82bac Update status code of pydantic errors response - ks129

odd spireBOT Dec 18, 2020, 5:23 PM

#

[python-discord/forms-backend] Checks Successful on PR: #40 Add PATCH method to form endpoint

GitHub Actions run 430978789 succeeded.

PR Author

Workflow Run

Forms Backend #144

Source Branch

ks123/form-patch

#

[python-discord/forms-backend] Checks Successful on PR: #40 Add PATCH method to form endpoint

GitHub Actions run 430979824 succeeded.

PR Author

Workflow Run

Forms Backend #145

Source Branch

ks123/form-patch

regal archBOT Dec 18, 2020, 8:12 PM

#

HassanAbouelela

[forms-backend:discord\-webhook] 1 new commit

d1cb420 Runs Initial Validation Asynchronously - HassanAbouelela

#

HassanAbouelela

[forms-backend:discord\-webhook] 1 new commit

5250e54 Adds Missing Annotation - HassanAbouelela

#

HassanAbouelela

[python-discord/forms-backend] Pull request review submitted: #39 Adds Discord Embed on Form Submission

#

HassanAbouelela

[python-discord/forms-backend] New review comment on pull request #39: Adds Discord Embed on Form Submission

Addressed in d1cb420, but I'll leave this open a little longer in case anyone has a better implementation.

regal archBOT Dec 19, 2020, 12:14 AM

#

renovate[bot]

[forms-frontend:renovate/eslint\-7\.x] 1 new commit

aa322af Update dependency eslint to v7.16.0 - renovate-bot

#

renovate[bot]

[python-discord/forms-frontend] New branch created: renovate/eslint\-7\.x

#

renovate[bot]

[python-discord/forms-frontend] Pull request opened: #47 Update dependency eslint to v7\.16\.0

This PR contains the following updates:

Package	Change	Age	Adoption	Passing	Confidence
eslint (source)	`7.15.0` -> `7.16.0`	[](https://docs.renovatebot.com/mer...

#

[forms-frontend] GitHub Actions checks success on pull request #47

#

[forms-frontend] GitHub Code Scanning checks success on pull request #47

#

[forms-frontend] GitHub Actions checks success on pull request #47

#

[forms-frontend] Netlify checks success on pull request #47

oak estuaryBOT Dec 19, 2020, 1:28 AM

#

Python Bot

Connected!

#

Python Bot

Connected!

regal archBOT Dec 19, 2020, 1:40 AM

#

jb3

[bot:native\-gating] 1 new commit

a8a3e82 Handling pending flag changes on users - jb3

#

jb3

[python-discord/bot] New branch created: native\-gating

#

jb3

[python-discord/bot] Pull request opened: #1329 Better handle native gated users

Until Discord.py emits the pending property in member_update events we must use a workaround to watch for users who pass the native gate.

This PR creates an internal set of members who are pending and then eventually removes them once the pending property gets set to false. It is only a temporary solution before the pending event is handled by Discord.py.

#

jb3

[bot:native\-gating] 1 new commit

0be0d86 Correctly check if the user is pending - jb3

#

coveralls

[python-discord/bot] New comment on pull request #1329: Better handle native gated users

Coverage decreased (-0.03%) to 56.537% when pulling 0be0d86271c84ca1b2980b552aaf31c78e84fcda on native-gating into 14e71609e8e40be0832b66df7a0c309ba262659a on master.

#

mbaruh

[python-discord/bot] Pull request review submitted: #1329 Better handle native gated users

odd spireBOT Dec 19, 2020, 1:46 AM

#

[python-discord/bot] Checks Successful on PR: #1329 Better handle native gated users

GitHub Actions run 431716250 succeeded.

PR Author

Workflow Run

Lint & Test #164

Source Branch

native-gating

#

[python-discord/bot] Checks Successful on PR: #1329 Better handle native gated users

GitHub Actions run 431717980 succeeded.

PR Author

Workflow Run

Lint & Test #165

Source Branch

native-gating

regal archBOT Dec 19, 2020, 1:48 AM

#

Den4200

[python-discord/bot] Pull request review submitted: #1329 Better handle native gated users

Let's do this thing.

#

jb3

[python-discord/bot] Pull request closed: #1329 Better handle native gated users

#

jb3

[bot:master] 3 new commits

a8a3e82 Handling pending flag changes on users - jb3
0be0d86 Correctly check if the user is pending - jb3
80dd230 Merge pull request #1329 from python-discord/na... - jb3

#

jb3

[python-discord/bot] branch deleted: native\-gating

oak estuaryBOT Dec 19, 2020, 1:52 AM

#

Python Bot

Connected!

odd spireBOT Dec 19, 2020, 1:52 AM

#

[python-discord/bot] Workflow Successful

GitHub Actions run 431724574 succeeded.

Actor

Workflow Run

Deploy #60

Commit

80dd230

regal archBOT Dec 19, 2020, 2:12 AM

#

jb3

[bot:master] 1 new commit

5837921 Fix minor verification bugs - jb3

oak estuaryBOT Dec 19, 2020, 2:15 AM

#

Python Bot

Connected!

odd spireBOT Dec 19, 2020, 2:16 AM

#

[python-discord/bot] Workflow Successful

GitHub Actions run 431750191 succeeded.

Actor

Workflow Run

Deploy #61

Commit

5837921

oak estuaryBOT Dec 19, 2020, 2:17 AM

#

Python Bot

Connected!

regal archBOT Dec 19, 2020, 2:24 AM

#

jb3

[python-discord/bot] Issue opened: #1330 Use on\_member\_update for verification DMs

We want to DM users some information once they pass the native Discord verification gate. To do this we need to watch for the pending attribute of the user change from True to False.

As of now we have implemented this by keeping track of a list of users who need to receive a welcome DM in a set called pending_members in the Verification cog but we should switch this to use on_member_update(before, after) as soon as discord.py makes this available.

Once a user passes the nati...

regal archBOT Dec 19, 2020, 2:41 AM

#

jb3

[bot:native\-gating\-two\-user\-boogaloo] 3 new commits

fb4d82c Install git in Docker container - jb3
6ed94da Bump discord.py to a unreleased ref - jb3
9e61bed Update verification.py to use on_member_update,... - jb3

#

jb3

[python-discord/bot] New branch created: native\-gating\-two\-user\-boogaloo

#

jb3

[python-discord/bot] Pull request opened: #1331 Use member update event for verification DMs

Install git in Docker container
Bump discord.py to a unreleased ref with pending flag
Update verification.py to use on_member_update, closes #1330

odd spireBOT Dec 19, 2020, 2:43 AM

#

[python-discord/bot] Checks Failed on PR: #1331 Use member update event for verification DMs

GitHub Actions run 431780119 failed.

PR Author

Workflow Run

Lint & Test #168

Source Branch

native-gating-two-user-boogaloo

regal archBOT Dec 19, 2020, 2:48 AM

#

jb3

[bot:native\-gating\-two\-user\-boogaloo] 1 new commit

fe7f7ad Lock Pipfile - jb3

odd spireBOT Dec 19, 2020, 2:49 AM

#

[python-discord/bot] Checks Failed on PR: #1331 Use member update event for verification DMs

GitHub Actions run 431785506 failed.

PR Author

Workflow Run

Lint & Test #169

Source Branch

native-gating-two-user-boogaloo

regal archBOT Dec 19, 2020, 2:50 AM

#

jb3

[bot:native\-gating\-two\-user\-boogaloo] 1 new commit

17e1ca3 Fix linting errors - jb3

#

coveralls

[python-discord/bot] New comment on pull request #1331: Use member update event for verification DMs

Coverage increased (+0.08%) to 56.606% when pulling 17e1ca32651ca9c16d94afc9987fecb80a2ea176 on native-gating-two-user-boogaloo into 583792136152eb4b06ed0dda1bc0b95a0d8ebad1 on master.

odd spireBOT Dec 19, 2020, 2:53 AM

#

[python-discord/bot] Checks Successful on PR: #1331 Use member update event for verification DMs

GitHub Actions run 431787697 succeeded.

PR Author

Workflow Run

Lint & Test #170

Source Branch

native-gating-two-user-boogaloo

regal archBOT Dec 19, 2020, 2:54 AM

#

mbaruh

[python-discord/bot] Pull request review submitted: #1331 Use member update event for verification DMs

#

jb3

[bot:native\-gating\-two\-user\-boogaloo] 1 new commit

d6e842f Remove member_gating_cache RedisDict - jb3

odd spireBOT Dec 19, 2020, 2:57 AM

#

[python-discord/bot] Checks Successful on PR: #1331 Use member update event for verification DMs

GitHub Actions run 431792648 succeeded.

PR Author

Workflow Run

Lint & Test #171

Source Branch

native-gating-two-user-boogaloo

regal archBOT Dec 19, 2020, 3:05 AM

#

jb3

[bot:native\-gating\-two\-user\-boogaloo] 1 new commit

539aaa2 Update discord.py name in Pipfile - jb3

odd spireBOT Dec 19, 2020, 3:06 AM

#

[python-discord/bot] Checks Failed on PR: #1331 Use member update event for verification DMs

GitHub Actions run 431804424 failed.

PR Author

Workflow Run

Lint & Test #172

Source Branch

native-gating-two-user-boogaloo

regal archBOT Dec 19, 2020, 3:20 AM

#

jb3

[bot:native\-gating\-two\-user\-boogaloo] 1 new commit

ee8c6d1 relock lockfile - jb3

#

Den4200

[python-discord/bot] Pull request review submitted: #1331 Use member update event for verification DMs

Let's do it again!

odd spireBOT Dec 19, 2020, 3:23 AM

#

[python-discord/bot] Checks Successful on PR: #1331 Use member update event for verification DMs

GitHub Actions run 431820733 succeeded.

PR Author

Workflow Run

Lint & Test #173

Source Branch

native-gating-two-user-boogaloo

regal archBOT Dec 19, 2020, 3:26 AM

#

jb3

[python-discord/bot] Issue closed: #1330 Use on\_member\_update for verification DMs

#

jb3

[python-discord/bot] Pull request closed: #1331 Use member update event for verification DMs

#

jb3

[bot:master] 9 new commits

fb4d82c Install git in Docker container - jb3
6ed94da Bump discord.py to a unreleased ref - jb3
9e61bed Update verification.py to use on_member_update,... - jb3
fe7f7ad Lock Pipfile - jb3
17e1ca3 Fix linting errors - jb3

#

jb3

[python-discord/bot] branch deleted: native\-gating\-two\-user\-boogaloo

oak estuaryBOT Dec 19, 2020, 3:31 AM

#

Python Bot

Connected!

odd spireBOT Dec 19, 2020, 3:31 AM

#

[python-discord/bot] Workflow Successful

GitHub Actions run 431830615 succeeded.

Actor

Workflow Run

Deploy #62

Commit

c727723

regal archBOT Dec 19, 2020, 3:40 AM

#

jb3

[python-discord/site] New branch created: metricity\-temp\-fix

#

jb3

[site:metricity\-temp\-fix] 1 new commit

e100887 Swap verified_at for joined_at - jb3

#

jb3

[site:metricity\-temp\-fix] 1 new commit

eff717b Update verified_at fields to joined_at fields - jb3

#

jb3

[python-discord/site] Pull request opened: #438 Swap Metricity verified\_at for joined\_at

Swap out the verified_at column in metricity for joined_at.

This might be temporary while we find a solution for recording accurate verified dates in future, though for now we'll keep it as joined at (hence why the response has also been changed).

#

coveralls

[python-discord/site] New comment on pull request #438: Swap Metricity verified\_at for joined\_at

Coverage remained the same at 100.0% when pulling eff717bb38e5b556751ba4f23a53cc6662baa7dd on metricity-temp-fix into 9a203c5f3e1fc11e366b9f2b4e487e1756ec0c7c on master.

odd spireBOT Dec 19, 2020, 3:46 AM

#

[python-discord/site] Checks Successful on PR: #438 Swap Metricity verified_at for joined_at

GitHub Actions run 431844999 succeeded.

PR Author

Workflow Run

Lint & Test #60

Source Branch

metricity-temp-fix

regal archBOT Dec 19, 2020, 3:55 AM

#

jb3

[bot:master] 1 new commit

fa60dc9 Remove usage of joined_at metricity API item - jb3

odd spireBOT Dec 19, 2020, 3:56 AM

#

[python-discord/bot] Workflow Failed

GitHub Actions run 431855781 failed.

Actor

Workflow Run

Lint & Test #175

Commit

fa60dc9

regal archBOT Dec 19, 2020, 3:57 AM

#

jb3

[bot:master] 1 new commit

3be657e Remove unused dateutil imports - jb3

odd spireBOT Dec 19, 2020, 3:59 AM

#

[python-discord/bot] Workflow Failed

GitHub Actions run 431857470 failed.

Actor

Workflow Run

Lint & Test #176

Commit

3be657e

regal archBOT Dec 19, 2020, 4:01 AM

#

jb3

[bot:master] 1 new commit

688908d Fix pending tests - jb3

oak estuaryBOT Dec 19, 2020, 4:04 AM

#

Python Bot

Connected!

odd spireBOT Dec 19, 2020, 4:05 AM

#

[python-discord/bot] Workflow Successful

GitHub Actions run 431864906 succeeded.

Actor

Workflow Run

Deploy #65

Commit

688908d

regal archBOT Dec 19, 2020, 4:09 AM

#

Den4200

[bot:master] 1 new commit

620fa3f Downgrade markdownify from 0.6.0 to 0.5.3. - Den4200

#

jb3

[bot:master] 1 new commit

9755622 Fix invalid config name in voice gate - jb3

oak estuaryBOT Dec 19, 2020, 4:13 AM

#

Python Bot

Connected!

odd spireBOT Dec 19, 2020, 4:14 AM

#

[python-discord/bot] Workflow Successful

GitHub Actions run 431874109 succeeded.

Actor

Den4200

Workflow Run

Deploy #66

Commit

9755622

#

[python-discord/bot] Workflow Successful

GitHub Actions run 431875089 succeeded.

Actor

Workflow Run

Deploy #67

Commit

9755622

regal archBOT Dec 19, 2020, 4:22 AM

#

jb3

[bot:master] 2 new commits

c4545d8 Fix silence command to use guild default role - jb3
d76a1a6 kaizen: remove role check from bot account comm... - jb3

odd spireBOT Dec 19, 2020, 4:23 AM

#

[python-discord/bot] Workflow Failed

GitHub Actions run 431883569 failed.

Actor

Workflow Run

Lint & Test #180

Commit

d76a1a6

regal archBOT Dec 19, 2020, 4:23 AM

#

jb3

[bot:master] 1 new commit

bc98110 Remove unused import - jb3

odd spireBOT Dec 19, 2020, 4:25 AM

#

[python-discord/bot] Workflow Failed

GitHub Actions run 431884980 failed.

Actor

Workflow Run

Lint & Test #181

Commit

bc98110

regal archBOT Dec 19, 2020, 4:27 AM

#

jb3

[bot:master] 1 new commit

9799020 Fix silence tests - jb3

oak estuaryBOT Dec 19, 2020, 4:30 AM

#

Python Bot

Connected!

odd spireBOT Dec 19, 2020, 4:30 AM

#

[python-discord/bot] Workflow Successful

GitHub Actions run 431891054 succeeded.

Actor

Workflow Run

Deploy #70

Commit

9799020

odd spireBOT Dec 19, 2020, 8:09 AM

#

[python-discord/sir-lancebot] Checks Successful on PR: #509 Spook Name Rate Game

GitHub Actions run 432114184 succeeded.

PR Author

Vthechamp22

Workflow Run

Lint #142

Source Branch

Vthechamp22:master

regal archBOT Dec 19, 2020, 9:29 AM

#

ks129

[python-discord/bot] New comment on issue #1324: Add an optional argument for the resources command

I put this currently to stalled and planning status. We can currently discuss, is this good feature and worth implementing, but until Dewikification project is done, nothing can be actually done. Currently resources is stored in HTML article and there is not way to fetch these without writing complex HTML parsers. After Dewikification is done, we are able to build simple API for such data.

#

ks129

[python-discord/bot] New comment on issue #1327: Extend Remind and Bookmark Command Functionality

Sir Lancebot already have issue about getting copy of bookmark: python-discord/sir-lancebot#341.

regal archBOT Dec 19, 2020, 1:45 PM

#

ks129

[bot:ks123/sentry] 2 new commits

8f425fc Pump Sentry SDK version from 0.14 to 0.19 - ks129
4ebbb5d Create workflow for creating Sentry release - ks129

#

ks129

[python-discord/bot] New branch created: ks123/sentry

#

ks129

[python-discord/bot] Pull request opened: #1332 Sentry SDK version bump and Sentry release creation

Bumped Sentry SDK from 0.14 to 0.19.
Added workflow for creating Sentry releases (used site's one as base).

#

coveralls

[python-discord/bot] New comment on pull request #1332: Sentry SDK version bump and Sentry release creation

Coverage remained the same at 56.57% when pulling 4ebbb5db954fb82a66341cbae13ce6db7641f891 on ks123/sentry into 9799020de67c9350ee57f9ee3edff348a718cf6b on master.

odd spireBOT Dec 19, 2020, 1:52 PM

#

[python-discord/bot] Checks Successful on PR: #1332 Sentry SDK version bump and Sentry release creation

GitHub Actions run 432469710 succeeded.

PR Author

Workflow Run

Lint & Test #183

Source Branch

ks123/sentry

regal archBOT Dec 19, 2020, 2:07 PM

#

ks129

[sir-lancebot:ks123/sentry] 3 new commits

7735b43 Bump Sentry SDK from 0.14 to 0.19 - ks129
760e89c Add Redis and aiohttp integrations to Sentry - ks129
1a6effd Create Sentry release workflow - ks129

#

ks129

[python-discord/sir-lancebot] New branch created: ks123/sentry

#

ks129

[python-discord/sir-lancebot] Pull request opened: #547 Sentry SDK version bump, integrations adding and Sentry release workflow

Relevant Issues

Approved in #dev-core.

Description

Bumped Sentry SDK version from 0.14 to 0.19.
Added Redis and aiohttp integrations to Sentry SDK initializing.
Added workflow for creating Sentry releases (based on site's one).

Did you:

[x] Join the Python Discord Community?
[x] If dependencies have been added or updated, run pipenv lock?
[x] Lint your code (pipenv run lint)?
[x] Set the PR to **allo...

odd spireBOT Dec 19, 2020, 2:16 PM

#

[python-discord/sir-lancebot] Checks Failed on PR: #547 Sentry SDK version bump, integrations adding and Sentry release workflow

GitHub Actions run 432497989 failed.

PR Author

Workflow Run

Lint #143

Source Branch

ks123/sentry

regal archBOT Dec 19, 2020, 2:17 PM

#

jb3

[python-discord/sir-lancebot] Pull request review submitted: #547 Sentry SDK version bump, integrations adding and Sentry release workflow

We should really be initialising the Sentry SDK with the release label. I think @lemonsaurus implemented this on site so we should look there for how the SHA is fetched (we can either use a .git folder or inject the SHA into the Docker build, I think I prefer the latter) and passed into the Sentry SDK.

regal archBOT Dec 19, 2020, 5:19 PM

#

ks129

[sir-lancebot:ks123/sentry] 5 new commits

45b4058 Add Git SHA build argument to Dockerfile - ks129
3f821a4 Add default development value to Git SHA - ks129
1f48dc4 Put Git SHA to container on container build - ks129
7d3180c Add Git SHA as constant - ks129
fe3ecae Add release tag to Sentry SDK initialization - ks129

odd spireBOT Dec 19, 2020, 5:21 PM

#

[python-discord/sir-lancebot] Checks Successful on PR: #547 Sentry SDK version bump, integrations adding and Sentry release workflow

GitHub Actions run 432700686 succeeded.

PR Author

Workflow Run

Lint #144

Source Branch

ks123/sentry

regal archBOT Dec 19, 2020, 5:22 PM

#

jb3

[python-discord/sir-lancebot] Pull request review submitted: #547 Sentry SDK version bump, integrations adding and Sentry release workflow

#

jb3

[python-discord/sir-lancebot] New review comment on pull request #547: Sentry SDK version bump, integrations adding and Sentry release workflow

This can just be sir-lancebot.

    release=f"sir-lancebot@{GIT_SHA}"

#

jb3

[python-discord/sir-lancebot] New review comment on pull request #547: Sentry SDK version bump, integrations adding and Sentry release workflow

We concluded that we do not need the AioHttp integration, it is for servers.

#

ks129

[sir-lancebot:ks123/sentry] 1 new commit

826ff04 Remove aiohttp integration from Sentry - ks129

#

ks129

[sir-lancebot:ks123/sentry] 1 new commit

9392a2a Use sir-lancebot instead pydis-sir-lancebot for... - ks129

odd spireBOT Dec 19, 2020, 5:27 PM

#

[python-discord/sir-lancebot] Checks Successful on PR: #547 Sentry SDK version bump, integrations adding and Sentry release workflow

GitHub Actions run 432708313 succeeded.

PR Author

Workflow Run

Lint #145

Source Branch

ks123/sentry

regal archBOT Dec 19, 2020, 5:27 PM

#

ks129

[sir-lancebot:ks123/sentry] 1 new commit

9ec7482 Use sir-lancebot release prefix in workflow - ks129

odd spireBOT Dec 19, 2020, 5:27 PM

#

[python-discord/sir-lancebot] Checks Successful on PR: #547 Sentry SDK version bump, integrations adding and Sentry release workflow

GitHub Actions run 432708960 succeeded.

PR Author

Workflow Run

Lint #146

Source Branch

ks123/sentry

#

[python-discord/sir-lancebot] Checks Successful on PR: #547 Sentry SDK version bump, integrations adding and Sentry release workflow

GitHub Actions run 432709556 succeeded.

PR Author

Workflow Run

Lint #147

Source Branch

ks123/sentry

regal archBOT Dec 19, 2020, 5:54 PM

#

jb3

[metricity:master] 1 new commit

115f97d Don't track bot messages - jb3

#

jb3

[python-discord/sir-lancebot] Pull request review submitted: #547 Sentry SDK version bump, integrations adding and Sentry release workflow

#

[metricity] Lint code failure on master

#

jb3

[metricity:master] 1 new commit

481adbf Remove whitespace - jb3

#

[metricity] Build & Publish Docker image success on master

#

[metricity] Lint code success on master

#

[metricity] Build & Publish Docker image success on master

regal archBOT Dec 19, 2020, 7:08 PM

#

MarkKoz

[bot:bug/backend/bot\-j8/api\-session\-closed] 3 new commits

d744dc7 APIClient: simplify session creation - MarkKoz
2021c78 APIClient: remove obsolete function - MarkKoz
598c4a5 Bot: cease support of Bot.clear() - MarkKoz

#

MarkKoz

[python-discord/bot] New branch created: bug/backend/bot\-j8/api\-session\-closed

#

MarkKoz

[python-discord/bot] Pull request opened: #1333 Hopefully fix the API session being closed when populating the filter cache on startup

Fixes BOT-J8.

I've simplified the code for creation sessions upon startup. This was facilitated by dropping support for Bot.clear(), which we never relied on. I go into a bit more detail in the commit messages. I've ran the bot locally and don't have any issues with closed sessions upon startup.

odd spireBOT Dec 19, 2020, 7:20 PM

#

[python-discord/bot] Checks Failed on PR: #1333 Hopefully fix the API session being closed when populating the filter cache on startup

GitHub Actions run 432831920 failed.

PR Author

MarkKoz

Workflow Run

Lint & Test #184

Source Branch

bug/backend/bot-j8/api-session-closed

regal archBOT Dec 19, 2020, 7:20 PM

#

jb3

[python-discord/bot] Pull request review submitted: #1333 Hopefully fix the API session being closed when populating the filter cache on startup

Looking good. Works locally.

#

MarkKoz

[bot:bug/backend/bot\-j8/api\-session\-closed] 2 new commits

c58fbe8 APIClient: create the session directly in init - MarkKoz
52ee0da Remove obsolete test cases - MarkKoz

#

coveralls

[python-discord/bot] New comment on pull request #1333: Hopefully fix the API session being closed when populating the filter cache on startup

Coverage decreased (-0.03%) to 56.541% when pulling 52ee0dab1e59089a7acc9f08078dee0df3fa40e6 on bug/backend/bot-j8/api-session-closed into 9799020de67c9350ee57f9ee3edff348a718cf6b on master.

odd spireBOT Dec 19, 2020, 7:37 PM

#

[python-discord/bot] Checks Successful on PR: #1333 Hopefully fix the API session being closed when populating the filter cache on startup

GitHub Actions run 432849750 succeeded.

PR Author

MarkKoz

Workflow Run

Lint & Test #185

Source Branch

bug/backend/bot-j8/api-session-closed

regal archBOT Dec 19, 2020, 7:41 PM

#

MarkKoz

[bot:bug/precommit\-pycharm] 1 new commit

d3492b9 Fix flake8 pre-commit hook running through PyCharm - MarkKoz

#

MarkKoz

[python-discord/bot] New branch created: bug/precommit\-pycharm

#

MarkKoz

[python-discord/bot] Pull request opened: #1334 Fix flake8 pre\-commit hook running through PyCharm

Running the hook through PyCharm used to work a long time ago due to a pipenv bug which eventually got fixed. Now it normally fails because PIPENV_IGNORE_VIRTUALENVS is unset. Manually editing the hook and adding os.putenv('PIPENV_IGNORE_VIRTUALENVS', '1') or similar fixes it.

Recently, I've found out about language: system for the hook, which is a better alternative to setting the environment variable. As per https://github.com/pre-commit/pre-commit/issues/1412#issuecomment-61926374...

#

coveralls

[python-discord/bot] New comment on pull request #1334: Fix flake8 pre\-commit hook running through PyCharm

Coverage remained the same at 56.57% when pulling d3492b961618f4d6d0ec6622cffda3dadcd4b7a4 on bug/precommit-pycharm into 9799020de67c9350ee57f9ee3edff348a718cf6b on master.

odd spireBOT Dec 19, 2020, 7:49 PM

#

[python-discord/bot] Checks Successful on PR: #1334 Fix flake8 pre-commit hook running through PyCharm

GitHub Actions run 432863219 succeeded.

PR Author

MarkKoz

Workflow Run

Lint & Test #186

Source Branch

bug/precommit-pycharm

regal archBOT Dec 19, 2020, 8:59 PM

#

HassanAbouelela

[python-discord/bot] New comment on issue #1327: Extend Remind and Bookmark Command Functionality

Thanks, edited my comment to remove the bookmark part.

regal archBOT Dec 19, 2020, 10:50 PM

#

jb3

[metricity:master] 3 new commits

dc455e8 Remove verification features - jb3
229e94b Version bump to 1.1.0 - jb3
43a963e Merge branch 'master' of github.com:python-disc... - jb3

#

[metricity] Lint code failure on master

#

[metricity] Build & Publish Docker image success on master

#

jb3

[metricity:master] 1 new commit

2e3c82e Remove unused import - jb3

#

[metricity] Lint code success on master

#

[metricity] Build & Publish Docker image success on master

regal archBOT Dec 19, 2020, 11:19 PM

#

jb3

[python-discord/site] Pull request closed: #438 Swap Metricity verified\_at for joined\_at

#

jb3

[site:master] 3 new commits

e100887 Swap verified_at for joined_at - jb3
eff717b Update verified_at fields to joined_at fields - jb3
9595e45 Merge pull request #438 from python-discord/met... - jb3

#

jb3

[python-discord/site] branch deleted: metricity\-temp\-fix

odd spireBOT Dec 19, 2020, 11:23 PM

#

[python-discord/site] Workflow Successful

GitHub Actions run 433084769 succeeded.

Actor

Workflow Run

Deploy #15

Commit

9595e45

regal archBOT Dec 20, 2020, 5:23 AM

#

Xithrius

[python-discord/bot] Pull request review submitted: #1269 User Events Management system

#

Xithrius

[python-discord/bot] New review comment on pull request #1269: User Events Management system

Why is this a constant, if it's only used once?

regal archBOT Dec 20, 2020, 6:22 AM

#

Xithrius

[python-discord/bot] New comment on pull request #1166: Enhance the \!server command\.

Updating:

The master branch now runs off of discord.py 1.5.1, and #1196 has been merged.

regal archBOT Dec 20, 2020, 7:05 AM

#

ks129

[bot:ks123/sentry] 6 new commits

756fa37 Consume Git SHA build arg and add to it to envi... - ks129
8da3423 Inject Git SHA in container build workflow - ks129
33ad294 Add constant for Git SHA - ks129
4c7ada8 Attach release on Sentry SDK initialization - ks129
e7ca3af Use bot prefix instead pydis-bot for Sentry rel... - ks129

#

ks129

[python-discord/bot] Pull request review submitted: #1269 User Events Management system

#

ks129

[python-discord/bot] New review comment on pull request #1269: User Events Management system

These strings had getter functions before, I asked moving them from functions to constants.

odd spireBOT Dec 20, 2020, 7:08 AM

#

[python-discord/bot] Checks Successful on PR: #1332 Sentry SDK version bump and Sentry release creation

GitHub Actions run 433542697 succeeded.

PR Author

Workflow Run

Lint & Test #187

Source Branch

ks123/sentry

regal archBOT Dec 20, 2020, 7:08 AM

#

ks129

[python-discord/bot] New comment on pull request #1332: Sentry SDK version bump and Sentry release creation

Implemented Git SHA injecting to container and using release for Sentry SDK initialization

#

ks129

[python-discord/bot] Pull request review submitted: #1333 Hopefully fix the API session being closed when populating the filter cache on startup

Code looks good, tested locally.

#

MarkKoz

[python-discord/bot] Pull request review submitted: #1325 Add dynamic message in \#how\-to\-get\-help

Thanks for your first contribution. Unfortunately, this PR is currently pretty far off the mark. I get the sense you did not attempt to run and test this feature. I understand setting up a dev environment in which to run the bot is tedious at the moment, but I encourage you to try to run it locally to test your changes before pushing them.

Speaking of dev environments, you really should get the pre-commit hook set up at the least. This is written in our contributing guidelines. Please re...

#

MarkKoz

[python-discord/bot] New review comment on pull request #1325: Add dynamic message in \#how\-to\-get\-help

This should be in a separate function which is defined in _message.py and called by move_to_available. The scope of get_available_candidate should remain narrow: to get a candidate channel for the available category and nothing more.

#

MarkKoz

[python-discord/bot] New review comment on pull request #1325: Add dynamic message in \#how\-to\-get\-help

fetch_message doesn't return None; it raises various exceptions which need to be handled.

#

MarkKoz

[python-discord/bot] New review comment on pull request #1325: Add dynamic message in \#how\-to\-get\-help

The default is to get 100 messages. Set the limit to 1.

#

ks129

[python-discord/bot] Pull request closed: #1333 Hopefully fix the API session being closed when populating the filter cache on startup

#

ks129

[bot:master] 6 new commits

d744dc7 APIClient: simplify session creation - MarkKoz
2021c78 APIClient: remove obsolete function - MarkKoz
598c4a5 Bot: cease support of Bot.clear() - MarkKoz
c58fbe8 APIClient: create the session directly in init - MarkKoz
52ee0da Remove obsolete test cases - MarkKoz

#

ks129

[python-discord/bot] branch deleted: bug/backend/bot\-j8/api\-session\-closed

oak estuaryBOT Dec 20, 2020, 7:21 AM

#

Python Bot

Connected!

odd spireBOT Dec 20, 2020, 7:21 AM

#

[python-discord/bot] Workflow Successful

GitHub Actions run 433557794 succeeded.

Actor

Workflow Run

Deploy #71

Commit

3fbb4db

regal archBOT Dec 20, 2020, 7:27 AM

#

sentry-io[bot]

[python-discord/bot] Issue opened: #1335 refresh\_peps\_urls should check the response status code

Sentry Issue: BOT-JD

In the case below, the GitHub rate limit was exceed and a 403 was returned.

TypeError: string indices must be integers
  File "bot/exts/utils/utils.py", line 224, in refresh_peps_urls
    name = file["name"]

Task exception was never retrieved
future: <Task finished name='Task-10' coro= exception=TypeError('string indices must be integers')>

#

sentry-io[bot]

[python-discord/bot] Issue opened: #1336 validate\_pep\_number should acount for last\_refreshed\_peps being None

Sentry Issue: BOT-JE

last_refreshed_peps may be unset if an error was encountered with refresh_peps_urls, such as the rate limit in #1335

TypeError: unsupported operand type(s) for +: 'NoneType' and 'datetime.timedelta'
  File "discord/ext/commands/core.py", line 85, in wrapped
    ret = await coro(*args, **kwargs)
  File "bot/exts/utils/utils.py", line 244, in pep_command
    if not (pep...

regal archBOT Dec 20, 2020, 8:31 AM

#

ks129

[bot:ks123/pep] 6 new commits

477af4e Add GitHub API key to config as environment var... - ks129
de3dd22 Move PEP related functions and command to own cog - ks129
d9fed58 Set last PEPs listing at beginning of function - ks129
b7ab159 Make last PEPs listing refresh non-optional - ks129
4527f9a Log warning and return early when can't get PEP... - ks129

#

ks129

[python-discord/bot] New branch created: ks123/pep

#

ks129

[bot:ks123/pep] 1 new commit

9870072 Remove unused constant - ks129

regal archBOT Dec 20, 2020, 8:56 AM

#

ks129

[python-discord/bot] Pull request opened: #1337 Move PEP command and utils to own cog, fix bug related not checking status and add GitHub token

Note: Before merging this PR, DevOps team should add GITHUB_API_TOKEN environment variable.

Should be merged fast, PEP command is broken currently because of rate limits.

Moved PEP related utilities and command to own cog.
Make last refresh non-optional.
Catch non-200 responses.
Add GitHub token.
Closes #1335
Closes #1336

#

coveralls

[python-discord/bot] New comment on pull request #1337: Move PEP command and utils to own cog, fix bug related not checking status and add GitHub token

Coverage increased (+0.01%) to 56.551% when pulling 9870072310e5a2d1ccf6d5a035d1f1044343ff5f on ks123/pep into 3fbb4db5c539ca3580342e5c4fc8ecb61bd2de42 on master.

odd spireBOT Dec 20, 2020, 8:59 AM

#

[python-discord/bot] Checks Successful on PR: #1337 Move PEP command and utils to own cog, fix bug related not checking status and add GitHub token

GitHub Actions run 433653552 succeeded.

PR Author

Workflow Run

Lint & Test #189

Source Branch

ks123/pep

regal archBOT Dec 20, 2020, 9:03 AM

#

ks129

[bot:ks123/pep] 7 new commits

d744dc7 APIClient: simplify session creation - MarkKoz
2021c78 APIClient: remove obsolete function - MarkKoz
598c4a5 Bot: cease support of Bot.clear() - MarkKoz
c58fbe8 APIClient: create the session directly in init - MarkKoz
52ee0da Remove obsolete test cases - MarkKoz

odd spireBOT Dec 20, 2020, 9:06 AM

#

[python-discord/bot] Checks Successful on PR: #1337 Move PEP command and utils to own cog, fix bug related not checking status and add GitHub token

GitHub Actions run 433660917 succeeded.

PR Author

Workflow Run

Lint & Test #190

Source Branch

ks123/pep

regal archBOT Dec 20, 2020, 10:57 AM

#

SebastiaanZ

[python-discord/async-rediscache] New comment on issue #8: Proposed changes regarding generics and symmetry with dictionaries

Increment and decrement now return the new value in version 0.3.0.

Could you explain your idea about generics for the type? The types supported by the framework are specific: By supporting generics on the various Redis types, we may give the impression that we support storing generic types. We don't, we only support the four specific types listed in the alias and we use the aliases to indicate which specific types we expect for the various arguments (and return types).

Now, my experienc...

regal archBOT Dec 20, 2020, 1:48 PM

#

ks129

[site:ks123/sentry] 5 new commits

bc2786d Bump Sentry SDK version from 0.14 to 0.19 - ks129
b6e13bc Add Git SHA consuming and putting to environmen... - ks129
73feb02 Set Git SHA during build workflow - ks129
c6d93bc Get Git SHA from environement instead of GitPyt... - ks129
01eaf48 Remove GitPython dependency - ks129

#

ks129

[python-discord/site] New branch created: ks123/sentry

#

ks129

[site:ks123/sentry] 1 new commit

03845e7 Don't include .git in Docker container - ks129

#

ks129

[python-discord/site] Pull request opened: #439 Sentry SDK version bump, Git SHA simplification

Bumped Sentry SDK from 0.14 to 0.19.
Inject Git SHA to container using build arguments.
Use Git SHA from environment instead from GitPython package.
Removed GitPython package.
Added .git directory back to .dockerignore.

#

coveralls

[python-discord/site] New comment on pull request #439: Sentry SDK version bump, Git SHA simplification

Coverage remained the same at 100.0% when pulling 03845e72c41d901e729209cbfbb8c9abdeebde2d on ks123/sentry into 9595e4546882211edd792f0ce73e429772ce5b7b on master.

odd spireBOT Dec 20, 2020, 1:55 PM

#

[python-discord/site] Checks Successful on PR: #439 Sentry SDK version bump, Git SHA simplification

GitHub Actions run 433968821 succeeded.

PR Author

Workflow Run

Lint & Test #62

Source Branch

ks123/sentry

regal archBOT Dec 20, 2020, 2:04 PM

#

ks129

[snekbox:ks123/sentry] 4 new commits

8135a65 Inject Git SHA to environment from build argument - ks129
b588af9 Add Git SHA build argument to Docker build in w... - ks129
eb7c21d Create workflow for Sentry release creation - ks129
e97f79a Add Sentry release to SDK initialization - ks129

#

ks129

[python-discord/snekbox] New branch created: ks123/sentry

#

ks129

[python-discord/snekbox] Pull request opened: #85 Implement Sentry releases

Added injecting Git SHA to container using build arguments. On SDK initialization this uses this SHA for release defining. Also added workflow for creating releases.

#

[snekbox] GitHub Actions checks startup failure on pull request #85

#

[snekbox] GitHub Actions checks startup failure on pull request #85

regal archBOT Dec 20, 2020, 3:20 PM

#

ks129

[sir-lancebot:ks123/sentry] 1 new commit

3e1ddb2 Fix wrong way for getting Git SHA - ks129

odd spireBOT Dec 20, 2020, 3:21 PM

#

[python-discord/sir-lancebot] Checks Successful on PR: #547 Sentry SDK version bump, integrations adding and Sentry release workflow

GitHub Actions run 434066635 succeeded.

PR Author

Workflow Run

Lint #148

Source Branch

ks123/sentry

regal archBOT Dec 20, 2020, 3:22 PM

#

ks129

[bot:ks123/sentry] 1 new commit

d598899 Fix wrong way for getting Git SHA - ks129

#

ks129

[site:ks123/sentry] 1 new commit

e5223de Fix Git SHA getting in workflow - ks129

odd spireBOT Dec 20, 2020, 3:24 PM

#

[python-discord/bot] Checks Successful on PR: #1332 Sentry SDK version bump and Sentry release creation

GitHub Actions run 434068806 succeeded.

PR Author

Workflow Run

Lint & Test #191

Source Branch

ks123/sentry

#

[python-discord/site] Checks Successful on PR: #439 Sentry SDK version bump, Git SHA simplification

GitHub Actions run 434070246 succeeded.

PR Author

Workflow Run

Lint & Test #63

Source Branch

ks123/sentry

regal archBOT Dec 20, 2020, 3:25 PM

#

ks129

[snekbox:ks123/sentry] 1 new commit

1a3c666 Fix getting Git SHA in workflow - ks129

#

coveralls

[python-discord/snekbox] New comment on pull request #85: Implement Sentry releases

Coverage increased (+0.04%) to 93.421% when pulling 1a3c666ad2887e1199be69d636b33fdab41447c9 on ks123/sentry into 08f1fb4d9e50e88ae288b7ebc1abb4ab06f491f5 on master.

#

[snekbox] GitHub Actions checks success on pull request #85

regal archBOT Dec 20, 2020, 3:56 PM

#

Akarys42

[bot:master] 1 new commit

6115f5a Add the clear alias to the clean command - Akarys42

oak estuaryBOT Dec 20, 2020, 3:59 PM

#

Python Bot

Connected!

odd spireBOT Dec 20, 2020, 4:00 PM

#

[python-discord/bot] Workflow Successful

GitHub Actions run 434109568 succeeded.

Actor

Akarys42

Workflow Run

Deploy #72

Commit

6115f5a

regal archBOT Dec 20, 2020, 4:32 PM

#

UgurCanAkcay

[python-discord/bot] Fork created: UgurCanAkcay/bot

#

swfarnsworth

[python-discord/bot] New comment on pull request #1328: Manual verification with \`\!verify\` sends the user the confirmation DM\.

I didn't realize the native gate rollout was imminent when making this PR. I'll submit a new version that checks if the user is pending, and if not, assigns and subsequently removes the developer role. I'll probably also rename the function accordingly.

regal archBOT Dec 20, 2020, 5:01 PM

#

MarkKoz

[python-discord/snekbox] Pull request review submitted: #85 Implement Sentry releases

What are the consequences of making a sentry release for a SHA that doesn't go into production (e.g. cause a build failed)? Does it not matter?

#

ks129

[python-discord/snekbox] New comment on pull request #85: Implement Sentry releases

@MarkKoz So much how I understand by reading docs, this doesn't affect anything. This will be just empty release then.

#

MarkKoz

[python-discord/snekbox] Pull request review submitted: #85 Implement Sentry releases

#

MarkKoz

[python-discord/snekbox] Pull request closed: #85 Implement Sentry releases

#

MarkKoz

[snekbox:master] 6 new commits

8135a65 Inject Git SHA to environment from build argument - ks129
b588af9 Add Git SHA build argument to Docker build in w... - ks129
eb7c21d Create workflow for Sentry release creation - ks129
e97f79a Add Sentry release to SDK initialization - ks129
1a3c666 Fix getting Git SHA in workflow - ks129

#

MarkKoz

[python-discord/snekbox] branch deleted: ks123/sentry

#

[snekbox] GitHub Actions checks success on master

#

MarkKoz

[python-discord/site] Pull request review submitted: #439 Sentry SDK version bump, Git SHA simplification

#

MarkKoz

[python-discord/bot] Pull request review submitted: #1332 Sentry SDK version bump and Sentry release creation

#

[snekbox] GitHub Actions checks success on master

regal archBOT Dec 20, 2020, 6:05 PM

#

yahya-dts

[python-discord/bot] Fork created: yahya-dts/bot

regal archBOT Dec 20, 2020, 6:40 PM

#

MarkKoz

[python-discord/sir-lancebot] Pull request review submitted: #547 Sentry SDK version bump, integrations adding and Sentry release workflow

#

ks129

[python-discord/sir-lancebot] Pull request closed: #547 Sentry SDK version bump, integrations adding and Sentry release workflow

#

ks129

[python-discord/sir-lancebot] branch deleted: ks123/sentry

#

ks129

[sir-lancebot:master] 13 new commits

7735b43 Bump Sentry SDK from 0.14 to 0.19 - ks129
760e89c Add Redis and aiohttp integrations to Sentry - ks129
1a6effd Create Sentry release workflow - ks129
45b4058 Add Git SHA build argument to Dockerfile - ks129
3f821a4 Add default development value to Git SHA - ks129

night lilyBOT Dec 20, 2020, 6:47 PM

#

Sir Lancebot

Connected!

odd spireBOT Dec 20, 2020, 6:47 PM

#

[python-discord/sir-lancebot] Workflow Successful

GitHub Actions run 434297536 succeeded.

Actor

Workflow Run

Build #85

Commit

1cdcd23

regal archBOT Dec 20, 2020, 8:20 PM

#

HassanAbouelela

[python-discord/forms-backend] Pull request review submitted: #39 Adds Discord Embed on Form Submission

#

HassanAbouelela

[python-discord/forms-backend] New review comment on pull request #39: Adds Discord Embed on Form Submission

So the most probable cause for this to happen would be if the webhook feature was enabled, but the webhook link wasn't entered/was removed from the website. This might happen if we for example have an on/off toggle for the hooks, and a separate field to enter the URLs.

What this means, is that it is probably not an error, rather the form is just waiting on a URL to be setup. Ideally, this would be some sort of warning icon on the frontend. If we want to alert the user to an error happening...

regal archBOT Dec 20, 2020, 11:48 PM

#

renovate[bot]

[forms-frontend:renovate/emotion\-monorepo] 1 new commit

96c0827 Update dependency @emotion/react to v11.1.3 - renovate-bot

#

renovate[bot]

[python-discord/forms-frontend] New branch created: renovate/emotion\-monorepo

#

renovate[bot]

[python-discord/forms-frontend] Pull request opened: #48 Update dependency @emotion/react to v11\.1\.3

This PR contains the following updates:

Package	Change	Age	Adoption	Passing	Confidence
@emotion/react	`11.1.2` -> `11.1.3`	[](https://docs.renovatebot.com/m...

#

[forms-frontend] GitHub Actions checks success on pull request #48

#

jb3

[python-discord/forms-frontend] Pull request review submitted: #47 Update dependency eslint to v7\.16\.0

#

jb3

[python-discord/forms-frontend] Pull request closed: #47 Update dependency eslint to v7\.16\.0

#

jb3

[forms-frontend:main] 2 new commits

aa322af Update dependency eslint to v7.16.0 - renovate-bot
8737aca Merge pull request #47 from python-discord/reno... - jb3

#

jb3

[python-discord/forms-frontend] branch deleted: renovate/eslint\-7\.x

#

[forms-frontend] GitHub Code Scanning checks success on pull request #48

#

[forms-frontend] GitHub Actions checks success on pull request #48

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] Netlify checks success on pull request #48

#

[forms-frontend] GitHub Actions checks success on main

#

jb3

[python-discord/forms-frontend] Pull request review submitted: #48 Update dependency @emotion/react to v11\.1\.3

#

jb3

[python-discord/forms-frontend] Pull request closed: #48 Update dependency @emotion/react to v11\.1\.3

#

jb3

[forms-frontend:main] 2 new commits

96c0827 Update dependency @emotion/react to v11.1.3 - renovate-bot
75a82ab Merge pull request #48 from python-discord/reno... - jb3

#

renovate[bot]

[python-discord/forms-frontend] branch deleted: renovate/emotion\-monorepo

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

jb3

[python-discord/forms-backend] Pull request review submitted: #40 Add PATCH method to form endpoint

regal archBOT Dec 21, 2020, 12:48 AM

#

jb3

[python-discord/forms-backend] Pull request closed: #40 Add PATCH method to form endpoint

#

jb3

[forms-backend:main] 6 new commits

30de873 Add PATCH method to form endpoint - ks129
3ab9ef5 Merge branch 'main' into ks123/form-patch - ks129
45db68a Merge branch 'main' into ks123/form-patch - ks129
473f61d Actually update form in DB in PATCH method - ks129
1b82bac Update status code of pydantic errors response - ks129

#

jb3

[python-discord/forms-backend] branch deleted: ks123/form\-patch

odd spireBOT Dec 21, 2020, 12:50 AM

#

[python-discord/forms-backend] Workflow Successful

GitHub Actions run 434700599 succeeded.

Actor

Workflow Run

Forms Backend #146

Commit

1f5446f

regal archBOT Dec 21, 2020, 12:51 AM

#

jb3

[python-discord/forms-backend] Pull request review submitted: #39 Adds Discord Embed on Form Submission

#

jb3

[python-discord/forms-backend] New review comment on pull request #39: Adds Discord Embed on Form Submission

Gotcha, the feature and webhook URL will most likely be separate fields (e.g. enable webhook feature, then a text box is displayed).

We can probably validate on the frontend that there is not a point where the webhook feature is enabled when no webhook is set.

regal archBOT Dec 21, 2020, 1:46 AM

#

HassanAbouelela

[forms-backend:discord\-webhook] 13 new commits

30de873 Add PATCH method to form endpoint - ks129
6d3b7d9 Remove DNS blacklist handling from submit route - ks129
de0a76c Remove DNS blacklist field from antispam model - ks129
9a54873 Remove pydnsbl dependency - ks129
3ab9ef5 Merge branch 'main' into ks123/form-patch - ks129

odd spireBOT Dec 21, 2020, 1:47 AM

#

[python-discord/forms-backend] Checks Successful on PR: #39 Adds Discord Embed on Form Submission

GitHub Actions run 434768751 succeeded.

PR Author

Workflow Run

Forms Backend #147

Source Branch

discord-webhook

regal archBOT Dec 21, 2020, 4:51 AM

#

Den4200

[python-discord/bot] Pull request review submitted: #1332 Sentry SDK version bump and Sentry release creation

#

Den4200

[bot:ks123/sentry] 8 new commits

d744dc7 APIClient: simplify session creation - MarkKoz
2021c78 APIClient: remove obsolete function - MarkKoz
598c4a5 Bot: cease support of Bot.clear() - MarkKoz
c58fbe8 APIClient: create the session directly in init - MarkKoz
52ee0da Remove obsolete test cases - MarkKoz

#

Den4200

[python-discord/site] Pull request review submitted: #439 Sentry SDK version bump, Git SHA simplification

odd spireBOT Dec 21, 2020, 4:54 AM

#

[python-discord/bot] Checks Successful on PR: #1332 Sentry SDK version bump and Sentry release creation

GitHub Actions run 434995732 succeeded.

PR Author

Workflow Run

Lint & Test #193

Source Branch

ks123/sentry

regal archBOT Dec 21, 2020, 6:03 AM

#

RohanJnr

[python-discord/bot] Pull request review submitted: #1269 User Events Management system

#

RohanJnr

[python-discord/bot] New review comment on pull request #1269: User Events Management system

what if I check if get_channel or get_role returns None, then I unload the cog with appropriate logging instead of fetch_channel raising an error and unloading the cog?

#

ks129

[python-discord/bot] Pull request review submitted: #1269 User Events Management system

#

ks129

[python-discord/bot] New review comment on pull request #1269: User Events Management system

We must be ready that channels or role(s) is not in cache, so we need to fetch them.

regal archBOT Dec 21, 2020, 6:45 AM

#

ks129

[python-discord/sir-lancebot] Issue opened: #548 Migrate IGDB v3 API to v4 for games command

Description

IGDB shut down APIv3 in October 2020. games command is currently built to use v3 of API, but now, when URLs is gone, this command is currently not usable. Command must be migrated APIv4, about to what migrating can be read https://api-docs.igdb.com/#upgrading-to-v4-from-v3.

Steps to Reproduce

When you have API key of IGDB, then genres refreshing task fails, because API not found, also command fails.

Expected Behaviour

Should successfully refresh genres an...

#

sentry-io[bot]

[python-discord/sir-lancebot] New comment on issue #548: Migrate IGDB v3 API to v4 for games command

Sentry issue: SIR-LANCEBOT-1S

odd spireBOT Dec 21, 2020, 10:16 AM

#

[python-discord/sir-lancebot] Checks Successful on PR: #509 Spook Name Rate Game

GitHub Actions run 435516322 succeeded.

PR Author

Vthechamp22

Workflow Run

Lint #150

Source Branch

Vthechamp22:master

regal archBOT Dec 21, 2020, 1:52 PM

#

ks129

[python-discord/sir-lancebot] Pull request review submitted: #509 Spook Name Rate Game

#

ks129

[python-discord/sir-lancebot] New review comment on pull request #509: Spook Name Rate Game

    debug = getenv('BOT_DEBUG', False)  # Enable if you do not want to limit the commands to October or if you do not want

regal archBOT Dec 21, 2020, 2:42 PM

#

jb3

[python-discord/bot] Pull request review submitted: #1332 Sentry SDK version bump and Sentry release creation

#

jb3

[python-discord/bot] Pull request closed: #1332 Sentry SDK version bump and Sentry release creation

#

jb3

[python-discord/bot] branch deleted: ks123/sentry

#

jb3

[bot:master] 11 new commits

8f425fc Pump Sentry SDK version from 0.14 to 0.19 - ks129
4ebbb5d Create workflow for creating Sentry release - ks129
756fa37 Consume Git SHA build arg and add to it to envi... - ks129
8da3423 Inject Git SHA in container build workflow - ks129
33ad294 Add constant for Git SHA - ks129

#

jb3

[python-discord/site] Pull request review submitted: #439 Sentry SDK version bump, Git SHA simplification

Let's take this as an opportunity to change from the pydis-site prefix to site

oak estuaryBOT Dec 21, 2020, 2:49 PM

#

Python Bot

Connected!

odd spireBOT Dec 21, 2020, 2:49 PM

#

[python-discord/bot] Workflow Successful

GitHub Actions run 435993797 succeeded.

Actor

Workflow Run

Deploy #73

Commit

7aec121

regal archBOT Dec 21, 2020, 2:51 PM

#

ks129

[site:ks123/sentry] 1 new commit

ebef198 Change pydis-site to site in workflow - ks129

#

ks129

[site:ks123/sentry] 1 new commit

ad6d901 Change pydis-site to site in Sentry SDK initial... - ks129

odd spireBOT Dec 21, 2020, 2:53 PM

#

[python-discord/site] Checks Successful on PR: #439 Sentry SDK version bump, Git SHA simplification

GitHub Actions run 435999237 succeeded.

PR Author

Workflow Run

Lint & Test #64

Source Branch

ks123/sentry

#

[python-discord/site] Checks Successful on PR: #439 Sentry SDK version bump, Git SHA simplification

GitHub Actions run 436001714 succeeded.

PR Author

Workflow Run

Lint & Test #65

Source Branch

ks123/sentry

regal archBOT Dec 21, 2020, 2:55 PM

#

jb3

[python-discord/site] Pull request review submitted: #439 Sentry SDK version bump, Git SHA simplification

#

jb3

[python-discord/site] Pull request closed: #439 Sentry SDK version bump, Git SHA simplification

#

jb3

[site:master] 10 new commits

bc2786d Bump Sentry SDK version from 0.14 to 0.19 - ks129
b6e13bc Add Git SHA consuming and putting to environmen... - ks129
73feb02 Set Git SHA during build workflow - ks129
c6d93bc Get Git SHA from environement instead of GitPyt... - ks129
01eaf48 Remove GitPython dependency - ks129

#

jb3

[python-discord/site] branch deleted: ks123/sentry

odd spireBOT Dec 21, 2020, 3:00 PM

#

[python-discord/site] Workflow Successful

GitHub Actions run 436013075 succeeded.

Actor

Workflow Run

Deploy #16

Commit

36dc580

regal archBOT Dec 21, 2020, 3:26 PM

#

ks129

[python-discord/site] New comment on issue #362: Implement a dark theme

As python-discord/django-simple-bulma#46 is closed, I'm moving this issue out from stalled status.

regal archBOT Dec 21, 2020, 4:54 PM

#

coveralls

[python-discord/site] New comment on pull request #410: User events management

Coverage remained the same at 100.0% when pulling fdcd5176c9c384542df3ffbe6443412fbdf1ec68 on RohanJnr:user_events_management into 36dc58008181944ff78b85f9ac64b7b3e089b10c on python-discord:master.

odd spireBOT Dec 21, 2020, 4:54 PM

#

[python-discord/site] Checks Successful on PR: #410 User events management

GitHub Actions run 436228116 succeeded.

PR Author

RohanJnr

Workflow Run

Lint & Test #67

Source Branch

RohanJnr:user_events_management

regal archBOT Dec 21, 2020, 5:45 PM

#

jb3

[metricity:master] 2 new commits

8f688f8 Add support for pending members - jb3
1bba515 Version bump - jb3

#

[metricity] Build & Publish Docker image failure on master

#

[metricity] Lint code success on master

#

jb3

[metricity:master] 1 new commit

5a84ac8 Add git to production image - jb3

#

[metricity] Lint code success on master

#

[metricity] Build & Publish Docker image success on master

odd spireBOT Dec 21, 2020, 6:17 PM

#

[python-discord/bot] Checks Successful on PR: #1269 User Events Management system

GitHub Actions run 436375523 succeeded.

PR Author

RohanJnr

Workflow Run

Lint & Test #195

Source Branch

RohanJnr:user_events_management

regal archBOT Dec 21, 2020, 6:22 PM

#

renovate[bot]

[forms-frontend:renovate/typescript\-eslint\-monorepo] 1 new commit

e50af0a Update typescript-eslint monorepo to v4.11.0 - renovate-bot

#

renovate[bot]

[python-discord/forms-frontend] New branch created: renovate/typescript\-eslint\-monorepo

#

renovate[bot]

[python-discord/forms-frontend] Pull request opened: #49 Update typescript\-eslint monorepo to v4\.11\.0

This PR contains the following updates:

Package	Change	Age	Adoption	Passing	Confidence
@typescript-eslint/eslint-plugin	`4.10.0` -> `4.11.0`	[![age](https://badges.renovateapi.com/packages/npm/@typescript-...

#

[forms-frontend] GitHub Actions checks success on pull request #49

#

[forms-frontend] Netlify checks success on pull request #49

#

[forms-frontend] GitHub Code Scanning checks success on pull request #49

#

[forms-frontend] GitHub Actions checks success on pull request #49

#

ks129

[python-discord/forms-backend] New branch created: big\-int\-fix

#

ks129

[forms-backend:big\-int\-fix] 1 new commit

faeb656 Convert user IDs to string to avoid JS bad beha... - ks129

#

ks129

[python-discord/forms-backend] Pull request opened: #42 Convert user IDs to string to avoid JS bad behaviours

Changed ID type to string and added pre validator for converting ints to string.

odd spireBOT Dec 21, 2020, 6:33 PM

#

[python-discord/forms-backend] Checks Successful on PR: #42 Convert user IDs to string to avoid JS bad behaviours

GitHub Actions run 436405617 succeeded.

PR Author

Workflow Run

Forms Backend #148

Source Branch

big-int-fix

regal archBOT Dec 21, 2020, 6:44 PM

#

jb3

[python-discord/forms-backend] Pull request review submitted: #42 Convert user IDs to string to avoid JS bad behaviours

Thanks for the contribution!

#

jb3

[python-discord/forms-backend] New review comment on pull request #42: Convert user IDs to string to avoid JS bad behaviours

This validator isn't required, Pydantic will automatically cast this data:

>>> from pydantic import BaseModel
>>> class MyObject(BaseModel):
...   user_id: str
... 
>>> MyObject(user_id="1")
MyObject(user_id='1')
>>> MyObject(user_id=1)
MyObject(user_id='1')

#

jb3

[python-discord/forms-backend] New review comment on pull request #42: Convert user IDs to string to avoid JS bad behaviours

We don't need this comment, the data is already a string when it comes from Discord.

#

jb3

[python-discord/forms-backend] New review comment on pull request #42: Convert user IDs to string to avoid JS bad behaviours

Providing you agree with my comment on the validator, this import is not required.

from pydantic import BaseModel

#

jb3

[python-discord/forms-backend] Pull request review submitted: #39 Adds Discord Embed on Form Submission

Excellent work, well done!

#

ks129

[forms-backend:big\-int\-fix] 1 new commit

d0e19ef Remove unnecessary comment about ID - ks129

#

ks129

[forms-backend:big\-int\-fix] 1 new commit

cee239b Remove unnecessary validator - ks129

#

ks129

[forms-backend:big\-int\-fix] 1 new commit

60388a8 Remove validator import - ks129

odd spireBOT Dec 21, 2020, 7:06 PM

#

[python-discord/forms-backend] Checks Successful on PR: #42 Convert user IDs to string to avoid JS bad behaviours

GitHub Actions run 436460183 succeeded.

PR Author

Workflow Run

Forms Backend #149

Source Branch

big-int-fix

#

[python-discord/forms-backend] Checks Successful on PR: #42 Convert user IDs to string to avoid JS bad behaviours

GitHub Actions run 436460209 succeeded.

PR Author

Workflow Run

Forms Backend #150

Source Branch

big-int-fix

#

[python-discord/forms-backend] Checks Failed on PR: #42 Convert user IDs to string to avoid JS bad behaviours

GitHub Actions run 436461315 failed.

PR Author

Workflow Run

Forms Backend #151

Source Branch

big-int-fix

#

[python-discord/forms-backend] Checks Successful on PR: #42 Convert user IDs to string to avoid JS bad behaviours

GitHub Actions run 436462004 succeeded.

PR Author

Workflow Run

Forms Backend #152

Source Branch

big-int-fix

regal archBOT Dec 21, 2020, 7:37 PM

#

jb3

[forms-backend:main] 1 new commit

899798c Update review-policy.yml - jb3

#

jb3

[forms-frontend:main] 1 new commit

874e910 Update review-policy.yml - jb3

#

jb3

[python-discord/forms-backend] Pull request review submitted: #39 Adds Discord Embed on Form Submission

Re-approving for policy bot!

#

jb3

[forms-backend:discord\-webhook] 2 new commits

899798c Update review-policy.yml - jb3
52887cc Merge branch 'main' into discord-webhook - jb3

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

jb3

[python-discord/forms-backend] Pull request closed: #39 Adds Discord Embed on Form Submission

#

jb3

[forms-backend:main] 16 new commits

b80b329 Adds Webhook Option - HassanAbouelela
b795197 Merge branch 'main' into discord-webhook - HassanAbouelela
595c19e Adds Frontend URL to Env - HassanAbouelela
20c72c4 Makes Webhooks Optional - HassanAbouelela
bbbfa15 Adds Webhook Sending Functionality - HassanAbouelela

#

jb3

[python-discord/forms-backend] branch deleted: discord\-webhook

#

jb3

[python-discord/forms-backend] Pull request review submitted: #42 Convert user IDs to string to avoid JS bad behaviours

Cheers!

odd spireBOT Dec 21, 2020, 7:39 PM

#

[python-discord/forms-backend] Checks Successful on PR: #39 Adds Discord Embed on Form Submission

GitHub Actions run 436515503 succeeded.

PR Author

Workflow Run

Forms Backend #154

Source Branch

discord-webhook

#

[python-discord/forms-backend] Workflow Successful

GitHub Actions run 436513834 succeeded.

Actor

Workflow Run

Forms Backend #153

Commit

899798c

regal archBOT Dec 21, 2020, 7:39 PM

#

jb3

[forms-backend:big\-int\-fix] 18 new commits

b80b329 Adds Webhook Option - HassanAbouelela
b795197 Merge branch 'main' into discord-webhook - HassanAbouelela
595c19e Adds Frontend URL to Env - HassanAbouelela
20c72c4 Makes Webhooks Optional - HassanAbouelela
bbbfa15 Adds Webhook Sending Functionality - HassanAbouelela

#

[forms-frontend] GitHub Actions checks success on main

odd spireBOT Dec 21, 2020, 7:41 PM

#

[python-discord/forms-backend] Checks Successful on PR: #42 Convert user IDs to string to avoid JS bad behaviours

GitHub Actions run 436517546 succeeded.

PR Author

Workflow Run

Forms Backend #156

Source Branch

big-int-fix

#

[python-discord/forms-backend] Workflow Successful

GitHub Actions run 436516571 succeeded.

Actor

Workflow Run

Forms Backend #155

Commit

443a322

regal archBOT Dec 21, 2020, 7:43 PM

#

jb3

[python-discord/forms-backend] Pull request closed: #42 Convert user IDs to string to avoid JS bad behaviours

#

jb3

[forms-backend:main] 6 new commits

faeb656 Convert user IDs to string to avoid JS bad beha... - ks129
d0e19ef Remove unnecessary comment about ID - ks129
cee239b Remove unnecessary validator - ks129
60388a8 Remove validator import - ks129
59a6052 Merge branch 'main' into big-int-fix - jb3

#

jb3

[python-discord/forms-backend] branch deleted: big\-int\-fix

#

jb3

[python-discord/forms-frontend] Pull request review submitted: #49 Update typescript\-eslint monorepo to v4\.11\.0

odd spireBOT Dec 21, 2020, 7:45 PM

#

[python-discord/forms-backend] Workflow Successful

GitHub Actions run 436523803 succeeded.

Actor

Workflow Run

Forms Backend #157

Commit

3396feb

regal archBOT Dec 21, 2020, 7:54 PM

#

jb3

[python-discord/forms-frontend] Pull request closed: #49 Update typescript\-eslint monorepo to v4\.11\.0

#

jb3

[forms-frontend:main] 2 new commits

e50af0a Update typescript-eslint monorepo to v4.11.0 - renovate-bot
58f22bb Merge pull request #49 from python-discord/reno... - jb3

#

jb3

[python-discord/forms-frontend] branch deleted: renovate/typescript\-eslint\-monorepo

#

dependabot[bot]

[forms-frontend:dependabot/npm\_and\_yarn/node\-notifier\-8\.0\.1] 1 new commit

56279e2 Bump node-notifier from 8.0.0 to 8.0.1 - dependabot[bot]

#

dependabot[bot]

[python-discord/forms-frontend] Pull request opened: #50 Bump node\-notifier from 8\.0\.0 to 8\.0\.1

Bumps node-notifier from 8.0.0 to 8.0.1.

Changelog
Sourced from node-notifier's changelog.

v8.0.1

fixes possible injection issue for notify-send

Commits

5d62799 v8.0.1
0c4a80d chore: adds changelog
f5a7bc6 fix: test cases
b9d148d patch: fixes possible injection issue for notify-send
See full diff in compare view

[![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=node-no...

#

dependabot[bot]

[python-discord/forms-frontend] New branch created: dependabot/npm\_and\_yarn/node\-notifier\-8\.0\.1

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on pull request #50

#

[forms-frontend] GitHub Actions checks success on main

#

jb3

[python-discord/forms-frontend] Pull request review submitted: #50 Bump node\-notifier from 8\.0\.0 to 8\.0\.1

#

[forms-frontend] GitHub Code Scanning checks success on pull request #50

#

[forms-frontend] GitHub Actions checks success on pull request #50

#

[forms-frontend] Netlify checks success on pull request #50

#

jb3

[python-discord/forms-frontend] Pull request closed: #50 Bump node\-notifier from 8\.0\.0 to 8\.0\.1

#

jb3

[forms-frontend:main] 2 new commits

56279e2 Bump node-notifier from 8.0.0 to 8.0.1 - dependabot[bot]
8cd6bc0 Merge pull request #50 from python-discord/depe... - jb3

#

jb3

[python-discord/forms-frontend] branch deleted: dependabot/npm\_and\_yarn/node\-notifier\-8\.0\.1

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

regal archBOT Dec 21, 2020, 11:06 PM

#

lemonsaurus

[site:lemon/bug/too\_many\_github\_api\_requests] 1 new commit

c68b090 Make last_updated field automatically update. - lemonsaurus

#

lemonsaurus

[python-discord/site] New branch created: lemon/bug/too\_many\_github\_api\_requests

#

lemonsaurus

[python-discord/site] Pull request opened: #440 Make last\_updated field automatically update\.

Previously, we were operating under a bad assumption that we would be
updating the last_updated field in the RepositoryMetadata objects
whenever we updated the objects with new data from the GitHub API.

Upon closer inspection, this is not at all what we're doing, and some of
those repository objects had not been updated in over a year!

This introduces auto_now to the field, which will ensure that it is
automatically updated whenever the object is updated.

odd spireBOT Dec 21, 2020, 11:09 PM

#

[python-discord/site] Checks Failed on PR: #440 Make last_updated field automatically update.

GitHub Actions run 436842469 failed.

PR Author

lemonsaurus

Workflow Run

Lint & Test #68

Source Branch

lemon/bug/too_many_github_api_r...

regal archBOT Dec 21, 2020, 11:20 PM

#

jb3

[site:lemon/bug/too\_many\_github\_api\_requests] 1 new commit

bb2f17a relax coverage requirement in home route - jb3

#

coveralls

[python-discord/site] New comment on pull request #440: Make last\_updated field automatically update\.

Coverage remained the same at 100.0% when pulling bb2f17a12913d7d67e2c90cabd302a9f27037dca on lemon/bug/too_many_github_api_requests into 36dc58008181944ff78b85f9ac64b7b3e089b10c on master.

odd spireBOT Dec 21, 2020, 11:21 PM

#

[python-discord/site] Checks Successful on PR: #440 Make last_updated field automatically update.

GitHub Actions run 436861923 succeeded.

PR Author

lemonsaurus

Workflow Run

Lint & Test #69

Source Branch

lemon/bug/too_many_github_api_r...

regal archBOT Dec 21, 2020, 11:21 PM

#

jb3

[python-discord/site] Pull request review submitted: #440 Make last\_updated field automatically update\.

#

jb3

[python-discord/site] Pull request closed: #440 Make last\_updated field automatically update\.

#

jb3

[site:master] 3 new commits

c68b090 Make last_updated field automatically update. - lemonsaurus
bb2f17a relax coverage requirement in home route - jb3
e00be6b Merge pull request #440 from python-discord/lem... - jb3

#

jb3

[python-discord/site] branch deleted: lemon/bug/too\_many\_github\_api\_requests

odd spireBOT Dec 21, 2020, 11:25 PM

#

[python-discord/site] Workflow Successful

GitHub Actions run 436867969 succeeded.

Actor

Workflow Run

Deploy #17

Commit

e00be6b

regal archBOT Dec 22, 2020, 1:59 AM

#

HassanAbouelela

[forms-backend:modify\-patch\-behavior] 2 new commits

1c0684f Dedents Webhook Object - HassanAbouelela
0f04292 Updates Patching Strategy - HassanAbouelela

#

HassanAbouelela

[python-discord/forms-backend] New branch created: modify\-patch\-behavior

#

HassanAbouelela

[python-discord/forms-backend] Pull request opened: #43 Modify Patch Behavior

Closes this notion ticket.

Changes the patch route behavior for dicts, so we don't need to include full objects whenever we update them. Does not affect behavior for other primitives or containers.

Additionally removes the meta object, and moves webhooks under the main form object for the reasons described in the ticket.

odd spireBOT Dec 22, 2020, 2:05 AM

#

[python-discord/forms-backend] Checks Successful on PR: #43 Modify Patch Behavior

GitHub Actions run 437075457 succeeded.

PR Author

Workflow Run

Forms Backend #158

Source Branch

modify-patch-behavior

regal archBOT Dec 22, 2020, 3:15 AM

#

jb3

[python-discord/forms-backend] Pull request review submitted: #43 Modify Patch Behavior

Nice stuff!

#

jb3

[python-discord/forms-backend] Pull request closed: #43 Modify Patch Behavior

#

jb3

[forms-backend:main] 3 new commits

1c0684f Dedents Webhook Object - HassanAbouelela
0f04292 Updates Patching Strategy - HassanAbouelela
a7dd78e Merge pull request #43 from python-discord/modi... - jb3

#

jb3

[python-discord/forms-backend] branch deleted: modify\-patch\-behavior

odd spireBOT Dec 22, 2020, 3:22 AM

#

[python-discord/forms-backend] Workflow Successful

GitHub Actions run 437172387 succeeded.

Actor

Workflow Run

Forms Backend #159

Commit

a7dd78e

regal archBOT Dec 22, 2020, 4:35 AM

#

renovate[bot]

[forms-frontend:renovate/axios\-0\.x] 1 new commit

bd45c6c Update dependency axios to v0.21.1 - renovate-bot

#

renovate[bot]

[python-discord/forms-frontend] New branch created: renovate/axios\-0\.x

#

renovate[bot]

[python-discord/forms-frontend] Pull request opened: #51 Update dependency axios to v0\.21\.1

This PR contains the following updates:

Package	Change	Age	Adoption	Passing	Confidence
axios	`0.21.0` -> `0.21.1`		[![adoption](https...

#

[forms-frontend] GitHub Actions checks success on pull request #51

#

[forms-frontend] Netlify checks success on pull request #51

#

[forms-frontend] GitHub Code Scanning checks success on pull request #51

#

[forms-frontend] GitHub Actions checks success on pull request #51

regal archBOT Dec 22, 2020, 6:19 AM

#

ks129

[python-discord/bot] New comment on pull request #1297: Implements Voice Channel Muting

@HassanAbouelela Can you please solve conflicts?

odd spireBOT Dec 22, 2020, 6:49 AM

#

[python-discord/bot] Checks Failed on PR: #1297 Implements Voice Channel Muting

GitHub Actions run 437457075 failed.

PR Author

Workflow Run

Lint & Test #196

Source Branch

HassanAbouelela:voicechann...

#

[python-discord/bot] Checks Successful on PR: #1297 Implements Voice Channel Muting

GitHub Actions run 437469944 succeeded.

PR Author

Workflow Run

Lint & Test #197

Source Branch

HassanAbouelela:voicechann...

#

[python-discord/bot] Checks Successful on PR: #1297 Implements Voice Channel Muting

GitHub Actions run 437472509 succeeded.

PR Author

Workflow Run

Lint & Test #198

Source Branch

HassanAbouelela:voicechann...

regal archBOT Dec 22, 2020, 8:00 AM

#

renovate[bot]

[forms-frontend:renovate/emotion\-monorepo] 1 new commit

7e16ec3 Update dependency @emotion/react to v11.1.4 - renovate-bot

#

renovate[bot]

[python-discord/forms-frontend] New branch created: renovate/emotion\-monorepo

#

renovate[bot]

[python-discord/forms-frontend] Pull request opened: #52 Update dependency @emotion/react to v11\.1\.4

This PR contains the following updates:

Package	Change	Age	Adoption	Passing	Confidence
@emotion/react	`11.1.3` -> `11.1.4`	[](https://docs.renovatebot.com/m...

#

[forms-frontend] GitHub Actions checks success on pull request #52

#

[forms-frontend] Netlify checks success on pull request #52

#

[forms-frontend] GitHub Code Scanning checks success on pull request #52

#

[forms-frontend] GitHub Actions checks success on pull request #52

#

ks129

[python-discord/forms-frontend] Pull request review submitted: #52 Update dependency @emotion/react to v11\.1\.4

#

ks129

[python-discord/forms-frontend] Pull request closed: #52 Update dependency @emotion/react to v11\.1\.4

#

ks129

[forms-frontend:main] 2 new commits

7e16ec3 Update dependency @emotion/react to v11.1.4 - renovate-bot
59533e4 Merge pull request #52 from python-discord/reno... - ks129

#

ks129

[python-discord/forms-frontend] branch deleted: renovate/emotion\-monorepo

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

ks129

[python-discord/forms-frontend] Pull request review submitted: #51 Update dependency axios to v0\.21\.1

#

ks129

[python-discord/forms-frontend] Pull request closed: #51 Update dependency axios to v0\.21\.1

#

[forms-frontend] GitHub Actions checks success on main

#

ks129

[forms-frontend:main] 2 new commits

bd45c6c Update dependency axios to v0.21.1 - renovate-bot
e8bffd3 Merge pull request #51 from python-discord/reno... - ks129

#

ks129

[python-discord/forms-frontend] branch deleted: renovate/axios\-0\.x

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

regal archBOT Dec 22, 2020, 12:23 PM

#

ks129

[forms-backend:ks123/admin\-adding] 1 new commit

feb7981 Create route for adding new admins - ks129

#

ks129

[python-discord/forms-backend] New branch created: ks123/admin\-adding

#

ks129

[python-discord/forms-backend] Pull request opened: #44 Create route for adding new admins

Admin-only route for adding new admins.

odd spireBOT Dec 22, 2020, 12:27 PM

#

[python-discord/forms-backend] Checks Successful on PR: #44 Create route for adding new admins

GitHub Actions run 438014345 succeeded.

PR Author

Workflow Run

Forms Backend #160

Source Branch

ks123/admin-adding

regal archBOT Dec 22, 2020, 2:22 PM

#

renovate[bot]

[forms-backend:renovate/pyjwt\-2\.x] 1 new commit

4976b75 Update dependency pyjwt to v2 - renovate-bot

#

renovate[bot]

[python-discord/forms-backend] New branch created: renovate/pyjwt\-2\.x

#

renovate[bot]

[python-discord/forms-backend] Pull request opened: #45 Update dependency pyjwt to v2

This PR contains the following updates:

Package	Type	Update	Change
pyjwt	dependencies	major	`^1.7.1` -> `^2.0.0`

Release Notes

jpadilla/pyjwt

`v2.0.0`

Compare Source

#...

odd spireBOT Dec 22, 2020, 2:24 PM

#

[python-discord/forms-backend] Checks Successful on PR: #45 Update dependency pyjwt to v2

GitHub Actions run 438213182 succeeded.

PR Author

renovate[bot]

Workflow Run

Forms Backend #161

Source Branch

renovate/pyjwt-2.x

oak estuaryBOT Dec 22, 2020, 5:31 PM

#

Python Bot

Connected!

regal archBOT Dec 22, 2020, 8:22 PM

#

[forms-frontend] GitHub Actions checks success on main

regal archBOT Dec 22, 2020, 9:05 PM

#

SebastiaanZ

[python-discord/sir-lancebot] Issue opened: #549 Investigate an upgrade to Python 3\.9\.1

Description

Python 3.9.1 has been released today (yay!), which means that it's mature enough for us to start looking into upgrading the Python version of sir-lancebot. We'll obviously have to make sure all our dependencies support 3.9 and everything works well, but it would be nice to use the new language features.

Reasoning

It's better to keep up with Python version, adjusting our code base as required, than making a major leap in major versions with a potential more sizeable mi...

regal archBOT Dec 22, 2020, 9:47 PM

#

renovate[bot]

[forms-frontend:renovate/font\-awesome] 1 new commit

b1afa49 Update dependency @fortawesome/react-fontawesom... - renovate-bot

#

renovate[bot]

[python-discord/forms-frontend] New branch created: renovate/font\-awesome

#

renovate[bot]

[python-discord/forms-frontend] Pull request opened: #53 Update dependency @fortawesome/react\-fontawesome to v0\.1\.14

This PR contains the following updates:

Package	Change	Age	Adoption	Passing	Confidence
@fortawesome/react-fontawesome	`0.1.13` -> `0.1.14`	[![age](https://badges.renovateapi.com/packages/npm/@fortawesome%2Freact-f...

#

[forms-frontend] GitHub Actions checks success on pull request #53

#

[forms-frontend] Netlify checks success on pull request #53

#

[forms-frontend] GitHub Code Scanning checks success on pull request #53

#

[forms-frontend] GitHub Actions checks success on pull request #53

regal archBOT Dec 22, 2020, 10:45 PM

#

jb3

[python-discord/forms-frontend] Pull request review submitted: #53 Update dependency @fortawesome/react\-fontawesome to v0\.1\.14

regal archBOT Dec 23, 2020, 1:33 AM

#

swfarnsworth

[bot:swfarnsworth/developer\_command] 43 new commits

a8a3e82 Handling pending flag changes on users - jb3
0be0d86 Correctly check if the user is pending - jb3
80dd230 Merge pull request #1329 from python-discord/na... - jb3
5837921 Fix minor verification bugs - jb3
fb4d82c Install git in Docker container - jb3

odd spireBOT Dec 23, 2020, 1:35 AM

#

[python-discord/bot] Checks Successful on PR: #1328 Manual verification with `!verify` sends the user the confirmation DM.

GitHub Actions run 439312542 succeeded.

PR Author

Workflow Run

Lint & Test #199

Source Branch

swfarnsworth/developer_command

regal archBOT Dec 23, 2020, 2:44 AM

#

HassanAbouelela

[forms-frontend:preview\-form\-rendering] 1 new commit

0055443 23 Dec Nightly Commit - HassanAbouelela

#

HassanAbouelela

[python-discord/forms-frontend] New branch created: preview\-form\-rendering

#

HassanAbouelela

[python-discord/forms-frontend] Pull request opened: #54 23 Dec Nightly Commit

Not ready for review

Signed-off-by: Hassan Abouelela

#

[forms-frontend] Netlify checks success on pull request #54

#

[forms-frontend] GitHub Actions checks success on pull request #54

#

[forms-frontend] GitHub Code Scanning checks success on pull request #54

#

[forms-frontend] GitHub Actions checks success on pull request #54

#

jb3

[python-discord/forms-frontend] Pull request closed: #53 Update dependency @fortawesome/react\-fontawesome to v0\.1\.14

#

jb3

[forms-frontend:main] 2 new commits

b1afa49 Update dependency @fortawesome/react-fontawesom... - renovate-bot
e989244 Merge pull request #53 from python-discord/reno... - jb3

#

jb3

[python-discord/forms-frontend] branch deleted: renovate/font\-awesome

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] GitHub Actions checks success on main

#

HassanAbouelela

[forms-frontend:preview\-form\-rendering] 1 new commit

886ff0a Pull Forms From Backend - HassanAbouelela

#

[forms-frontend] GitHub Actions checks success on main

#

[forms-frontend] Netlify checks success on pull request #54

#

[forms-frontend] GitHub Actions checks success on pull request #54

#

[forms-frontend] GitHub Code Scanning checks success on pull request #54

#

[forms-frontend] GitHub Actions checks success on pull request #54

regal archBOT Dec 23, 2020, 3:36 AM

#

HassanAbouelela

[forms-frontend:preview\-form\-rendering] 1 new commit

5f93b2f Stop Forms Fetch on Rerender - HassanAbouelela

#

[forms-frontend] Netlify checks success on pull request #54

#

[forms-frontend] GitHub Actions checks success on pull request #54

#

[forms-frontend] GitHub Code Scanning checks success on pull request #54

#

[forms-frontend] GitHub Actions checks success on pull request #54

regal archBOT Dec 23, 2020, 5:28 AM

#

HassanAbouelela

[forms-frontend:preview\-form\-rendering] 1 new commit

bf091fa Lay Foundation For Rendering - HassanAbouelela

#

[forms-frontend] Netlify checks success on pull request #54

#

[forms-frontend] GitHub Actions checks success on pull request #54

#

[forms-frontend] GitHub Code Scanning checks success on pull request #54

#

[forms-frontend] GitHub Actions checks success on pull request #54

#

HassanAbouelela

[forms-frontend:preview\-form\-rendering] 1 new commit

c00c93e Add Placeholders For All Types - HassanAbouelela

#

[forms-frontend] Netlify checks success on pull request #54

#

[forms-frontend] GitHub Actions checks success on pull request #54

#

[forms-frontend] GitHub Code Scanning checks success on pull request #54

#

[forms-frontend] GitHub Actions checks success on pull request #54

regal archBOT Dec 23, 2020, 8:48 AM

#

ks129

[forms-backend:ks123/responses\-bulk\-delete] 2 new commits

9763dab Add bulk DELETE method to responses endpoint - ks129
f99dff1 Check does form exists in responses bulk DELETE - ks129

#

ks129

[python-discord/forms-backend] New branch created: ks123/responses\-bulk\-delete

#

ks129

[python-discord/forms-backend] Pull request opened: #46 Responses bulk deletion

Added DELETE method to responses endpoint that mass deletes responses by IDs. Admin-only method.

odd spireBOT Dec 23, 2020, 8:57 AM

#

[python-discord/forms-backend] Checks Successful on PR: #46 Responses bulk deletion

GitHub Actions run 439901696 succeeded.

PR Author

Workflow Run

Forms Backend #162

Source Branch

ks123/responses-bulk-delete

regal archBOT Dec 23, 2020, 1:57 PM

#

jb3

[python-discord/forms-backend] Pull request review submitted: #46 Responses bulk deletion

We should define a Pydantic model of the request body we are expecting in the route file and pass it into the API validator.

A few other routes implement this so you should look there for how to do it.

#

jb3

[python-discord/forms-backend] Pull request review submitted: #46 Responses bulk deletion

#

jb3

[python-discord/forms-backend] New review comment on pull request #46: Responses bulk deletion

This would be handled by the pydantic validator with @api.validate

#

jb3

[python-discord/forms-backend] Pull request review submitted: #44 Create route for adding new admins

Looks good!

#

jb3

[python-discord/forms-backend] New comment on pull request #44: Create route for adding new admins

whoops, wrong button

#

jb3

[python-discord/forms-backend] Pull request review submitted: #44 Create route for adding new admins

Minor feedback! Thanks!

#

jb3

[python-discord/forms-backend] New review comment on pull request #44: Create route for adding new admins

This is used in API docs, so let's phrase it a bit differently.

        """Grant a user administrator privileges."""

#

jb3

[python-discord/forms-backend] New review comment on pull request #44: Create route for adding new admins

Let's perform body validation here instead of inside the request logic.

#

jb3

[python-discord/forms-backend] New review comment on pull request #44: Create route for adding new admins

Won't this raise if the user is already an administrator?

#

jb3

[python-discord/forms-backend] New comment on pull request #45: Update dependency pyjwt to v2

Testing this now.

#

jb3

[python-discord/forms-backend] New comment on pull request #45: Update dependency pyjwt to v2

JWT validation works correctly.

#

jb3

[python-discord/forms-backend] Pull request review submitted: #45 Update dependency pyjwt to v2

#

jb3

[python-discord/forms-backend] Pull request closed: #45 Update dependency pyjwt to v2

#

jb3

[forms-backend:main] 2 new commits

4976b75 Update dependency pyjwt to v2 - renovate-bot
457e4d5 Merge pull request #45 from python-discord/reno... - jb3

#

jb3

[python-discord/forms-backend] branch deleted: renovate/pyjwt\-2\.x

#

jb3

[python-discord/site] branch deleted: joseph/add\-form\-system

#

jb3

[python-discord/site] branch deleted: revert\-348\-off\-topic\-non\-random

odd spireBOT Dec 23, 2020, 2:09 PM

#

[python-discord/forms-backend] Workflow Successful

GitHub Actions run 440392829 succeeded.

Actor

Workflow Run

Forms Backend #163

Commit

457e4d5

regal archBOT Dec 23, 2020, 2:29 PM

#

HassanAbouelela

[forms-frontend:preview\-form\-rendering] 2 new commits

0264db8 Splits Form CSS Into Its Own File - HassanAbouelela
b16f278 Style Experiment - HassanAbouelela

#

[forms-frontend] GitHub Actions checks failure on pull request #54

#

[forms-frontend] Netlify checks success on pull request #54

#

[forms-frontend] GitHub Code Scanning checks success on pull request #54

#

[forms-frontend] GitHub Actions checks success on pull request #54

#

HassanAbouelela

[forms-frontend:preview\-form\-rendering] 1 new commit

95f5d22 Fix Failing Tests - HassanAbouelela

#

[forms-frontend] Netlify checks success on pull request #54

#

[forms-frontend] GitHub Actions checks failure on pull request #54

#

[forms-frontend] GitHub Code Scanning checks success on pull request #54

#

[forms-frontend] GitHub Actions checks success on pull request #54

regal archBOT Dec 23, 2020, 2:52 PM

#

digasred

[python-discord/sir-lancebot] New comment on issue #246: \[Feature Request\] Battleship

How about a music quiz command like mee6 has ?

regal archBOT Dec 23, 2020, 3:23 PM

#

Xithrius

[python-discord/bot] Pull request review submitted: #1337 Move PEP command and utils to own cog, fix bug related not checking status and add GitHub token

#

Xithrius

[python-discord/bot] New review comment on pull request #1337: Move PEP command and utils to own cog, fix bug related not checking status and add GitHub token

Same here

            description=f"[Link]({BASE_PEP_URL}{pep_nr:04})",

#

Xithrius

[python-discord/bot] New review comment on pull request #1337: Move PEP command and utils to own cog, fix bug related not checking status and add GitHub token

If the constants are moved outside the class, this should be renamed.

           PEPS_LISTING_API_URL,

#

Xithrius

[python-discord/bot] New review comment on pull request #1337: Move PEP command and utils to own cog, fix bug related not checking status and add GitHub token

I feel like these constants should be outside the class, next to ICON_URL.

#

ks129

[python-discord/sir-lancebot] New comment on issue #246: \[Feature Request\] Battleship

@digasred Hello, your comment is not related with this issue, what is also already closed. Please open new issue instead.

#

HassanAbouelela

[forms-frontend:preview\-form\-rendering] 1 new commit

b2911b9 Fix Seperator Bug - HassanAbouelela

#

ks129

[bot:ks123/pep] 1 new commit

dd546b8 Move constants out from class - ks129

#

[forms-frontend] GitHub Actions checks failure on pull request #54

#

[forms-frontend] Netlify checks success on pull request #54

#

[forms-frontend] GitHub Code Scanning checks success on pull request #54

#

[forms-frontend] GitHub Actions checks success on pull request #54

odd spireBOT Dec 23, 2020, 3:33 PM

#

[python-discord/bot] Checks Successful on PR: #1337 Move PEP command and utils to own cog, fix bug related not checking status and add GitHub token

GitHub Actions run 440539237 succeeded.

PR Author

Workflow Run

Lint & Test #200

Source Branch

ks123/pep

regal archBOT Dec 23, 2020, 3:39 PM

#

Xithrius

[python-discord/bot] Pull request review submitted: #1337 Move PEP command and utils to own cog, fix bug related not checking status and add GitHub token

LGTM!

#

Xithrius

[python-discord/bot] Pull request review submitted: #1328 Manual verification with \`\!verify\` sends the user the confirmation DM\.

Besides this one line in the ALTERNATE_VERIFIED_MESSAGE docstring that may need to be changed, everything looks good to me.

#

Xithrius

[python-discord/bot] New review comment on pull request #1328: Manual verification with \`\!verify\` sends the user the confirmation DM\.

It may just be me, but I feel this would be "easier" to read for more users by saying

You are now verified!

regal archBOT Dec 23, 2020, 4:09 PM

#

Xithrius

[bot:feat/1169/codespaces] 800 new commits

12cbfe7 (Aliases, discord.py 1.3.x Migration): Replaced... - ks129
61a93c1 (Snekbox, discord.py 1.3.x Migration): Replaced... - ks129
88b4c72 (Patches, discord.py 1.3.x Migration): Removed ... - ks129
1a14f4f (Off-Topic Names, discord.py 1.3.x Migration): ... - ks129
a4a4b98 (Reddit, discord.py 1.3.x Migration): Replaced ... - ks129

odd spireBOT Dec 23, 2020, 4:11 PM

#

[python-discord/bot] Checks Successful on PR: #1203 Add GitHub codespaces support

GitHub Actions run 440603539 succeeded.

PR Author

Akarys42

Workflow Run

Lint & Test #201

Source Branch

feat/1169/codespaces

regal archBOT Dec 23, 2020, 4:12 PM

#

coveralls

[python-discord/bot] New comment on pull request #1203: Add GitHub codespaces support

Coverage remained the same at 56.541% when pulling a24eefa950d620b0045af80fd92c26f54dc932eb on feat/1169/codespaces into 7aec1215e5d059342e5aa158276094d1932da8c1 on master.

regal archBOT Dec 23, 2020, 4:28 PM

#

Akarys42

[python-discord/bot] New comment on pull request #1203: Add GitHub codespaces support

I won't have time to finish this, if someone feels up to the task, feel free to continue the work here!

regal archBOT Dec 23, 2020, 5:30 PM

#

Gravitar63

[python-discord/metricity] New comment on commit `4122571`

Erstellt am 23.12.2020
Neues Videoprojekt

#

Gravitar63

[python-discord/metricity] New comment on commit `4122571`

tassentalk

#

swfarnsworth

[python-discord/bot] Pull request review submitted: #1328 Manual verification with \`\!verify\` sends the user the confirmation DM\.

#

swfarnsworth

[python-discord/bot] New review comment on pull request #1328: Manual verification with \`\!verify\` sends the user the confirmation DM\.

I made that change under the pretense that we'd still have bot verification. Does anyone actually get the alternate verified message anymore?

regal archBOT Dec 23, 2020, 6:05 PM

#

Den4200

[python-discord/bot] Pull request review submitted: #1337 Move PEP command and utils to own cog, fix bug related not checking status and add GitHub token

This cog would better fit into the info directory.

#

ks129

[bot:ks123/pep] 1 new commit

361a212 Move PEP cog to info extensions category - ks129

#

ks129

[python-discord/bot] New comment on pull request #1337: Move PEP command and utils to own cog, fix bug related not checking status and add GitHub token

Moved PEP cog to info extensions group.

odd spireBOT Dec 23, 2020, 6:14 PM

#

[python-discord/bot] Checks Successful on PR: #1337 Move PEP command and utils to own cog, fix bug related not checking status and add GitHub token

GitHub Actions run 440803463 succeeded.

PR Author

Workflow Run

Lint & Test #202

Source Branch

ks123/pep

regal archBOT Dec 23, 2020, 6:18 PM

#

Den4200

[python-discord/bot] Pull request review submitted: #1337 Move PEP command and utils to own cog, fix bug related not checking status and add GitHub token

Thanks!

#

Den4200

[bot:ks123/pep] 13 new commits

8f425fc Pump Sentry SDK version from 0.14 to 0.19 - ks129
4ebbb5d Create workflow for creating Sentry release - ks129
756fa37 Consume Git SHA build arg and add to it to envi... - ks129
8da3423 Inject Git SHA in container build workflow - ks129
33ad294 Add constant for Git SHA - ks129

#

Den4200

[python-discord/bot] Issue closed: #1335 refresh\_peps\_urls should check the response status code

#

Den4200

[python-discord/bot] Issue closed: #1336 validate\_pep\_number should acount for last\_refreshed\_peps being None

#

Den4200

[python-discord/bot] Pull request closed: #1337 Move PEP command and utils to own cog, fix bug related not checking status and add GitHub token

#

Den4200

[bot:master] 12 new commits

477af4e Add GitHub API key to config as environment var... - ks129
de3dd22 Move PEP related functions and command to own cog - ks129
d9fed58 Set last PEPs listing at beginning of function - ks129
b7ab159 Make last PEPs listing refresh non-optional - ks129
4527f9a Log warning and return early when can't get PEP... - ks129

#

Den4200

[python-discord/bot] branch deleted: ks123/pep

odd spireBOT Dec 23, 2020, 6:21 PM

#

[python-discord/bot] Checks Successful on PR: #1337 Move PEP command and utils to own cog, fix bug related not checking status and add GitHub token

GitHub Actions run 440813075 succeeded.

PR Author

Workflow Run

Lint & Test #203

Source Branch

ks123/pep

oak estuaryBOT Dec 23, 2020, 6:25 PM

#

Python Bot

Connected!

odd spireBOT Dec 23, 2020, 6:25 PM

#

[python-discord/bot] Workflow Successful

GitHub Actions run 440822373 succeeded.

Actor

Den4200

Workflow Run

Deploy #74

Commit

1c7971d

regal archBOT Dec 23, 2020, 8:30 PM

#

renovate[bot]

[forms-frontend:renovate/node\-14\.x] 1 new commit

32908eb Update dependency @types/node to v14.14.16 - renovate-bot

#

renovate[bot]

[python-discord/forms-frontend] New branch created: renovate/node\-14\.x

#

renovate[bot]

[python-discord/forms-frontend] Pull request opened: #55 Update dependency @types/node to v14\.14\.16

This PR contains the following updates:

Package	Change	Age	Adoption	Passing	Confidence
@types/node	`14.14.14` -> `14.14.16`	[](https://docs.ren...

#

[forms-frontend] GitHub Actions checks success on pull request #55

#

[forms-frontend] Netlify checks success on pull request #55

#

[forms-frontend] GitHub Code Scanning checks success on pull request #55

#

[forms-frontend] GitHub Actions checks success on pull request #55

regal archBOT Dec 23, 2020, 11:55 PM

#

swfarnsworth

[bot:swfarnsworth/developer\_command] 14 new commits

477af4e Add GitHub API key to config as environment var... - ks129
de3dd22 Move PEP related functions and command to own cog - ks129
d9fed58 Set last PEPs listing at beginning of function - ks129
b7ab159 Make last PEPs listing refresh non-optional - ks129
4527f9a Log warning and return early when can't get PEP... - ks129

odd spireBOT Dec 23, 2020, 11:58 PM

#

[python-discord/bot] Checks Successful on PR: #1328 Manual verification with `!verify` sends the user the confirmation DM.

GitHub Actions run 441305572 succeeded.

PR Author

Workflow Run

Lint & Test #205

Source Branch

swfarnsworth/developer_command

regal archBOT Dec 24, 2020, 1:57 AM

#

Xithrius

[python-discord/bot] Pull request review submitted: #1328 Manual verification with \`\!verify\` sends the user the confirmation DM\.

LGTM!

regal archBOT Dec 24, 2020, 4:44 AM

#

ks129

[python-discord/bot] Pull request review submitted: #1328 Manual verification with \`\!verify\` sends the user the confirmation DM\.

#

ks129

[python-discord/bot] New review comment on pull request #1328: Manual verification with \`\!verify\` sends the user the confirmation DM\.

When user have pending True, I think this means that user isn't already verified?

#

swfarnsworth

[python-discord/bot] Pull request review submitted: #1328 Manual verification with \`\!verify\` sends the user the confirmation DM\.

#

swfarnsworth

[python-discord/bot] New review comment on pull request #1328: Manual verification with \`\!verify\` sends the user the confirmation DM\.

I think you're right--this should probably be if not user.pending.

regal archBOT Dec 24, 2020, 5:52 AM

#

mudkipdev

[python-discord/bot] Pull request review submitted: #1325 Add dynamic message in \#how\-to\-get\-help

#

mudkipdev

[python-discord/bot] New review comment on pull request #1325: Add dynamic message in \#how\-to\-get\-help

Right, forgot about that.

#

mudkipdev

[python-discord/bot] Pull request review submitted: #1325 Add dynamic message in \#how\-to\-get\-help

#

mudkipdev

[python-discord/bot] New review comment on pull request #1325: Add dynamic message in \#how\-to\-get\-help

That will raise Forbidden on edit.

#

mudkipdev

[python-discord/bot] New comment on pull request #1325: Add dynamic message in \#how\-to\-get\-help

@MarkKoz I'm so sorry, and yeah I tried to setup precommit, doesn't seem to work. I'll try again and fix all of the things you mentioned.

regal archBOT Dec 24, 2020, 6:12 AM

#

ks129

[forms-backend:ks123/admin\-adding] 2 new commits

d30ac68 Use Pydantic validation instead manual validation - ks129
e0efac8 Check is user already admin before adding - ks129

odd spireBOT Dec 24, 2020, 6:13 AM

#

[python-discord/forms-backend] Checks Successful on PR: #44 Create route for adding new admins

GitHub Actions run 441788570 succeeded.

PR Author

Workflow Run

Forms Backend #164

Source Branch

ks123/admin-adding

regal archBOT Dec 24, 2020, 6:14 AM

#

ks129

[forms-backend:ks123/admin\-adding] 1 new commit

d04f23c Update admin add route docstring - ks129

odd spireBOT Dec 24, 2020, 6:15 AM

#

[python-discord/forms-backend] Checks Successful on PR: #44 Create route for adding new admins

GitHub Actions run 441789966 succeeded.

PR Author

Workflow Run

Forms Backend #165

Source Branch

ks123/admin-adding

regal archBOT Dec 24, 2020, 7:29 AM

#

ks129

[forms-backend:ks123/responses\-bulk\-delete] 1 new commit

d41140f Let Pydantic validate bulk responses delete data - ks129

odd spireBOT Dec 24, 2020, 7:30 AM

#

[python-discord/forms-backend] Checks Successful on PR: #46 Responses bulk deletion

GitHub Actions run 441895187 succeeded.

PR Author

Workflow Run

Forms Backend #166

Source Branch

ks123/responses-bulk-delete

regal archBOT Dec 24, 2020, 3:35 PM

#

swfarnsworth

[bot:swfarnsworth/developer\_command] 1 new commit

68cbca0 if user.pending -> if not user.pending - swfarnsworth

odd spireBOT Dec 24, 2020, 3:38 PM

#

[python-discord/bot] Checks Successful on PR: #1328 Manual verification with `!verify` sends the user the confirmation DM.

GitHub Actions run 442592003 succeeded.

PR Author

Workflow Run

Lint & Test #206

Source Branch

swfarnsworth/developer_command

regal archBOT Dec 24, 2020, 3:57 PM

#

ks129

[python-discord/bot] Pull request review submitted: #1328 Manual verification with \`\!verify\` sends the user the confirmation DM\.

#

ks129

[python-discord/bot] Pull request closed: #1328 Manual verification with \`\!verify\` sends the user the confirmation DM\.

#

ks129

[bot:master] 9 new commits

fc1f7ac User gets the bot DM when verified via !verify. - swfarnsworth
2b09d73 Responses from the bot mention the user. - swfarnsworth
2b73df6 Merge branch 'master' of https://github.com/pyt... - swfarnsworth
ce46567 Rewrite !verify to account for new native-gat... - swfarnsworth
e565515 Merge branch 'master' of https://github.com/pyt... - swfarnsworth

#

ks129

[python-discord/bot] branch deleted: swfarnsworth/developer\_command

oak estuaryBOT Dec 24, 2020, 4:02 PM

#

Python Bot

Connected!

odd spireBOT Dec 24, 2020, 4:03 PM

#

[python-discord/bot] Workflow Successful

GitHub Actions run 442628870 succeeded.

Actor