rfellows commented on PR #7811: URL: https://github.com/apache/nifi/pull/7811#issuecomment-1769205342
This is good and works pretty well. The only improvement I could recommend would be to refresh the `current-user` after configuring a registry client. As it stands now, we are relying on the background polling of the current-user endpoint to update the current user which is where it is determined if `canVersionFlows` is true or false. It is this property that determines if we show the `No Registry Client available` dialog or the `Import Version` dialog. If you delete the registry client and then try to drag on a new import before the polling happens, you can be shown import dialog with no registries: <img width="766" alt="Screenshot 2023-10-18 at 3 35 34 PM" src="https://github.com/apache/nifi/assets/713866/a81a3a19-549d-4fdd-b138-9396d3d0fa44";> Similarly, if you do the same too soon after configuring a registry client, you can get a stale `No Registry Client available` dialog even though you have one configured. Maybe @scottyaslan can provide some more insight into if this is consistent with other areas of the application. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@nifi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
