Repository navigation
Release 1.12.0: the same-day handover switch, with the server calendar loans (#58) - #59
Merged
Merged
Conversation
Library settings -> Loans gains the web's Closures calendar section with same_day_handover_after_closure (Pinakes 0.9.0), sent only when changed and only when the server sends it. With #58 (usable return day and server lateness) this is the 1.12.0 release.
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes
I reviewed the 1.12.0 release commit. It adds the same-day handover switch to Library settings → Loans and bumps the version.
- New
CLOSURESloan section:same_day_handover_after_closureis aBOOLfield placed betweenapp_timezoneandpickup_expiry_days.LoansTabiteratesLoanSection.entriesand skips empty sections, so the Closures calendar heading only appears when the server sends the key. - Behaviour on older servers:
LoansForm.fields()filters onloans.settings, so a server that doesn't send the key shows nothing and puts nothing about it in the body. When the key is present but not ineditable_fields, the existing read-only On/Off path handles it. - Strings: label, help and section title are added to
en/it/de/fr. The four files still have the same keys, and the new copy has no em dashes. - Tests: the new 0.9.1 staff fixture and test check the field's kind, section, order, editability and the single-key patch body. The test also checks that the rc.2 fixture produces no field and a
nullbody. - Release metadata:
versionCode26 /versionName1.12.0, and a newSTATUS.mdsection that also covers #58.
claude-opus-5-5 | 𝕏
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Release 1.12.0 (versionCode 26), paired with Pinakes 0.9.0 and 0.9.1.
Changes
same_day_handover_after_closureto the loan settings: when a loan is due on a closed day, another reader can receive the book on the first open day. The Mobile API sends it inGET /staff/settings/loans, but the app's loan form skips keys it does not know. The setting was safe, because a save sends only changed fields, but it could not be seen or changed from the app. Library settings → Loans now has the web's Closures calendar section with this switch, its help text in it, en, de and fr (the server's wording), and the same order as the web page.main: loans showeffective_due_atand follow the server'sis_overdue.Older servers
A server that does not send the key shows no switch and sends nothing about it.
Tests
Staff090LibraryFormsTest.theSameDayHandoverSwitchFollowsTheClosuresCalendarAndIsSentWhenChangeduses a 0.9.1 response (s090_library_loans_staff_091.json), staff role. It checks:app_timezoneandpickup_expiry_days;The same test checks that the rc.2 response yields no switch and no body.
Staff090LibraryFormsTest(26) andStaff090LibraryUiTest(5) pass locally.