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

Feature: Stocks #289

Closed
davepl opened this issue May 28, 2023 · 7 comments
Closed

Feature: Stocks #289

davepl opened this issue May 28, 2023 · 7 comments

Comments

@davepl
Copy link
Contributor

davepl commented May 28, 2023

It would be nice to have an effect to show a stock price. Since we do/will soon support multiple copies of an effect with different parameters, I think the effect only needs to handle one stock at a time.

You'd need to sign up for a free-to-use stock ticker service and provide a way to set the API key, ala the Weather example.

@mggates39
Copy link
Contributor

This one looks like a good way to cut my teeth on the code.

@mggates39
Copy link
Contributor

I have a dev branch working on this now. Based it on Subscriber so I had the configuration parameter available to set the ticker code. Still need to work on the display layout and I would love a good library for SVG since that is what the Stock Services provides for ticker company icons.

@rbergen
Copy link
Collaborator

rbergen commented Jun 10, 2023

@mggates39 This is a heads-up that there's a "mild" overhaul of the SettingSpec on the way, aimed at significantly reducing duplication of specs-related data. That also impacts the way effects have to define and register their specific setting specs. If you'd like a preview you can take a look at PatternSubscribers.h in the respective branch in my fork.

I'll update this issue once the upcoming PR has been merged into the main tree.

@mggates39
Copy link
Contributor

Thank you. I will take a look at it this evening. I am at my eldest grandsons highschool graduation today.

@rbergen
Copy link
Collaborator

rbergen commented Jun 10, 2023

Congratulations on your grandson's graduation! That's obviously a very good reason to not look at this today. :)

@mggates39
Copy link
Contributor

mggates39 commented Jun 11, 2023

Thank you. We all had a good time.
Now, having looked at the code you indicated, I don't see any issues with my code. I like how you refactored the setting code. I will back port it as soon as I see it hit main.

@rbergen
Copy link
Collaborator

rbergen commented Jul 4, 2023

@mggates39 If you are still working on this - or planning to - please take note of issue #341, as I'm closing this issue in favor of that one. Dave's ideas about the effect have evolved a bit, and #341 reflects the current ideas on how it should work.

@rbergen rbergen closed this as completed Jul 4, 2023
@rbergen rbergen closed this as not planned Won't fix, can't repro, duplicate, stale Jul 4, 2023
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

No branches or pull requests

3 participants