Page MenuHomePhabricator

Use proper messages for log entries
Open, LowPublic4 Estimated Story Points

Description

Instead of just showing the type and action, use the existing messages to clarify
Additionally, use the log parameters (when relevant)

Rather than trying to implement a version of LogFormatter in javascript, just send the display to use in the api

Event Timeline

Restricted Application added a project: User-DannyS712. · View Herald TranscriptAug 12 2020, 6:12 AM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript
DannyS712 set the point value for this task to 4.Aug 12 2020, 6:13 AM
DannyS712 moved this task from Unsorted to Next on the User-DannyS712 board.
DannyS712 updated the task description. (Show Details)

...so it turns out that each log type has its own handling of the log parameters, etc. Rather than recreating the handling in javascript, I suggest adding an additional property to the watchlist api for the log display instead.

DannyS712 changed the task status from Open to Stalled.Aug 12 2020, 7:24 AM
DannyS712 triaged this task as Low priority.
DannyS712 moved this task from Next to Later on the User-DannyS712 board.

Stalled on subtask

DannyS712 renamed this task from Use `logentry-*` messages for log entries to Use proper messages for log entries.Aug 12 2020, 7:30 AM
DannyS712 updated the task description. (Show Details)
DannyS712 changed the task status from Stalled to Open.Sun, Nov 22, 5:24 AM
DannyS712 moved this task from Backlog to Later on the MediaWiki-extensions-GlobalWatchlist board.

Ready to go once extension requires 1.36+