Add the Paratext 9 import WebView experience with a first-open offer - #274
Add the Paratext 9 import WebView experience with a first-open offer#274jasonleenaylor wants to merge 6 commits into
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Team Run ID: Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
591ebbf to
7706947
Compare
352ac8a to
96db39b
Compare
7706947 to
5ce31d8
Compare
a7b6817 to
3ca877f
Compare
d5f84f5 to
378ab64
Compare
3ca877f to
d4070a8
Compare
e23f7d9 to
d74a2cf
Compare
|
❓ |
378ab64 to
6b4b8d7
Compare
e5c7fe8 to
3fd34ea
Compare
6b4b8d7 to
2504539
Compare
|
Deliberate. No resolver runs during import yet, so folding the sense counters would show zero resolved and everything unresolved on every run, which reads as a failure for work that is not wired up. The counters are populated and pinned on the converter side so they are ready when a resolver lands. Folding and rendering them belongs with the resolver work tracked in #280, which now lists it as a scope point; ReportTotals carries a TODO pointing there (ccb28ea). --drafted by Claude-- |
ccb28ea to
3f6df95
Compare
The UI half of the PT9 import. The select modal gains the import button (shown only when the source serves convertible data, via usePt9ImportAvailability); Pt9ImportModal carries the run, its report, and its failures, including the too-large refusal recognized by the RESOURCE_EXHAUSTED platform error code with the documented message marker as fallback. An import opens read-only: every editing affordance stays away, a banner carries sync and copy-to-editable, and CopyToEditableModal clones an import into an editable project. On the first open of a source with convertible PT9 data and no stored state, Pt9ConvertPromptModal offers the conversion up front: Yes runs the import as the only project created, No (or dismissing) persists the empty draft so the offer never repeats, per the user-questions entry. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
The Loader imports isPt9ImportReport from the converter surface that now owns it, the frontend-mock helper rides this tree with its users, and the offer and hasDraft tests use the shared ENOENT fixture. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
3f6df95 to
ef8e70b
Compare
The UI half of the PT9 import, stacked on #273 (converter: #272).
the source serves convertible data (usePt9ImportAvailability probes the
manifest through the projectInterface).
the too-large refusal, recognized by the RESOURCE_EXHAUSTED platform error
code with the documented message marker as fallback (pt9-import-error).
carries manual sync and copy-to-editable, and CopyToEditableModal clones an
import into an editable project through the normal draft-open flow.
stored interlinearizer state asks "This project has Paratext 9 interlinear
data. Would you like to convert it now?" Yes runs the import as the only
project created; No (or dismissing) persists the empty draft so the offer
never repeats. Recorded in user-questions.md for outside review.
This change is