Skip to content

Commit

Permalink
After cloning new project, go to project when done (#1810)
Browse files Browse the repository at this point in the history
The "Clone new project" page was setting a locationChange event handler to cancel the clone-status timer. However, since the clone-status page counts as a different locaiton, the event was firing as soon as the user pressed the "Get Started" button, cancelling the clone-status timer too soon so the "cloning new project" page was never detecting that the clone completed.

By simply removing the locationChange event handler, we can let the clone status proceed. It changes from "Getting initial data" to "syncing" pretty soon, but that's fine. Once the status goes to idle, the existing code already cancels the timer and loads the project page, so there's no need for any further changes.
  • Loading branch information
rmunn authored May 7, 2024
1 parent 160bdcf commit 80ade38
Showing 1 changed file with 0 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,6 @@ export class LexiconNewProjectController implements angular.IController {
this.sendReceive.clearState();
this.sendReceive.setCloneProjectStatusSuccessCallback(this.gotoEditor);
this.sendReceive.setCloneProjectStatusFailedCallback(this.cloneFailed);
this.$scope.$on('$locationChangeStart', this.sendReceive.cancelCloneStatusTimer);
}

// ----- Step 2: Send Receive Clone -----
Expand Down

0 comments on commit 80ade38

Please sign in to comment.