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

Adds options menu to library #31

Merged
merged 4 commits into from
Oct 15, 2015
Merged

Adds options menu to library #31

merged 4 commits into from
Oct 15, 2015

Conversation

ehsanmasdar
Copy link
Member

@smo-key Please review and sign off.
@ethan-schaffer @Shwinn Also please code review.

Menu is now extensible to allow for different input types. Sample Categories included are:
Single select dropdown
Text input
Number only input
@Shwinn
Copy link
Contributor

Shwinn commented Oct 15, 2015

@Oksisane everything looks good to me after a quick scan

@ethan-schaffer
Copy link
Contributor

I trust you three. I am still a bit behind on the library. Now, if
something breaks, we know where the mistake probably is. ;-)

On Wed, Oct 14, 2015 at 11:23 PM, Shwinn [email protected] wrote:

@Oksisane https://github.com/Oksisane everything looks good to me after
a quick scan


Reply to this email directly or view it on GitHub
#31 (comment)
.

@smo-key
Copy link
Member

smo-key commented Oct 15, 2015

👍
@homu r+

@homu
Copy link
Contributor

homu commented Oct 15, 2015

📌 Commit efc64fc has been approved by smo-key

@homu
Copy link
Contributor

homu commented Oct 15, 2015

⌛ Testing commit efc64fc with merge 636740a...

homu added a commit that referenced this pull request Oct 15, 2015
Adds options menu to library

@smo-key Please review and sign off.
@ethan-schaffer @Shwinn Also please code review.
@homu
Copy link
Contributor

homu commented Oct 15, 2015

☀️ Test successful - status

@homu homu merged commit efc64fc into staging Oct 15, 2015
@ehsanmasdar ehsanmasdar deleted the feature-options branch October 15, 2015 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants