Skip to content

Conversation

@Nathan219
Copy link
Member

Now runnable.io/username will now be responsible for choosing which instance to go to (last loaded), or if none, /new
This is where everything should redirect to when deciding on an instance to go to.

… and even using the local storage to record the last instance.
Fixing Tests,
Creating service for fetching instances
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are we sure "Home" is the name we want to use here? This is a smart redirect.

@SomeKittens
Copy link
Contributor

I'd also like some tests on controllerInstanceHome and serviceFetchInstances

Make sure you merge in master for the latest coverage limits

Fixing some things that had issues, found during testing
Conflicts:
	test/unit/directives/directiveEditRepoCommit.unit.js
SomeKittens added a commit that referenced this pull request Dec 31, 2014
@SomeKittens SomeKittens merged commit 9ab5855 into master Dec 31, 2014
@SomeKittens SomeKittens deleted the Fix-instance-controller-ui-routes branch December 31, 2014 00:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants