-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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: stock market service and info widget #3617
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, this is great overall and I think people will be happy with it. Especially like that its extensible.
A couple small things before I double-check the code is that as-is this will allow duplicates in the list (thats prob OK, people can do crazy things, then again it'd be easy enough to solve) but Im also wondering if the limit of 30 (which I know comes from the API) is too much. It just starts to get too large. Maybe thats OK but my gut tells me we should trim it down. What do you think?
I appreciate your feedback!
I will take care of this.
Agreed. A max of 6 kind of feels like the sweet spot. With a known max, I would also like to handle some viewport sizes a little differently. If it works, I can have these updates included tomorrow. If there is anything else that looks like it could be improved, I am happy to make the changes. |
Yea no problem, sounds good on all accounts. I’ll take a look at the code more closely after you push the changes but overall seems like pretty a clean PR thanks 👍 |
Yay! Love this! |
I could do both. The component looks like it is pretty easily adaptable: @shamoon What are your thoughts? Would it be better to pick one? Or would it be reasonable to have both component types as potential options? |
It's not clear from the FR if people want a service widget or info, so it's of course debatable. But given this is closest to the CoinMarketCap widget (which btw is better suited to the list format) perhaps that is the better way to go, sorry for the changeup. |
Happy to make the change. I agree that the format of the CoinMarketCap widget does work better overall. I can get started later today. |
I think both should be options. On the layout of the info widget, I think keeping the font size the same, and lowering the opacity of the price, would be more inline with the Homepage design direction. This would also allow it to use slightly less vertical space. |
For my personal use, I do prefer having a few items in the header in most instances.
I will look into these changes. Thanks for the tips. Apologies for the delay. I have been busy with work :) I was testing some other APIs to see if it makes sense to include them in the widget, but I think, at least for now, Finnhub seems like a better option with their free tier and limits. I also have tested getting data from I suggested this could be a |
I would just focus on this version, especially given it will now support two versions of the widget let’s keep the PR focused ie MVP. I’m confident this can be adapted for other APIs if there is need |
Looking good. Small thing, I wouldn’t bother with displaying (or translating) the errors or displaying things like has duplicates, it’s basically a config error so just throw an error with details in the log and / or truncate the list / remove the duplicates. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I havent tested yet, just some thoughts looking at the code. Thanks
Reviewing now. Thank you for these great suggestions. I will try and get them all added today. |
ran pre-commit hook improved translations and error handling include service widget fix: update docs image use original spacing in yaml file fix: typo in docs use Error component use regular error messaging here fix: make use of standard config error fix: map through data in jsx fix: simplify parsing/error logic
Mentioned issues should be resolved: https://github.com/gethomepage/homepage/compare/878d62b1f832f17bb57d18ac3b00ad70dec0c2df..ae3df9eeeb045f256bce33751d1d504ca96cf6cd I really appreciate the feedback :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it possible to add crypto? |
…0.9.5@e23fcd3 by renovate (#24087) This PR contains the following updates: | Package | Update | Change | |---|---|---| | [ghcr.io/gethomepage/homepage](https://togithub.com/gethomepage/homepage) | patch | `v0.9.2` -> `v0.9.5` | --- > [!WARNING] > Some dependencies could not be looked up. Check the Dependency Dashboard for more information. --- ### Release Notes <details> <summary>gethomepage/homepage (ghcr.io/gethomepage/homepage)</summary> ### [`v0.9.5`](https://togithub.com/gethomepage/homepage/releases/tag/v0.9.5) [Compare Source](https://togithub.com/gethomepage/homepage/compare/v0.9.4...v0.9.5) #### What's Changed - Fix: correct k8s service status check from v0.9.4 by [@​shamoon](https://togithub.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3753](https://togithub.com/gethomepage/homepage/pull/3753) - Enhancement: support pfsense v2 api by [@​shamoon](https://togithub.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3716](https://togithub.com/gethomepage/homepage/pull/3716) - Feature: Frigate service widget by [@​j0ton](https://togithub.com/j0ton) in [https://github.com/gethomepage/homepage/pull/3743](https://togithub.com/gethomepage/homepage/pull/3743) **Full Changelog**: gethomepage/homepage@v0.9.3...v0.9.5 ### [`v0.9.4`](https://togithub.com/gethomepage/homepage/compare/v0.9.3...v0.9.4) [Compare Source](https://togithub.com/gethomepage/homepage/compare/v0.9.3...v0.9.4) ### [`v0.9.3`](https://togithub.com/gethomepage/homepage/releases/tag/v0.9.3) [Compare Source](https://togithub.com/gethomepage/homepage/compare/v0.9.2...v0.9.3) #### What's Changed - Fix: dont force lowercase URL in QuickLaunch by [@​shamoon](https://togithub.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3634](https://togithub.com/gethomepage/homepage/pull/3634) - Fix: Improve error handling for Glances widgets by [@​mjsully](https://togithub.com/mjsully) and [@​shamoon](https://togithub.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3657](https://togithub.com/gethomepage/homepage/pull/3657) - Feature: MySpeed widget by [@​gnmyt](https://togithub.com/gnmyt) and [@​shamoon](https://togithub.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3662](https://togithub.com/gethomepage/homepage/pull/3662) - Fix: Sort truenas pools by name by [@​marktitorenkov](https://togithub.com/marktitorenkov) and [@​shamoon](https://togithub.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3669](https://togithub.com/gethomepage/homepage/pull/3669) - Fix: search dropdown overlay with boxedheaders option by [@​shamoon](https://togithub.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3680](https://togithub.com/gethomepage/homepage/pull/3680) - Feature: stock market service and info widget by [@​eldyl](https://togithub.com/eldyl) and [@​shamoon](https://togithub.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3617](https://togithub.com/gethomepage/homepage/pull/3617) #### New Contributors - [@​mjsully](https://togithub.com/mjsully) made their first contribution in [https://github.com/gethomepage/homepage/pull/3657](https://togithub.com/gethomepage/homepage/pull/3657) - [@​gnmyt](https://togithub.com/gnmyt) made their first contribution in [https://github.com/gethomepage/homepage/pull/3662](https://togithub.com/gethomepage/homepage/pull/3662) - [@​marktitorenkov](https://togithub.com/marktitorenkov) made their first contribution in [https://github.com/gethomepage/homepage/pull/3669](https://togithub.com/gethomepage/homepage/pull/3669) - [@​eldyl](https://togithub.com/eldyl) made their first contribution in [https://github.com/gethomepage/homepage/pull/3617](https://togithub.com/gethomepage/homepage/pull/3617) **Full Changelog**: gethomepage/homepage@v0.9.2...v0.9.3 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Enabled. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://togithub.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy40NDAuNiIsInVwZGF0ZWRJblZlciI6IjM3LjQ0MC42IiwidGFyZ2V0QnJhbmNoIjoibWFzdGVyIiwibGFiZWxzIjpbImF1dG9tZXJnZSIsInVwZGF0ZS9kb2NrZXIvZ2VuZXJhbC9ub24tbWFqb3IiXX0=-->
I appreciate you sharing this! I was not aware these symbols received proper values from the Finnhub API.
I will have some time within the next week to take a closer look at this. I hope you are well. |
Hi @eldyl this is working for me, but I'm not sure if I'm missing something for non-US stocks (syntax) or whether I must open a new issue for a feature. I'm looking to for example list a stock STX40 from Johannesburg stock exchange. Finnhub gives the exchange the code JO. But I tried adding STX40.JO and it gives an API error. |
After using this for a bit, I can't recommend the crypto tickers from finnhub unfortunately. It does work, and during the day the numbers are correct, but after the markets close (before 9:30am or after 4pm Eastern) the crypto prices get wacky. Sometimes it just throws an API error, sometimes it shows the "close" or "open" price of that day, sometimes it shows a price from 2-3 days ago. It doesn't seem to follow any pattern that I can decipher. I have noticed that the ticker values on finnhub's website still appear to be correct, so maybe it's just a problem with the API outside of standard trading hours. |
This PR contains the following updates: | Package | Update | Change | |---|---|---| | [ghcr.io/gethomepage/homepage](https://redirect.github.com/gethomepage/homepage) | patch | `v0.9.2` -> `v0.9.6` | --- ### Release Notes <details> <summary>gethomepage/homepage (ghcr.io/gethomepage/homepage)</summary> ### [`v0.9.6`](https://redirect.github.com/gethomepage/homepage/releases/tag/v0.9.6) [Compare Source](https://redirect.github.com/gethomepage/homepage/compare/v0.9.5...v0.9.6) #### What's Changed - Fix: wg-easy widget show o.filter is not a function when wg-easy api return error by [@​BugGeeker](https://redirect.github.com/BugGeeker) in [https://github.com/gethomepage/homepage/pull/3768](https://redirect.github.com/gethomepage/homepage/pull/3768) - Fix: correctly decode tab names with special chars by [@​shamoon](https://redirect.github.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3797](https://redirect.github.com/gethomepage/homepage/pull/3797) - Chore(deps): Bump systeminformation from 5.22.7 to 5.23.2 by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/gethomepage/homepage/pull/3799](https://redirect.github.com/gethomepage/homepage/pull/3799) - Chore(deps-dev): Bump eslint-plugin-prettier from 4.2.1 to 5.2.1 by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/gethomepage/homepage/pull/3802](https://redirect.github.com/gethomepage/homepage/pull/3802) - Chore(deps): Bump urbackup-server-api from 0.8.9 to 0.52.0 by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/gethomepage/homepage/pull/3801](https://redirect.github.com/gethomepage/homepage/pull/3801) - Chore(deps): Bump raw-body from 2.5.2 to 3.0.0 by [@​dependabot](https://redirect.github.com/dependabot) in [https://github.com/gethomepage/homepage/pull/3803](https://redirect.github.com/gethomepage/homepage/pull/3803) - Change: handle errored truenas pools by [@​shamoon](https://redirect.github.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3815](https://redirect.github.com/gethomepage/homepage/pull/3815) - Feature: Linkwarden service widget by [@​PyrokineticDarkElf](https://redirect.github.com/PyrokineticDarkElf) and [@​shamoon](https://redirect.github.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3836](https://redirect.github.com/gethomepage/homepage/pull/3836) - Enhancement: use tubearchivist stats endpoints by [@​shamoon](https://redirect.github.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3814](https://redirect.github.com/gethomepage/homepage/pull/3814) #### New Contributors - [@​BugGeeker](https://redirect.github.com/BugGeeker) made their first contribution in [https://github.com/gethomepage/homepage/pull/3768](https://redirect.github.com/gethomepage/homepage/pull/3768) - [@​PyrokineticDarkElf](https://redirect.github.com/PyrokineticDarkElf) made their first contribution in [https://github.com/gethomepage/homepage/pull/3836](https://redirect.github.com/gethomepage/homepage/pull/3836) **Full Changelog**: gethomepage/homepage@v0.9.5...v0.9.6 ### [`v0.9.5`](https://redirect.github.com/gethomepage/homepage/releases/tag/v0.9.5) [Compare Source](https://redirect.github.com/gethomepage/homepage/compare/v0.9.4...v0.9.5) #### What's Changed - Fix: correct k8s service status check from v0.9.4 by [@​shamoon](https://redirect.github.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3753](https://redirect.github.com/gethomepage/homepage/pull/3753) - Enhancement: support pfsense v2 api by [@​shamoon](https://redirect.github.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3716](https://redirect.github.com/gethomepage/homepage/pull/3716) - Feature: Frigate service widget by [@​j0ton](https://redirect.github.com/j0ton) in [https://github.com/gethomepage/homepage/pull/3743](https://redirect.github.com/gethomepage/homepage/pull/3743) **Full Changelog**: gethomepage/homepage@v0.9.3...v0.9.5 ### [`v0.9.4`](https://redirect.github.com/gethomepage/homepage/compare/v0.9.3...v0.9.4) [Compare Source](https://redirect.github.com/gethomepage/homepage/compare/v0.9.3...v0.9.4) ### [`v0.9.3`](https://redirect.github.com/gethomepage/homepage/releases/tag/v0.9.3) [Compare Source](https://redirect.github.com/gethomepage/homepage/compare/v0.9.2...v0.9.3) #### What's Changed - Fix: dont force lowercase URL in QuickLaunch by [@​shamoon](https://redirect.github.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3634](https://redirect.github.com/gethomepage/homepage/pull/3634) - Fix: Improve error handling for Glances widgets by [@​mjsully](https://redirect.github.com/mjsully) and [@​shamoon](https://redirect.github.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3657](https://redirect.github.com/gethomepage/homepage/pull/3657) - Feature: MySpeed widget by [@​gnmyt](https://redirect.github.com/gnmyt) and [@​shamoon](https://redirect.github.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3662](https://redirect.github.com/gethomepage/homepage/pull/3662) - Fix: Sort truenas pools by name by [@​marktitorenkov](https://redirect.github.com/marktitorenkov) and [@​shamoon](https://redirect.github.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3669](https://redirect.github.com/gethomepage/homepage/pull/3669) - Fix: search dropdown overlay with boxedheaders option by [@​shamoon](https://redirect.github.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3680](https://redirect.github.com/gethomepage/homepage/pull/3680) - Feature: stock market service and info widget by [@​eldyl](https://redirect.github.com/eldyl) and [@​shamoon](https://redirect.github.com/shamoon) in [https://github.com/gethomepage/homepage/pull/3617](https://redirect.github.com/gethomepage/homepage/pull/3617) #### New Contributors - [@​mjsully](https://redirect.github.com/mjsully) made their first contribution in [https://github.com/gethomepage/homepage/pull/3657](https://redirect.github.com/gethomepage/homepage/pull/3657) - [@​gnmyt](https://redirect.github.com/gnmyt) made their first contribution in [https://github.com/gethomepage/homepage/pull/3662](https://redirect.github.com/gethomepage/homepage/pull/3662) - [@​marktitorenkov](https://redirect.github.com/marktitorenkov) made their first contribution in [https://github.com/gethomepage/homepage/pull/3669](https://redirect.github.com/gethomepage/homepage/pull/3669) - [@​eldyl](https://redirect.github.com/eldyl) made their first contribution in [https://github.com/gethomepage/homepage/pull/3617](https://redirect.github.com/gethomepage/homepage/pull/3617) **Full Changelog**: gethomepage/homepage@v0.9.2...v0.9.3 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR was generated by [Mend Renovate](https://mend.io/renovate/). View the [repository job log](https://developer.mend.io/github/jamesdkelly88/db-lab). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzOC41OS4yIiwidXBkYXRlZEluVmVyIjoiMzguNTkuMiIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOltdfQ==--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
--------- Co-authored-by: shamoon <[email protected]>
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion for related concerns. See our contributing guidelines for more details. |
Proposed change
Hello!
I have been working on an implementation of a stock market widget.
Relevant Feature Request - #297
Type of change
Checklist:
Notes
Desktop
Mobile