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

Bring in the UI work #702

Merged
merged 4 commits into from
Nov 4, 2016
Merged

Bring in the UI work #702

merged 4 commits into from
Nov 4, 2016

Conversation

kerneltime
Copy link
Contributor

PR for UI work from @marksoper.
To build the UI code run

 INCLUDE_UI=true make

Copy link
Contributor

@pdhamdhere pdhamdhere left a comment

Choose a reason for hiding this comment

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

Considering there is a kill switch, I am fine with merging this change.

"url" : "https://github.com/vmware/docker-volume-vsphere.git"
},
"author": {
"name": "Mark Soper",
Copy link
Contributor

Choose a reason for hiding this comment

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

We generally avoid individual's name in code/repository. I would suggest to replace with following;
name = SABU, VMware
email = cna-storage@vmware.com
url = https://github.com/vmware/docker-volume-vsphere

// Here when the app is loaded we're temporarily making
// a call to DvolVsanService to test connectivity
//
// TODO: remove this
Copy link
Contributor

Choose a reason for hiding this comment

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

I assume this will be removed at later point when UI is fully integrated with backend.

Copy link
Contributor

Choose a reason for hiding this comment

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

yes

@kerneltime
Copy link
Contributor Author

Merging this change in. From here on, UI changes should be incremental small PRs.

@kerneltime kerneltime merged commit 8b43502 into master Nov 4, 2016
@kerneltime kerneltime deleted the ui-work.msoper branch November 4, 2016 19:28
@kerneltime
Copy link
Contributor Author

Fixes #637

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants