Arvados

New version first commit: https://dev.arvados.org/projects/arvados-workbench-2/repository/revisions/df0c0d462a52003cd722d25520cd7a4ad6583c57
Test run: developer-tests-workbench2: #503
Branch: 18207-Workbench2-is-not-clearing-the-project-content-when-switching

Project switch should trigger loader

The fix seems to work as expected: the data explorer component immediately shows an "in progress" indicator when changing projects, but I think the code updates make the ProjectPanel component a bit more coupled with external variables and/or state.

We already have an app-wide "in progress" indicator that is being used, and also state.dataExplorer?.projectPanel.items could be reset (or set to undefined?) on every panel change, what do you think? I feel that duplicating the previous query results & browser's href shouldn't be necessary and makes things a little more complicated to manage.

Related to this, the "Shared with me" at the left side bar doesn't seem to be using this new change, this may also be a symptom of needing to fix the problem in a deeper level (at DataExplorer's level for example) so any part of the app could benefit.

New version first commit: https://dev.arvados.org/projects/arvados-workbench-2/repository/revisions/394ebdfd13fe40a7096f484c46a353d2537f4c9a
Test run: developer-tests-workbench2: #551
Branch: 18207-Workbench2-is-not-clearing-the-project-content-when-switching

Merged main

Lucas Di Pentima wrote:

The fix seems to work as expected: the data explorer component immediately shows an "in progress" indicator when changing projects, but I think the code updates make the ProjectPanel component a bit more coupled with external variables and/or state.

We already have an app-wide "in progress" indicator that is being used, and also state.dataExplorer?.projectPanel.items could be reset (or set to undefined?) on every panel change, what do you think? I feel that duplicating the previous query results & browser's href shouldn't be necessary and makes things a little more complicated to manage.

Related to this, the "Shared with me" at the left side bar doesn't seem to be using this new change, this may also be a symptom of needing to fix the problem in a deeper level (at DataExplorer's level for example) so any part of the app could benefit.

The updates made in arvados-workbench2|df0c0d4 are just a main merge. Could you explain here why you dismissed my 3 comments? I think it's important for historic reasons that the discussion about the change reasonings is kept in the ticket.

I merged this code before going AFK for the longer time, so it can be deployed within the next release as I was asked about this fix. I wrote explaination in the private message to you.

Daniel Kutyła wrote:

I merged this code before going AFK for the longer time, so it can be deployed within the next release as I was asked about this fix. I wrote explaination in the private message to you.

Well, I don't think it's ready to be merged yet, so going AFK isn't enough to make it ready.

As I mentioned #18207-7 and #18207-12, there're still some issues, being one of them another instance of this same bug, maybe you overlooked my comments, or perhaps you decided they aren't worth addressing.

If the latter is the case, please explain the reasons why. We're an open-source project, and we should record all discussions related to it in public because it'll surely be helpful to have the reasons documented for future reference.

Hi Lucas
I tried to reuse this loader
that we use commonly
unfortunately it is way to hard
because it is triggered by the services
and thus it is very unpredictable
even though I remove some loaders it is not possible to prevent for new one to happen (because of the ongoing network reuquests)
I rebased branch with the newest main
so if this is not crucial I would opt for merging this one
and maybe we could create a ticket to change the way loader is shown

Lucas Di Pentima wrote:

Related to this, the "Shared with me" at the left side bar doesn't seem to be using this new change, this may also be a symptom of needing to fix the problem in a deeper level (at DataExplorer's level for example) so any part of the app could benefit.

So how about this? Please make sure the "Shared with me" listing behaves the same way as the rest. As I mentioned on my first comment, the fact that it isn't showing the progress indicator is a clue that there might be code duplication, so it would be great to eliminate that.

I check shared with me and it seems to be working as other only the main project seems to be not showing the loader but for me it is loading reallyfast that is why loader is not shown

New version first commit: https://dev.arvados.org/projects/arvados-workbench-2/repository/revisions/f528cc85e7687e1d899860c4226984bd067c5bd3
Test run: developer-tests-workbench2: #560
Branch: 18207-Workbench2-is-not-clearing-the-project-content-when-switching

Removed duplicated loader to make the UI consistent, but kept the check with 'null' as a value when application is 'working' in order to force React to re-render and thus remove old project contents from the screen

For UI consistency, if you want, you could add some images and messages similar to the ones we show on empty projects but notify the user that they should be getting the data soon, as it's being loaded.
The 'null' option is acceptable by me, too; I just think it would be nicer to add some content to the blank space.
About my previous comment, the UI inconsistency I was referring to is that the "Shared with me" panel doesn't behave the same way as the project panel in the sense that its contents don't get cleared before loading the new data.
I've looked at the code, and I think that passing the loading prop to DataExplorer in src/views/shared-with-me-panel/shared-with-me-panel.tsx would do the trick.

New version first commit: https://dev.arvados.org/projects/arvados-workbench-2/repository/revisions/89a4eb90454f12be703f711279d0867781013631
Test run: developer-tests-workbench2: #562
Branch: 18207-Workbench2-is-not-clearing-the-project-content-when-switching

Added common icon, removed code duplication

Updates at arvados-workbench2|facccf58

• The 'Shared with me' issue from #18207-19 kept existing, and I also detected that the 'Groups' section was suffering from the same problem.
• To avoid further back and forth, I took the liberty to fix those, and I also replaced the "folder" icon with another more generic one, as the DataTable component is used not only on project listings, but on other object type listings.
• If you agree with the changes, I think this is ready to be merged.