-
Notifications
You must be signed in to change notification settings - Fork 155
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
speedy: add T5 #2097
base: master
Are you sure you want to change the base?
speedy: add T5 #2097
Conversation
case 11: // template talk | ||
case 828: // module | ||
case 829: // module talk | ||
appendList('Templates and modules', Twinkle.speedy.templateList); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you please walk me through what this code is doing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This enables T5 for the template and module namespaces, as well as their talk pages
I think we need to create {{Db-t5-notice}} and {{Db-t5-deleted}} on enwiki in order for the various talk page notifications to work. Can you please create those then let me know in this ticket? You can look at similar templates for other CSDs and copy/paste then adapt those. |
Do we want any of these notifications to be on by default? If so, they'll need to be added to twinkle.js -> Twinkle.defaultConfig. |
There's no need for any talk page notice, since the criterion is usually for non-controversial maintenance. |
Fixes #2094