Skip to content

Passages fixes and new features#281

Merged
PurpleGuitar merged 6 commits intodoc-dev.walink.orgfrom
passages-fixes-and-new-features
Nov 26, 2025
Merged

Passages fixes and new features#281
PurpleGuitar merged 6 commits intodoc-dev.walink.orgfrom
passages-fixes-and-new-features

Conversation

@linearcombination
Copy link
Contributor

  • Fix OT RG parser
  • Fix Passages app passages feature:

Make checkboxes for each passage group type only be visible if the
language chosen has at least one of the members of that group. Only
show select all for OT rg if all four rg groups (rg1, rg2, rg3, rg4)
are present. Only add those passages from each group type which
actually exist for the language chosen. Add e2e tests to test this.

Translation words are scoped by language and not by book. This could
have been changed a long time ago, but it was doing no harm. Just
cleaning up the design a bit.
Make checkboxes for each passage group type only be visible if the
language chosen has at least one of the members of that group. Only
show select all for OT rg if all four rg groups (rg1, rg2, rg3, rg4)
are present. Only add those passages from each group type which
actually exist for the language chosen. Add e2e tests to test this.
tq_book
for tq_book in tq_books
if tq_book.book_code == tq_book.book_code
if tq_book.book_code == book_code
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LOL, good catch!

@PurpleGuitar PurpleGuitar merged commit e7ae0c6 into doc-dev.walink.org Nov 26, 2025
39 of 43 checks passed
@github-project-automation github-project-automation bot moved this from In progress to Done in DOC Project Management Nov 26, 2025
@linearcombination linearcombination deleted the passages-fixes-and-new-features branch January 12, 2026 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants