Can we have separate strings for each screen instead of a common template? This will allow us to customise the messages. Like "Library Section: Find all your books here. You can Read, Share, or Delete them."

If the current locale is English, we dont have to show a loading.. text. we can directly display the text from the API

this is also fine. thanks

The book title URL should be linked to ws_url from the API

Can we add some visual feedback while the book is downloading? a small progress bar at the top of the library saying re-downloading book? Because right now when we click on the refresh button, it looks as if the book is deleted and nothing is happening.

Let us check with Readium and try to fix it there

Let us add a refresh type icon for each book in the Library. When the user clicks on it, they should first see a warning that any existing notes, highlights, and progress might be lost. We can then fetch the file from the ws_export URL and replace the existing file with the new one.

We are assuming this was caused due to down time on Wikimedia Commons. It looks resolved as of now.

Resolved through https://github.com/cis-india/Wikisource-Reader/pull/70

If any value is unknown, we should not display that line. example, it should not say, "Publisher: Unknown Publisher"
change label from "places of publication" to "place of publication"

Merged

@Muguro Thank you, merged. I'm no expert but in general these are various versions (1 to 5) of the db schema tracking all the migrations?

can you rename the ids for example from "high_to_low" to "sort_high_to_low"? That way, they are easily understood

Yeah it is normal. Merged

Can you rename the text to just "Random" instead of "Randomised Sort"? I've merged the random sort on the API side so you can test.

merged

Random sort should be added as an additional sort key. Should not alter the current default sorting behaviour of popularity. In the app, another sort option should be added for "Random".

@Muguro As I've mentioned in my earlier comment, the package name is org.cis_india.wsreader so please do not use com.starry.wikisourcesreader.

Some things I noticed:

Sure

@Muguro The TTS icon is missing in the playstore production release but it is working in local debugging. Can you check if this is related?

@Muguro Books with "?" in title are throwing an error on Download. Feel free to create a new task if this merged PR did not cause it.

If we do "--languages bn", it should only update Bengali books and leave other languages as it is. But right now, all books in the db are removed and only bn books are updated.

The timestamp in the API is not about the actual content in the work. It is when the specific item's metadata is updated (currently daily). Let us wait to hear other opinions and see if we can just add an update icon beside the share icon.

Afaik, you can contribute till end of the month? Anyway, would love to know how you plan to implement this

All Outreachy participants, please make sure you submit your applications before Nov. 3, 4pm UTC if you wish to.

Book titles for French and Italian books are not being displayed in the notification for some reason. Can you investigate? Also better to show the title in native language instead of the English one.

@System625 Can you make a slight change to show book title instead of the file name in notification? Because when we download works in languages other than English, it is showing a confusing random file name.

Merged