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

Frontend ui #4

Open
wants to merge 21 commits into
base: master
Choose a base branch
from
Open

Frontend ui #4

wants to merge 21 commits into from

Conversation

chushao
Copy link
Contributor

@chushao chushao commented Apr 18, 2013

Please Kthx.

# Establish a connection
@connection = nano([url, @query and "?" + @query].join(""))

_.extend Miso.Importers.Couchdb::,
Copy link
Member

Choose a reason for hiding this comment

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

might want to explicitly say .prototype just to be clear.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Actually I don't need to use these anymore ever since I switched from miso dataset to google

@ssampang
Copy link

image


esPort = 22
esUser = process.env['EC2_ES_SSH_USER']
esUser = cfg.EC2_ES_SSH_USER

stageDBLoadAvg = ""
Copy link
Member

Choose a reason for hiding this comment

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

stage:
  db:
    mem:
      total: ""
      free: ""
  load: ""

Also why initialize these as strings if we're going to assign floats to them later?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Careless mistake. Will fix when I get back.

@chushao
Copy link
Contributor Author

chushao commented Apr 18, 2013

Pretty much a shitton of copy paste code. I wanted to get it working first then i'll refactor it.

@jnwng
Copy link
Contributor

jnwng commented Apr 22, 2013

Address @jenrzzz's comments and let's get this in on Wednesday.

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