Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Get the static UI working again #105

Merged
merged 2 commits into from
May 5, 2014
Merged

Get the static UI working again #105

merged 2 commits into from
May 5, 2014

Conversation

dborkan
Copy link
Contributor

@dborkan dborkan commented May 2, 2014

Get the static UI working again:

  • Update uistatic dependencies to add 'core' object.
  • Updated MockCore to contain status: {connected: true} and fix constructor method name
  • Moved StatusObject interface to uproxy.ts so both extension and uistatic can use it
  • Removed 'appChannel' that was registered with angular - this has been replaced by 'core'
  • Add "networks" field to uistatic's state. We will fully sync the state in uistatic to match a real valid extension state after the extension state is redefined.

Tested by running uistatic/view.html, existing app/extension interaction, and jasmine tests.

},
"networks": {
"google": {"name": "google", "online": true,"roster": {}},
"websocket":{"name": "websocket", "online": false,"roster": {}}
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: space after "websocket": for consistency

@keroserene
Copy link
Collaborator

One small nit, otherwise LGTM :)

Add space after "websocket":
dborkan added a commit that referenced this pull request May 5, 2014
@dborkan dborkan merged commit a31d80e into dev May 5, 2014
trevj pushed a commit that referenced this pull request Dec 7, 2016
Accept public key (-k option) for install-cloud scripts
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.

2 participants