Bug 35401 - ERM > Titles > Import from a list needs a full page reload to get the up to date lists
Summary: ERM > Titles > Import from a list needs a full page reload to get the up to d...
Status: NEW
Alias: None
Product: Koha
Classification: Unclassified
Component: ERM (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal (vote)
Assignee: Jonathan Druart
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-11-26 23:06 UTC by Victor Grousset/tuxayo
Modified: 2023-11-29 21:50 UTC (History)
5 users (show)

See Also:
Change sponsored?: ---
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Victor Grousset/tuxayo 2023-11-26 23:06:28 UTC
Test plan

1. enable ERMModule syspref
2. go anywhere in the ERM module
3. in another tab, create a new list
4. in the ERM tab, go to eHoldings > Local > Titles > Import from a list
5. you don't see the list
6. refresh the page
7. the list is here
Comment 1 Victor Grousset/tuxayo 2023-11-27 22:37:01 UTC
> 3. in another tab, create a new list

And add records. Even though an empty list is visible with a full reload.
Comment 2 Jonathan Druart 2023-11-28 08:21:06 UTC
Yes, data are loaded when the app is mounted. That's "expected". Not a bug for me, we don't want to fetch all the data the app needs every time we go to a new view.
Comment 3 Pedro Amorim 2023-11-28 09:23:15 UTC
(In reply to Jonathan Druart from comment #2)
> Yes, data are loaded when the app is mounted. That's "expected". Not a bug
> for me, we don't want to fetch all the data the app needs every time we go
> to a new view.

Even with that being the case, it's still a confusing experience for the user!
Comment 4 Jonathan Druart 2023-11-28 09:46:05 UTC
(In reply to Pedro Amorim from comment #3)
> (In reply to Jonathan Druart from comment #2)
> > Yes, data are loaded when the app is mounted. That's "expected". Not a bug
> > for me, we don't want to fetch all the data the app needs every time we go
> > to a new view.
> 
> Even with that being the case, it's still a confusing experience for the
> user!

We "just" need a route for lists. Bug 28965 (but maybe not enough).
Comment 5 Victor Grousset/tuxayo 2023-11-29 21:50:45 UTC
(In reply to Jonathan Druart from comment #2)
> we don't want to fetch all the data the app needs every time we go
> to a new view.

It wasn't about all data.


(In reply to Pedro Amorim from comment #3)
> Even with that being the case, it's still a confusing experience for the
> user!

Indeed, no data consistency with the chronological order of page navigation (not sure about the phrasing ^^") even if it's on different tabs should be considered a bug. I think the only close thing to an exception are stuff when there are like tabs when editing a record or a framework subfield. The UI is in-page tabs, it's not perceived as going to another page, it's the same functional page. And there is no important external data loaded there. Inside the ERM it's completely different pages functionally and some display data from other part of Koha that might be updated in the meantime. Here the workflow make it likely part of the time to go create a list in another tab, before being in the right page in the ERM. Whereas the record editor being the same page functionally, it's not expected to have the framework fields refreshed by switching between tabs. So one would refresh the page. The same for the ERM if being already on ERM > Titles > "Import from a list" before going to create the list elsewhere. There, it's expected to refresh the page.


(In reply to Jonathan Druart from comment #4)
> We "just" need a route for lists. Bug 28965 (but maybe not enough).

Great. The important was to have a ticket here documenting the behavior and being known if patches are accepted. If no users with a support contract or in-house dev capabilities ever triggers the making of a fix, it's okay. So many fishes for so few frying pans ^^