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

WIP: Webform #3

Open
wants to merge 2 commits into
base: gh-pages
Choose a base branch
from
Open

WIP: Webform #3

wants to merge 2 commits into from

Conversation

jakirkham
Copy link
Member

Provides a simple webform that one can fill out online and download a JSON config file from. To see a live demo, please follow this link.

@jakirkham jakirkham mentioned this pull request Jul 28, 2016
@jakirkham jakirkham force-pushed the gh-pages branch 5 times, most recently from 76b23b9 to e45e06e Compare July 28, 2016 21:01
@jakirkham
Copy link
Member Author

I'm pretty sure I'm still off on some things here, but maybe it is a good time for a review. Also, feel free to make comments on the diff if there are particular lines that are off or wrong. This actually makes it pretty easy to keep track of what has and hasn't been done.

@jakirkham
Copy link
Member Author

Any thoughts on this, @dudmanj? Would really love to check this one off the list.

@yttrie
Copy link

yttrie commented Jul 26, 2017

it'd be good to add a AP, ML, and Z value for the electrode, as well as type of electrode used (first 3 are floats, last one is string)

</label>
<br>
<label>Laser Power :
<input type="number" step="any" value=0 name="LaserPower" id="LaserPower" placeholder="LaserPower" />
Copy link
Member Author

Choose a reason for hiding this comment

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

Should specify this is milliwatts.

<br>
<label>Laser Power :
<input type="number" step="any" value=0 name="LaserPower" id="LaserPower" placeholder="LaserPower" />
</label>
Copy link
Member Author

Choose a reason for hiding this comment

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

Could be handy to have a notes section for miscellaneous info about how the recording is done.

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