[comixed-dev] Re: 0.7 RC testing.

  • From: "Darryl L. Pierce" <mcpierce@xxxxxxxxx>
  • To: comixed-dev@xxxxxxxxxxxxx
  • Date: Sun, 13 Sep 2020 14:50:06 -0400

On Sun, Sep 13, 2020 at 6:22 AM Sean Alexander
<dmarc-noreply@xxxxxxxxxxxxx> wrote

Initial round of comments:

Import

selected and imported 47 comics.  After import, the import screen is still 
showing 47 selected comics.

Webp formatted comics appear to have fully imported – including multiple 
trades that previously threw a memory error and didn’t import.  Yay (pending 
further confirmation).

Glad to hear this! We'll need to keep an eye on WebP processing and
see if/when that library is updated so we can update our dependency.
Do you know what the current version is?

Importing without metadata works as expected.

Account details tab

no user friendly name for user-details.label.last-logged-in

This is fixed on the release branch, but has not been pushed yet. I
want to wait until we get a good set of bugs fixed before I roll
another RC so we don't go through as many release candidates as last
time.

Restoring deleted comics

works as expected.

Reading list created in library view

attempted to add all selected comics to it – newly created reading list was 
not immediately displayed – unsure of delay time.  Not an issue once a user 
is aware there could be a delay.

I've got some ideas for improving the library updates that will
address this and the moving issue below with things appearing to
double the library size. Long story short, I'm going to move
maintaining the user's library state to the server and have it send
updates to the frontend as the page changes.

Organize library

seemed to work, file named didn’t seem to work as expected and five files 
weren’t renamed – they were left with a “cbz-old” extension.  All comics in 
question have “:” in the name.

Hrm, this is odd. The : should be changed to a _ as part o the file
name scrubbing piece. What did the new filename look like for them?

Using the example naming rule ($PUBLISHER/$SERIES/$VOLUME/$SERIES v$VOLUME 
#$ISSUE ($COVERDATE)), comics were put in a weird directory – using Catwoman 
as an example:  \DC Comics\Catwoman\2018\DC Comics\Catwoman\2018.  
Essentially the path was duplicated twice.  Could be user error though – will 
try again tomorrow night.

The five “cbz-old” comics are not showing in the Missing page as I would have 
expected.  Requires library refresh?

If you reload the app do they show up as missing? If so then there's
an issue with the library updates detecting missing comics, which is
highly likely. Looking for missing comics is a hard one since it would
require the server to constantly look at what's in the library and
what's on disk and see if one of them is gone.

“New parent directory” and “file renaming rule” didn’t keep values previously 
used.

Yeah, that's an enhancement to the feature that we need to open an
issue to get done.

No way to tell if process is underway or completed (outside looking at logs). 
 Ideally an indicator similar to the importing one would be added.

That's a good idea. With the next release I'll see about having the
server-side state track things like that s well.

Empty directories would ideally be removed – probably caution needed here 
though.

Yeah, I'm leery of doing that just yet. Until we hit a good threshold
of comfort in the library management I'd avoid having it delete
directories just yet.

Task logs

would like to be able to sort them, as a user I’d rather see the latest log 
first, rather than having to navigate to the last page to see it.

Definitely an enhancement issue that should be created.

Convert comics

worked as expected, however once the process had finished the library 
appeared to have twice as many comics.  A browser refresh was required to 
remove the extra entries.  Reading lists were correctly updated with the new 
file names.

As I mentioned above, I have plans for tightening up the library
refreshes on the server side that should solve this problem.

Task logs

Unable to read full entry text.  Would be useful if this was wraparound or 
expandable.

Yeah, that's another enhancement that should have an issue created, to
have clicking on an entry pop open a sidebar to show the full text.
Like how the Web Audit Log works currently.

-- 
Darryl L. Pierce <mcpierce@xxxxxxxxx>
"Le centre du monde est partout." - Blaise Pascal
"Let's try and find some point of transcendence and leap together." -
Gord Downie

Other related posts:

  1. Home
  2. comixed-dev
  3. Archive
  4. 09-2020