[Koha-bugs] [Bug 18823] Advanced editor - Rancor - add ability to edit records in import batches

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=18823

Nick Clemens <nick at bywatersolutions.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|Signed Off                  |Failed QA

--- Comment #150 from Nick Clemens <nick at bywatersolutions.com> ---
Minor conflict from new clipboard feature on rancor

Preferences.Save( [% loggelogged_in_user.borrowernumber | $raw %] );
Should be 'logged_in_user'?

This is very awesome! It seems to work as expected for the most part, but I
found some functional issues that I think must be addressed:

After saving duplicate we should use the new record number and duplicate should
be unchecked - is essentially 'Save as new'?

Checking 'Catalog record ##' and 'New catalog record' would imply duplicate,
but it doesn't do that - maybe we shouldn't have 'New catalog record' when
editing an existing record?

Similarly, saving as 'Duplicate' and 'New catalog record' creates two new
records if I uncheck the current record.

Saving to "Batch: test" and "test: #5" is confusing, do I need to check both?
the Batch seems superfluous, I am not sure that unchecking the specific record
and selecting only the batch is clear that we are creating a new record, maybe
if a specific record is not checked we can show "Batch: test (new record)"

The export feature is awesome - it would be nice to have this on the 'Manage
staged records' page too, but not a blocker here

Not a blocker, but if I revert and imported batch should I be able to manage
that batch in Rancor?
Similarly, if I stage a new batch I must reload rancor to be able to manage
that batch

Some overall GUI cleanup would be nice, but I think this can go as is with
improvement later

-- 
You are receiving this mail because:
You are watching all bug changes.