Skip to content
This repository has been archived by the owner on Apr 15, 2019. It is now read-only.

Add routing functionality - Fixes #117 #158

Merged
merged 45 commits into from
Apr 28, 2017

Conversation

reyraa
Copy link
Contributor

@reyraa reyraa commented Apr 28, 2017

  • Add ui.router module for managing routings
  • Remove hard coded component from the templates
  • Create separate component for header
  • Add run method to manipulate the state of the application
  • Create Account service to supply client's account informations to components
  • Other code cleanups

Fixes #117

reyraa added 30 commits April 25, 2017 13:53
…scope of main component. - store user's data in rootScope as we won't pass through HTML tag
…fer to account related info through rootScope
@reyraa reyraa changed the title Add routing functionality Add routing functionality fixes 117 Apr 28, 2017
@reyraa reyraa added this to the Version 1.0.0 milestone Apr 28, 2017
@karmacoma karmacoma changed the title Add routing functionality fixes 117 Add routing functionality - Fixes #117 Apr 28, 2017
@coveralls
Copy link

Coverage Status

Coverage increased (+0.6%) to 85.949% when pulling f7128d5 on 117-add-routing-functionality into 09dd2c9 on development.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.6%) to 85.907% when pulling aa96947 on 117-add-routing-functionality into 3ca9112 on development.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.6%) to 85.907% when pulling 1bd7bb4 on 117-add-routing-functionality into 3ca9112 on development.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.6%) to 85.907% when pulling 5f39a4e on 117-add-routing-functionality into 3ca9112 on development.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.6%) to 85.928% when pulling 3620b2f on 117-add-routing-functionality into 6f61314 on development.

@karmacoma karmacoma merged commit 99b6c1c into development Apr 28, 2017
@karmacoma karmacoma deleted the 117-add-routing-functionality branch April 28, 2017 16:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants