Skip to content

Comments

added thread subject to post reports#9

Open
tzdlr wants to merge 4 commits intoSvePu:masterfrom
tzdlr:master
Open

added thread subject to post reports#9
tzdlr wants to merge 4 commits intoSvePu:masterfrom
tzdlr:master

Conversation

@tzdlr
Copy link

@tzdlr tzdlr commented Feb 15, 2024

just a small enhancement to include the thread subject, cuz on big boards you run out of overview when you just see repoted post by you will never find the corresponding report again :D

@Sama34
Copy link

Sama34 commented Feb 15, 2024

I think you did add the post subject in the sprintf() method, but is unused in the language string.

@tzdlr
Copy link
Author

tzdlr commented Feb 15, 2024

right, i just moved the post title at pos4, but thought it might be TOO much information in a thread, maybe creating a setting for it would solve the dilemma which one someone would prefer to use

@Sama34
Copy link

Sama34 commented Feb 16, 2024

Since people report posts and not threads, I would say that the post subject is more reliable (say, people actually change the post's subject).

Also, it seems this will generate a warning if $thread or $post aren't set. You need to move the full line into each switch case, as well as use a different language string for each case.

@tzdlr
Copy link
Author

tzdlr commented Feb 16, 2024

ouh yeah i missed that point, our board is strictly moderated so we dont have any, nor had any profile reports but therefore more thread reports regarding to board subject but you're right will revamp to suit all use cases!

… nicer overview with more information in thread title
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.

2 participants