Skip to content

Fix edit remove snippets#54

Open
Kuuuube wants to merge 2 commits intoOpenTabletDriver:masterfrom
Kuuuube:fix-edit-remove-snippets
Open

Fix edit remove snippets#54
Kuuuube wants to merge 2 commits intoOpenTabletDriver:masterfrom
Kuuuube:fix-edit-remove-snippets

Conversation

@Kuuuube
Copy link
Copy Markdown
Member

@Kuuuube Kuuuube commented Apr 15, 2026

Fixes #52 and somewhat #53

The result of autocomplete_snippet is the id and title, not the id. So using get_snippet instead of get_snippet_lazy is possible here. Edit and remove snippet are also slash command only so this doesnt create an issue for trying to use it as a prefix command.

If there's a snippet with both the same id and title, this wont help but that isnt supposed to be possible.

Edit snippet now actually edits the snippet instead of creating a new snippet.

@Kuuuube Kuuuube added the bug Something isn't working label Apr 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Editing snippet creates a new snippet with the same ID

1 participant