Skip to content

Conversation

@sebkuip
Copy link
Member

@sebkuip sebkuip commented May 23, 2024

This update changes the ?snippet command.
When the snippet command is invoked with no snippet name, but snippets do exist, it will not only show a list of all snippets, but also show a (possibly truncated to a length of 350 characters) preview.
Also changed the amount of snippets per page from 15 to 10 per page to accommodate the more space taken up.

Copy link
Member

@lorenzo132 lorenzo132 left a comment

Choose a reason for hiding this comment

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

Works as intented, can be merged in my opinnion.

@StephenDaDev StephenDaDev self-requested a review April 15, 2025 18:24
Copy link
Member

@StephenDaDev StephenDaDev left a comment

Choose a reason for hiding this comment

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

I think this adds functionality that is useful to our member base. LGTM.

Copy link
Member

@StephenDaDev StephenDaDev left a comment

Choose a reason for hiding this comment

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

Also tested as working on 4.1.2

Copy link
Collaborator

@Taaku18 Taaku18 left a comment

Choose a reason for hiding this comment

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

Reviewed by stephen and lorenzo.

@Taaku18 Taaku18 merged commit 3ab5161 into modmail-dev:development Apr 16, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants