Skip to content
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

push from pootle #2

Merged
merged 3 commits into from Apr 10, 2017
Merged

push from pootle #2

merged 3 commits into from Apr 10, 2017

Conversation

leonardcj
Copy link
Contributor

No description provided.

@leonardcj
Copy link
Contributor Author

Ping. Can we land this please?

@quozl
Copy link
Contributor

quozl commented Feb 20, 2017

Thanks, that reminds me. Isn't this a older version of https://github.com/sugarlabs/get-books-activity ? A quick look at the history of both repositories suggests this one might be orphaned after the changes are merged.

get-books-activity and getiabooks share a common ancestor commit at 7bb6f39 in 2009, when OPDS support was added to get-books-activity.

getiabooks last functional change was in 2013 with 5bad15f.

get-books-activity has continued to receive functional changes right up to the end of 2016.

Both are listed on translate.sugarlabs.org.

I've not compared the activities functionally.

@jdsimmons
Copy link
Contributor

jdsimmons commented Feb 20, 2017 via email

@quozl
Copy link
Contributor

quozl commented Feb 20, 2017

Thanks. Looks like E-Book Enlightenment has not kept up with developments. I can't find a published date; when was it? Get Books uses the Advanced Search form query for Internet Archive, supports the other download formats including Deja Vu and EPUB, and shows other metadata such as language and publisher. The code that does the search is in opds.py but it does not use OPDS; the code should move to a different file. Get Books uses OPDS for other book repositories.

Here is Get Books 16 with the same query as in the E-Book Enlightenment screenshot:

get-books

@jdsimmons
Copy link
Contributor

jdsimmons commented Feb 20, 2017 via email

@quozl
Copy link
Contributor

quozl commented Feb 20, 2017

Thanks. I'll review the two activities to figure out if there is any feature missing, and push the feature into the one with the most features. I don't know which one will end up kept, so this translation pull request can be merged. Sorry for distracting!

@walterbender walterbender merged commit 299040a into sugarlabs:master Apr 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants