Skip to content

TEAMFOUR-144 - Implement the states and notifications#171

Merged
sean-sq-chen merged 8 commits into
masterfrom
app-details-notification
Apr 7, 2016
Merged

TEAMFOUR-144 - Implement the states and notifications#171
sean-sq-chen merged 8 commits into
masterfrom
app-details-notification

Conversation

@sean-sq-chen
Copy link
Copy Markdown
Contributor

JIRA: https://jira.hpcloud.net/browse/TEAMFOUR-144

This PR includes:

  • Maintain application states in model
  • Visualizing application state in application-action-bar
  • Disable/Enable application buttons accordingly based on it's state

Not included:

  • server side http request handlers in node server and mock in mocker server. Asumming we will be able to use real end-point soon.
  • unit test. Unit test will be in another PR
  • ete test. e2e tests are not possible without a real endpoint.

@ongk @tara-gibbs

…-ui into app-details-notification

* 'app-details-notification' of github.com:hpcloud/stratos-ui:
  Implementing app lifecycle notification

# Conflicts:
#	src/plugins/cloud-foundry/cloud-foundry.scss
#	src/plugins/cloud-foundry/view/applications/application/application.html
@WAT-TeamFour
Copy link
Copy Markdown

From Jenkins: There was a test failure while running Jenkins tests.

@sean-sq-chen sean-sq-chen changed the title TEAMFOUR-21 - Implement the states and notifications TEAMFOUR-144 - Implement the states and notifications Apr 6, 2016
<span class="btn-group">
<button class="btn btn-default btn-info"
translate
ng-disabled="appCtrl.model.appStateSwitchTo==='PENDING'"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I may not be familiar with the states yet, but should the user be able to restart a stopped app?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It should be "appCtrl.model.application.summary.state==='PENDING'"

When a request to change the application's state has been sent to server, the state of the app on client side is PENDING, during this time, all the buttons should be disabled until a response has been received.

@ongk
Copy link
Copy Markdown
Contributor

ongk commented Apr 7, 2016

+1 LGTM

@sean-sq-chen sean-sq-chen merged commit 5ba1c6c into master Apr 7, 2016
@sean-sq-chen sean-sq-chen deleted the app-details-notification branch April 7, 2016 20:28
irfanhabib added a commit that referenced this pull request Jun 1, 2017
richard-cox added a commit that referenced this pull request Jan 17, 2020
Allow database path to be set - support persistent DB with AIO
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.

4 participants