aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
...
| | * | | 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
* | | | Tests: fix setting bot-commands ID in information testsGravatar MarkKoz2020-02-23-4/+4
| | | |
* | | | Constants: rename Guild.Constant.ignored to modlog_blacklistGravatar MarkKoz2020-02-23-7/+8
| | | | | | | | | | | | | | | | This name better explains what the list is for.
* | | | Config: replace abbreviated lists with normal onesGravatar MarkKoz2020-02-23-4/+34
| | | | | | | | | | | | | | | | | | | | Lists were getting too long to be readable as one line. Having each element on a separate line also reduces merge conflicts.
* | | | Config: add underscore to devcontribGravatar MarkKoz2020-02-23-2/+2
| | | |
* | | | Config: fix DEV_LOG variable thingyGravatar MarkKoz2020-02-23-2/+2
| | | |
* | | | Always load doc and verification extensionsGravatar MarkKoz2020-02-23-5/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | They used to only be loaded in "debug mode" because the main guild was used to test the bot. However, we have since moved to using a separate test guild so it's no longer a concern if these cogs get loaded. It was confusing to some contributors as to why these cogs were not being loaded since the debug mode isn't really documented anywhere.
* | | | Config: rename channels to match their names in the guildGravatar MarkKoz2020-02-23-76/+76
| | | |
* | | | Constants: remove code jam champions roleGravatar MarkKoz2020-02-23-2/+0
| | | | | | | | | | | | | | | | Nothing was using it.
* | | | Constants: rename developers role back to verifiedGravatar MarkKoz2020-02-23-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | It makes the code which uses it more readable. A comment was added to explain the discrepancy between the constant's name and the name in the guild.
* | | | Constants: rename roles to match their names in the guildGravatar MarkKoz2020-02-23-28/+28
| | | |
* | | | Config: remove checkpoint_test and devtestGravatar MarkKoz2020-02-23-6/+0
| | | | | | | | | | | | | | | | | | | | | | | | They no longer exist in the guild. * Move devlog under the "Logs" category
* | | | Config: shorten name of PYTHON_COMMUNITY_ROLEGravatar MarkKoz2020-02-23-7/+7
| | | |
* | | | Config: split channels into categoriesGravatar MarkKoz2020-02-23-21/+39
| | | |
* | | | Config: split roles into categoriesGravatar MarkKoz2020-02-23-12/+16
| | | |
* | | | Config: rename roles to match their names in the guildGravatar MarkKoz2020-02-23-18/+18
|/ / /
* | | Merge pull request #749 from python-discord/reminder-enhancementsGravatar Leon Sandøy2020-02-23-15/+47
|\ \ \ | |_|/ |/| | Reminder Embed Enhancements & Whitelist Expansion
| * | Merge branch 'master' into reminder-enhancementsGravatar S. Co12020-02-23-607/+951
| |\ \ | |/ / |/| |
* | | 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
|/ / / / / / / /