Skip to content

patch-855 - #247

Closed
gmt2001 wants to merge 16 commits into
masterfrom
patch-855
Closed

gmt2001 wants to merge 16 commits into
masterfrom
patch-855

Conversation

@gmt2001

@gmt2001 gmt2001 commented Jul 21, 2026

Copy link
Copy Markdown
Owner

No description provided.

dependabot Bot and others added 16 commits July 20, 2026 08:22
Bumps [actions/setup-python](https://github.com/actions/setup-python) from 6 to 7.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](actions/setup-python@v6...v7)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-version: '7'
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Updated `getGuildId` and `getGuild` in `DiscordAPI.java` to handle Reactor `timeout` events via explicit `doOnError` logging for `TimeoutException` and using `onErrorReturn` for defaults. Added an extra 500ms to `getGuild` timeout.
Updated `getGuildId` and `getGuild` in `DiscordAPI.java` to explicitly filter `onErrorReturn` and `onErrorResume` on `TimeoutException.class`, ensuring fallbacks are only triggered upon timeouts as requested. Substituted `onErrorReturn(null)` with `onErrorResume(e -> Mono.empty())` to safely return `null` when `.block()` is called without throwing an assembly-time `NullPointerException`.
…timeout-behavior-14360002224792268229

Fix Discord guild timeout behavior in PhantomBot
…actions/actions/setup-python-7

Bump actions/setup-python from 6 to 7
Use guild ID check instead of guild object check to reduce API calls
Fix not checking for 0 guild ID before attempting to get guild
@gmt2001 gmt2001 closed this Jul 21, 2026
Comment on lines +110 to +111
setTimeout(function () {
say(message);
@gmt2001
gmt2001 deleted the patch-855 branch July 21, 2026 22:55
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants