-
-
Notifications
You must be signed in to change notification settings - Fork 601
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
Add archived talk pages access in the app #3451
Conversation
…s-android-wikipedia into archived-talk-pages-design
…s-android-wikipedia into archived-talk-pages-design
"&inprop=varianttitles" + | ||
"&srwhat=text&srinfo=suggestion&srprop=&sroffset=0&srlimit=1" | ||
) | ||
suspend fun searchSubPages(@Query("gpssearch") searchTerm: String?, |
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.
A trailing slash is missing. For example, https://de.wikipedia.org/w/api.php?action=query&list=prefixsearch&pssearch=Diskussion:Gl%C3%A4ubiger returns Diskussion:Gläubigerschutz
as well, even though it’s definitely not an archive of Diskussion:Gläubiger. https://de.wikipedia.org/w/api.php?action=query&list=prefixsearch&pssearch=Diskussion:Gl%C3%A4ubiger/ returns the expected result (nothing, as the given talk page has no archives).
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.
Already handled in #3475.
@@ -90,6 +90,7 @@ | |||
<string name="menu_page_new_tab">New tab</string> | |||
<string name="menu_page_reading_lists">Reading lists</string> | |||
<string name="menu_page_recently_viewed">Recently viewed</string> | |||
<string name="menu_page_archive">Archive</string> |
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.
Why singular? There may be several archive pages, and for this reason it’s usually in plural (cf. Template:Archives on enwiki).
@@ -1196,6 +1197,7 @@ | |||
<string name="talk_footer_view_user_page">View user page</string> | |||
<string name="talk_footer_view_article">View article</string> | |||
<string name="talk_footer_last_modified"><![CDATA[Last edited %1$s by %2$s]]></string> | |||
<string name="talk_archived_title">Talk archive: %s</string> |
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 should also be plural, as it lists the several archives.
https://phabricator.wikimedia.org/T275917