Skip to content

Conversation

@aaldrich
Copy link

Added delta to the counter widget to support the new ducksboard changes

@jhsu
Copy link
Owner

jhsu commented Mar 18, 2013

awesome! I'll try and take a look at this later tonight.

@jhsu
Copy link
Owner

jhsu commented Mar 18, 2013

@aaldrich I'm thinking about simplifying the push api to a single class Ducksboard::Slot and see some examples

What do you think?

@aaldrich
Copy link
Author

What is your reasoning for it? I kinda like being able to extend new
widgets via polymorphism. I guess my biggest concern would be that the push
class might become very large and bloated. I could be wrong but that is my
initial thoughts. What do you think?

On Mon, Mar 18, 2013 at 2:00 AM, Joseph Hsu notifications@github.comwrote:

@aaldrich https://github.com/aaldrich I'm thinking about simplifying
the push api to a single class Ducksboard::Slothttps://github.com/jhsu/ducksboard/blob/simple-slot-push/lib/ducksboard/slot.rband see
some exampleshttps://github.com/jhsu/ducksboard/tree/simple-slot-push#sending-data

What do you think?


Reply to this email directly or view it on GitHubhttps://github.com//pull/7#issuecomment-15041587
.

@aaldrich
Copy link
Author

aaldrich commented Apr 1, 2013

I've started trying the Slot example just to see how I like it and having some issues. It seems like the post method inside the push api isn't being recognized on the class:

push.rb:45:in post': undefined methodpost' for Ducksboard::Slot:Class (NoMethodError)

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