Skip to content
This repository has been archived by the owner on May 19, 2020. It is now read-only.

Individual pages #30

Merged
merged 33 commits into from Aug 5, 2015
Merged

Individual pages #30

merged 33 commits into from Aug 5, 2015

Conversation

geramirez
Copy link
Contributor

  • creating pretty searchable tables for spaces and orgs
  • setting up system for efficiently rendering individual pages for orgs.
    tldr - org data from the /v2/organizations endpoint is only loaded on entry, if prompted, or if an error occurs. This should keep costly api calls down especially when we create space/app/service pages.

@geramirez geramirez self-assigned this Jul 30, 2015
@@ -1,6 +1,7 @@
(function() {
// CloudFoundry Service
var cloudfoundry = function($http, $location) {
angular.module('cfconsole').service('$cloudfoundry', function($http, $location, $log) {
Copy link
Contributor

Choose a reason for hiding this comment

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

change this to cfdeck

@geramirez geramirez removed their assignment Aug 4, 2015
@@ -22,7 +22,7 @@
<div class="site-wrapper-inner">
<div class="container">
<!-- Where our view will load -->
<div ng-view></div>
<div ng-view ng-controller="DashboardCtrl"></div>
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are you putting this into the base file and not the partials/index.html?

@codecov-io
Copy link

Current coverage is 81.13%

Merging #30 into master will decrease coverage by -9.35% as of efffd15

@@            master     #30   diff @@
======================================
  Files            3       3       
  Stmts           42      53    +11
  Branches         3       3       
  Methods         26      34     +8
======================================
+ Hit             38      43     +5
  Partial          0       0       
- Missed           4      10     +6

Review entire Coverage Diff

Powered by Codecov

@geramirez
Copy link
Contributor Author

@jcscottiii This should be done for now. The coverage on the front end dropped a little but we can add the small tests later.

jcscottiii added a commit that referenced this pull request Aug 5, 2015
@jcscottiii jcscottiii merged commit e0dd426 into master Aug 5, 2015
@geramirez geramirez mentioned this pull request Aug 7, 2015
@jcscottiii jcscottiii deleted the individual-pages branch August 10, 2015 15:29
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.

None yet

3 participants