Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Harvest: Automatically update manage client page as harvesting completes. #3260

Closed
kcondon opened this issue Aug 11, 2016 · 4 comments
Closed

Comments

@kcondon
Copy link
Contributor

kcondon commented Aug 11, 2016

Currently when you run a harvest client, the UI has a message indicating you need to refresh the page to see the status. Although as designed, It would be nice if this happened automatically in a future release.

@kcondon kcondon added UX & UI: Design This issue needs input on the design of the UI and from the product owner Type: Suggestion an idea Priority 1: Low labels Aug 11, 2016
@djbrooke
Copy link
Contributor

Removing from 4.5, possibly will be added in a future release.

@landreev
Copy link
Contributor

Adding my comment I added earlier to Kevin's "test checklist" document:

I'm afraid we'll have to live with this in 4.5. That's why the message says "refresh the page to see the results..." This approach is being used in other pages... so it's ok, right?

(end of quote)

Yeah, it's ugly... But it's not as trivial as it one would expect... It took some effort to make the tabular ingest status of a file auto-update on the files page. I want to put some thought into designing this properly... And yeah, the fact that some other pages (such the group page) use this "refresh the page to check on the progress..." message solution makes me feel a tiny bit better about this.

@pdurbin pdurbin added the User Role: Superuser Has access to the superuser dashboard and cares about how the system is configured label Jul 12, 2017
@pdurbin pdurbin added Status: Code Review and removed UX & UI: Design This issue needs input on the design of the UI and from the product owner Type: Suggestion an idea User Role: Superuser Has access to the superuser dashboard and cares about how the system is configured labels Oct 30, 2018
@pdurbin
Copy link
Member

pdurbin commented Oct 30, 2018

Pull request #5248 just came in from @tcoupin and I believe he is addressing this issue so I moved it to code review.

@pdurbin
Copy link
Member

pdurbin commented Jun 7, 2019

Pull request #5248 was merged as part of 4.10 but #5735 (Manage Harvesting Client page keeps refreshing. Unable to add or edit clients) cropped up and pull request #5926 was just merged to revert the original pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants