Page MenuHomePhabricator

Allow newsletter extension to publish issues with section anchors
Closed, ResolvedPublicFeature

Description

Feature summary: Allow newsletter extension to publish issues that point to page sections (as opposed to only being limited to pages).

Use case(s) : Reported by User:Gryllida on MW:Newsletter Talk page about having using the extension to send Echo notification to a group of subscribers, but in this case, with page sections (instead of new pages) as issues.

Benefits: Seems like we had a discussion about it back during the original discussions around the Newsletter extension. In general, this should make the extension more flexible (I guess)?

Event Timeline

Pppery subscribed.

This would require a schema change to the nli_issues table.

Actually it doesn't since the nl_issues table is never read from. Something in Echo is losing the fragment, and I'm not inclined to spend my time determining what it is.

I played around with a bit, and saw that it was Echo that was killing the fragments by relying simply on the pageId. I thought about providing the fragment as an extra param while creating the issue, and using it when rendering it. Works, but, to be honest, this has to be implemented at Echo level, IMHO. But since that is a big ask, probably we might be able to live with this as well?

Change #1147111 had a related patch set uploaded (by 01tonythomas; author: 01tonythomas):

[mediawiki/extensions/Newsletter@master] Support fragments to issue titles when announcing

https://gerrit.wikimedia.org/r/1147111

Change #1147111 merged by jenkins-bot:

[mediawiki/extensions/Newsletter@master] Support fragments to issue titles when announcing

https://gerrit.wikimedia.org/r/1147111

Pppery assigned this task to 01tonythomas.

I'm happy this is resolved, and added an entry for the Tech News. Please fill free to fix or remove it, if I'm wrong. Thank you.