Group authoring fields together - #308
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughIl blocco UI Authors con Choices.js è stato spostato dopo il campo Lingua nel form del libro. Gli identificativi, i nomi degli elementi e la logica server/JavaScript restano invariati. ChangesForm del libro
Estimated code review effort: 1 (Trivial) | ~2 minuti Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
|
I've incorporated this. The Authors picker now sits with the other authoring fields — right after the language field and before the contributor-role pickers (illustrator / translator / curator / colorist) — instead of being stranded after the publishers block, in The change lives on the frontend design branch that's next in line to merge, so I'm closing this PR here. |
CSV alias - PR #308 "Group authoring fields together": move the Authors picker up so it sits with the other authoring fields (right after language, before the contributor-role pickers) instead of being stranded after the publishers block. - PR #305: the en_US wording and README ("number in series") were already shipped in v0.7.45; the only missing bit was the CSV import header alias — added 'number in series' alongside 'series number' so a re-exported "Number in Series" column maps back to numero_serie.
Request from our librarians:
Seems reasonable
Summary by CodeRabbit