Skip to content

fix: breaking threadmenu fixes.#3425

Open
lorenzo132 wants to merge 6 commits into
developmentfrom
users/lorenzo/breaking-threadmenu-fix
Open

fix: breaking threadmenu fixes.#3425
lorenzo132 wants to merge 6 commits into
developmentfrom
users/lorenzo/breaking-threadmenu-fix

Conversation

@lorenzo132

Copy link
Copy Markdown
Member

This PR solves the following:

  • thread creation menu null pointer bugs
  • false cancelled cache entry
  • If a menu times out, and the user send multiiple messages in the meantime the user would not be able to create a new thread with previous code.

@StephenDaDev

Copy link
Copy Markdown
Member

I'll lump this in with v4.3.0 as well even though it wasn't included in our previous discussions

@StephenDaDev StephenDaDev added changelog Changes in PR have been added to draft release that will be used for the changelog on the next ver. and removed changelog Changes in PR have been added to draft release that will be used for the changelog on the next ver. labels Dec 22, 2025
martinbndr
martinbndr previously approved these changes Dec 23, 2025
Copilot AI review requested due to automatic review settings December 24, 2025 15:50
@lorenzo132 lorenzo132 requested review from StephenDaDev, martinbndr and sebkuip and removed request for Copilot December 24, 2025 15:51
martinbndr
martinbndr previously approved these changes Dec 28, 2025
@lorenzo132 lorenzo132 enabled auto-merge (rebase) May 12, 2026 15:42
@lorenzo132 lorenzo132 disabled auto-merge May 12, 2026 15:42
@lorenzo132 lorenzo132 requested a review from martinbndr May 12, 2026 15:46
Comment thread core/thread.py Outdated
Comment thread core/thread.py
Comment thread core/clients.py Outdated
@StephenDaDev StephenDaDev added the changelog Changes in PR have been added to draft release that will be used for the changelog on the next ver. label May 13, 2026
@StephenDaDev

Copy link
Copy Markdown
Member

Added to changelog for 4.3.0

StephenDaDev added a commit that referenced this pull request May 13, 2026
sebkuip
sebkuip previously approved these changes Jun 14, 2026

@sebkuip sebkuip left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After the changes requested have been resolved, this looks fine now.

StephenDaDev
StephenDaDev previously approved these changes Jun 15, 2026

@StephenDaDev StephenDaDev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and seems to function properly without error or sideeffects on test instance.

martinbndr
martinbndr previously approved these changes Jun 15, 2026
martinbndr and others added 6 commits June 17, 2026 18:01
Replaces the claim plugin by fourjr to my claim plugin due to being fundamentally broken as of the current time. It has been created few support issues already that were not successfull to use the plugin.
This adds two commands for renaming snippets and aliases for easier name editing.
Improves the make_alias function.
This PR solves the following:
- thread creation menu null pointer bugs
- false cancelled cache entry
- If a menu times out, and the user send multiiple messages in the meantime the user would not be able to create a new thread with previous code.
This resolves an issue introduced in: #3423
Autocloses would fail.
@lorenzo132 lorenzo132 dismissed stale reviews from StephenDaDev and sebkuip via 2b9856d June 17, 2026 16:10
@lorenzo132 lorenzo132 force-pushed the users/lorenzo/breaking-threadmenu-fix branch from 6c23d29 to 2b9856d Compare June 17, 2026 16:10
@lorenzo132 lorenzo132 requested review from StephenDaDev, martinbndr and sebkuip and removed request for martinbndr and sebkuip June 17, 2026 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog Changes in PR have been added to draft release that will be used for the changelog on the next ver.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants