aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | * | | | | | | | | | | | Sync: fix overwriting message with None after editing itGravatar MarkKoz2020-02-12-1/+1
| | | | | | | | | | | | | |
| | * | | | | | | | | | | | Sync: fix passing context instead of message to _confirm()Gravatar MarkKoz2020-02-12-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Mention possibility of timing out as a reason for aborting a sync
| | * | | | | | | | | | | | Sync: fix missing await for fetch_channelGravatar MarkKoz2020-02-12-1/+1
| | | | | | | | | | | | | |
| | * | | | | | | | | | | | Sync: allow for None values in DiffsGravatar MarkKoz2020-02-12-4/+4
| | | | | | | | | | | | | |
| | * | | | | | | | | | | | Sync: make Role, User, and Diff privateGravatar MarkKoz2020-02-12-15/+15
| | | | | | | | | | | | | |
| | * | | | | | | | | | | | Sync: remove generic type from DiffGravatar MarkKoz2020-02-12-19/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It doesn't play along well with NamedTuple due to metaclass conflicts. The workaround involved created a NamedTuple-only base class, which does work but at the cost of confusing some static type checkers. Since Diff is now an internal data structure, it no longer really needs to have precise type annotations. Therefore, a normal namedtuple is adequate.
| | * | | | | | | | | | | | Sync: move sync logic into Syncer base classGravatar MarkKoz2020-02-12-93/+54
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The interface was becoming cumbersome to work with so it was all moved to a single location. Now just calling Syncer.sync() will take care of everything. * Remove Optional type annotation from Diff attributes * _confirm() can edit the original message and use it as the prompt * Calculate the total diff and compare it against the max before sending a confirmation prompt * Remove abort message from sync(); _confirm() will handle that
| | * | | | | | | | | | | | Sync: prompt to confirm when diff is greater than 10Gravatar MarkKoz2020-02-12-37/+158
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The confirmation prompt will be sent to the dev-core channel or the specified context. Confirmation is done via reactions and waits 5 minutes before timing out. * Add name property to Syncers * Make _get_diff private; only sync() needs to be called now * Change spelling of synchronize to synchronise * Update docstrings
| | * | | | | | | | | | | | Sync: create classes for syncersGravatar MarkKoz2020-02-12-242/+158
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replaces the functions with a class for each syncer. The classes inherit from a Syncer base class. A NamedTuple was also created to replace the tuple of the object differences that was previously being returned. * Use namedtuple._asdict to simplify converting namedtuples to JSON
| | * | | | | | | | | | | | Sync: support sending messages to a context in sync()Gravatar MarkKoz2020-02-12-17/+11
| | | | | | | | | | | | | |
| | * | | | | | | | | | | | Sync: support multiple None totals returns from a syncerGravatar MarkKoz2020-02-12-10/+7
| | | | | | | | | | | | | |
| | * | | | | | | | | | | | Sync: create function for running a single syncerGravatar MarkKoz2020-02-12-22/+24
| | | | | | | | | | | | | |
| | * | | | | | | | | | | | Sync: refactor cogGravatar MarkKoz2020-02-12-25/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Use ID from constants directly instead of SYNC_SERVER_ID * Use f-strings instead of %s for logging * Fit into margin of 100 * Invert condition to reduce nesting * Use Any instead of incorrect function annotation for JSON values
| | * | | | | | | | | | | | API: create request function which has a param for the HTTP methodGravatar MarkKoz2020-02-12-18/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reduces code redundancy.
| | * | | | | | | | | | | | API: define functions with keyword-only argumentsGravatar MarkKoz2020-02-12-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This seems to have been the intent of the original implementation.
| | * | | | | | | | | | | | Constants: add dev-core channel and check mark emojiGravatar MarkKoz2020-02-12-0/+4
| | | | | | | | | | | | | |
| | * | | | | | | | | | | | Use wait_until_guild_available instead of wait_until_readyGravatar MarkKoz2020-02-12-13/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It has a much better guarantee that the cache will be available.
| | * | | | | | | | | | | | Bot: make the connector attribute privateGravatar MarkKoz2020-02-12-4/+4
| | | | | | | | | | | | | |
| | * | | | | | | | | | | | Bot: add wait_until_guild_availableGravatar MarkKoz2020-02-12-0/+39
| |/ / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This coroutine waits until the configured guild is available and ensures the cache is present. The on_ready event is inadequate because it only waits 2 seconds for a GUILD_CREATE gateway event before giving up and thus not populating the cache for unavailable guilds.
| | * / / / / / / / / / / Reminders: don't cancel task if reminder is invalid when reschedulingGravatar MarkKoz2020-02-23-6/+11
| |/ / / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If a reminder is invalid, it won't get rescheduled. Therefore, there wouldn't exist a task to cancel and it'd raise a warning. Fixes BOT-1C
| | | | | | | | * | | | Caching all the tags when the bot has loaded(caching only once) insted of ↵Gravatar RohanJnr2020-02-29-21/+18
| | | | | | | | |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | caching it after the tags command is used.
| | | | | | | | * | | Re-corrected the lines which I had changed by mistakeGravatar RohanJnr2020-02-29-1/+2
| | | | | | | | | | |
| | | | | | | | * | | added white spaces on statements before bullet points for proper rendering ↵Gravatar RohanJnr2020-02-29-75/+75
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | of points on github
| | | | | | | | * | | Added all the tag files in resources and modified cogs/tags.py file to ↵Gravatar RohanJnr2020-02-29-84/+919
| |_|_|_|_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | access the static tag files rather than sending an API get request. Removed all methods calling the API so the tags cannot be edited, added nor deleted.
* | | | | | | | | | Add Sentdex server to whitelistGravatar Leon Sandøy2020-02-23-0/+1
| | | | | | | | | |
* | | | | | | | | | Merge pull request #773 from python-discord/reminder_missing_usersGravatar Joseph2020-02-23-17/+37
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | Check reminder user and channel before send and schedule.
| * \ \ \ \ \ \ \ \ \ Merge branch 'master' into reminder_missing_usersGravatar Joseph2020-02-23-120/+101
| |\ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / |/| | | | | | | | | |
* | | | | | | | | | | Merge pull request #776 from python-discord/checkout-msg-logGravatar S. Co12020-02-23-14/+13
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Change verification post log level to info, tidy code.
| * \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into checkout-msg-logGravatar S. Co12020-02-23-2/+6
| |\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #756 from python-discord/bug/backend/s329/docker-compose-ttyGravatar scragly2020-02-24-0/+2
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Fix missing Django logs when using Docker Compose
| * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into bug/backend/s329/docker-compose-ttyGravatar scragly2020-02-24-571/+888
| |\ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | Fix missing Django logs when using Docker ComposeGravatar MarkKoz2020-02-16-0/+2
| | |_|_|_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed by allocating a pseudo-tty to the web and bot services in Docker Compose.
* | | | | | | | | | | | | Merge pull request #774 from python-discord/cooldown-log-levelGravatar S. Co12020-02-23-2/+4
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|/ / / / / / / / / / / |/| | | | | | | | | | | | Reduce log level of tag cooldown notice.
| * | | | | | | | | | | | Reduce log level of tag cooldown notice.Gravatar scragly2020-02-24-2/+4
|/ / / / / / / / / / / /
| | * / / / / / / / / / Change verification post log level to info, tidy code.Gravatar scragly2020-02-24-14/+13
| |/ / / / / / / / / / |/| | | | | | | | | |
* | | | | | | | | | | Merge pull request #741 from Denayder/information-refactorGravatar S. Co12020-02-23-73/+50
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Refactor information.py with lots of changes that makes the overall code cleaner and faster
| * \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into information-refactorGravatar S. Co12020-02-23-562/+978
| |\ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / |/| | | | | | | | | | |
* | | | | | | | | | | | Merge pull request #771 from python-discord/paginator-react-404Gravatar S. Co12020-02-23-18/+22
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | Fix paginator erroring on clearing reactions from non-existing messages.
| * \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into paginator-react-404Gravatar S. Co12020-02-23-14/+11
| |\ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / |/| | | | | | | | | | | |
* | | | | | | | | | | | | Merge pull request #759 from ↵Gravatar S. Co12020-02-23-1/+1
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | python-discord/optimize/pagination/emoji_collection_structure Pagination migrations - Emoji Data Structure Modified
| * \ \ \ \ \ \ \ \ \ \ \ \ Merge branch 'master' into optimize/pagination/emoji_collection_structureGravatar S. Co12020-02-23-478/+814
| |\ \ \ \ \ \ \ \ \ \ \ \ \ | |/ / / / / / / / / / / / / |/| | | | | | | | | | | | |
* | | | | | | | | | | | | | Merge pull request #772 from python-discord/dm_failure_fixGravatar S. Co12020-02-23-12/+9
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't log exception traceback on Forbidden for welcomes.
| * | | | | | | | | | | | | | Don't log exception traceback on Forbidden for welcomes.Gravatar scragly2020-02-23-12/+9
|/ / / / / / / / / / / / / /
| * | | | | | | | / / / / / Pagination migrations - Emoji Data Structure ModifiedGravatar F4zi2020-02-17-1/+1
| | |_|_|_|_|_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Changed the pagination emoji collection from list to tuple This change was suggested since this collection is constant
| | * | | | | | | | | | | Define `_count` in `__init__`.Gravatar scragly2020-02-23-0/+1
| | | | | | | | | | | | |
| | * | | | | | | | | | | Suppress NotFound on react clear, tidy imports.Gravatar scragly2020-02-23-18/+21
| | | | | | | | | | | | |
| | | * | | | | | | | | | Remove the space that makes the test failGravatar Deniz2020-02-21-1/+1
| | | | | | | | | | | | |
| | | * | | | | | | | | | Use the code provided by sco1 to fix the checks failing.Gravatar Deniz2020-02-21-5/+10
| | | | | | | | | | | | |
| | | * | | | | | | | | | Add thousand separators to Members count, closes #744Gravatar Deniz2020-02-13-4/+4
| | | | | | | | | | | | |
| | | * | | | | | | | | | Update the tests accordingly to reflect the new changesGravatar Deniz2020-02-13-3/+3
| | | | | | | | | | | | |
generated by cgit v1.2.3 (git 2.39.1) at 2025-10-30 18:57:19 +0000