Skip to content

Show desc on lists page#1108

Merged
TatevikGr merged 3 commits into
release-3.7.0from
show_desc_on_lists_page
May 8, 2026
Merged

Show desc on lists page#1108
TatevikGr merged 3 commits into
release-3.7.0from
show_desc_on_lists_page

Conversation

@TatevikGr
Copy link
Copy Markdown

@TatevikGr TatevikGr commented May 6, 2026

Description

Show description on Subscriber lists page

Related Issue

issue

Screenshots (if appropriate):

image

sandholzerk and others added 3 commits May 5, 2026 10:31
Currently the new version ist not recognised when upgrading. This results that no database-upgrade is initiated.
The program worked perfectly without this, it's just for correctness. :)
@TatevikGr TatevikGr requested a review from marianaballa May 7, 2026 13:49
@TatevikGr TatevikGr merged commit a30bf71 into release-3.7.0 May 8, 2026
32 of 39 checks passed
@bramley
Copy link
Copy Markdown
Contributor

bramley commented May 11, 2026

Sorry, a bit late commenting on this.

The change should probably use mb_substr() instead of substr() otherwise there is a possibility of splitting a multi-byte character at the 50th byte position.

For example, if the description is (yes a bit silly) then the second € is split and the description not displayed

This the description of a list which is long €€

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.

5 participants