Skip to content

BookMap: fix help text wording #10544

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

Merged
merged 1 commit into from
Jun 5, 2023
Merged

Conversation

poire-z
Copy link
Contributor

@poire-z poire-z commented Jun 5, 2023

See #10530 (comment).

@Frenzie: please have a look at the new strings appearing in weblate, that I didn't miss/mess anything else :)


This change is Reviewable

@poire-z poire-z added this to the 2023.06 milestone Jun 5, 2023
@Frenzie
Copy link
Member

Frenzie commented Jun 5, 2023

A bit hard to tell. It seems to have been broken since #10492.

@poire-z
Copy link
Contributor Author

poire-z commented Jun 5, 2023

A bit hard to tell. It seems to have been broken

What "it" ?
Weblate and our translations abilities are broken?

@Frenzie
Copy link
Member

Frenzie commented Jun 5, 2023 via email

@poire-z
Copy link
Contributor Author

poire-z commented Jun 5, 2023

Not really following you :/
Was just wondering if my split of long help texts into 2 or 3 bits was ok, ie. translators are able to notice that, reuse the previous longer single text as the suggestion for the 2 or 3 bits, etc...
Anyway, is this PR good to go ?

@Frenzie
Copy link
Member

Frenzie commented Jun 5, 2023

https://github.com/koreader/koreader/runs/13993281963 fails, so docs and translation are blocked from running.

@Frenzie Frenzie merged commit 18cd6b7 into koreader:master Jun 5, 2023
@poire-z poire-z deleted the bookmap_wording branch June 5, 2023 16:57
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.

2 participants