--- old+++ new@@ -1 +1,3 @@+Ensure we set the status when a clone starts (from a migration, SVN import, or fork).+
Show something more useful and obvious, rather than a tiny "initing" or "analyzing" in the sidebar.
milestone: forge-backlog --> forge-jul-27
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
This change removes the repo status from the sidebar and includes it more obviously in the body of all repo pages. If the repo status is not 'ready':
Status is displayed at top of page content (in new before_content block), with progress spinner
Status is checked and updated every 15s via ajax.
If/when status changes to 'ready', spinner stops, status is updated, and a link is appended prompting user to refresh the page.
On the repo index page, the 'Empty Repo' page is not displayed if the repo status is not 'ready' (gives repo setup/import/clone time to complete before showing the 'Empty Repo' page.
To test, create a Git repo on a sandbox. Find the repo in mongo:
I'm getting a js error when I go to the repo in analyzing status (see screenshot). Also, the yellow shadow around analyzing looks weird. I think the message is obvious enough from the placement, loading icon, and font size. It would be nice to make the page take up the whole container width since it looks strange to have a blank space over there where the sidebar would go.
Diff:
forge:tv/4568
sftheme:tv/4568
This change removes the repo status from the sidebar and includes it more obviously in the body of all repo pages. If the repo status is not 'ready':
before_content
block), with progress spinnerTo test, create a Git repo on a sandbox. Find the repo in mongo:
After changing the status to 'analyzing', refresh the repo page in a browser. Confirm status/spinner displayed. Change status to 'ready' in mongo:
Within 15 seconds, the status on the repo web page should update to 'ready' with a link to refresh.
Can also test with Hg/Svn, and with allura theme.
I'm getting a js error when I go to the repo in analyzing status (see screenshot). Also, the yellow shadow around analyzing looks weird. I think the message is obvious enough from the placement, loading icon, and font size. It would be nice to make the page take up the whole container width since it looks strange to have a blank space over there where the sidebar would go.
Changes on forge and sftheme: