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

Enhancement: extend hdhomerun widget #2757

Merged

Conversation

JeffRandall
Copy link
Contributor

@JeffRandall JeffRandall commented Jan 26, 2024

Proposed change

  • maintain the previous behavior if no new configuration is done
  • define additional available fields and present the data in order configured
    • global: "tunerCount"
    • per-tuner: "channelNumber", "channelNetwork", "signalStrength", "signalQuality", "symbolQuality", "networkRate", "clientIP"
  • add handling for refreshInterval
  • update docs

An example of two instances of the widget with the fields configured to be fields: [ "channels", "hd", "tunerCount" ] and fields: [ "clientIP", "channelNumber", "channelNetwork", "signalQuality" ] respectively:

image

Closes #772

Type of change

  • New service widget
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation only
  • Other (please explain)

Checklist:

  • If applicable, I have added corresponding documentation changes.
  • If applicable, I have reviewed the feature and / or service widget guidelines.
  • I have checked that all code style checks pass using pre-commit hooks and linting checks.
  • If applicable, I have tested my code for new features & regressions on both mobile & desktop devices, using the latest version of major browsers.

 - maintain the previous behavior if no additional configuration is done
 - define additional available fields and present the data in order configured
   - global: "tunerCount"
   - per-tuner: "channelNumber", "channelNetwork", "signalStrength", "signalQuality", "symbolQuality", "networkRate", "clientIP"
 - add handling for refreshInterval
 - update docs
@shamoon shamoon changed the title discussion #772: Extend the hdhomerun widget Enhancement: extend hdhomerun widget Jan 26, 2024
Copy link
Collaborator

@shamoon shamoon left a comment

Choose a reason for hiding this comment

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

Thanks, this needs a bit of work.

Please also provide sample data for status.json

docs/widgets/services/hdhomerun.md Outdated Show resolved Hide resolved
docs/widgets/services/hdhomerun.md Outdated Show resolved Hide resolved
src/widgets/hdhomerun/component.jsx Outdated Show resolved Hide resolved
src/widgets/hdhomerun/component.jsx Outdated Show resolved Hide resolved
src/widgets/hdhomerun/component.jsx Outdated Show resolved Hide resolved
public/locales/en/common.json Outdated Show resolved Hide resolved
docs/widgets/services/hdhomerun.md Outdated Show resolved Hide resolved
@JeffRandall
Copy link
Contributor Author

Sample status.json from a two tuner hdhomerun:

[ { "Resource": "tuner0", "VctNumber": "15.3", "VctName": "TBD", "Frequency": 581000000, "SignalStrengthPercent": 100, "SignalQualityPercent": 77, "SymbolQualityPercent": 100, "TargetIP": "10.11.2.10", "NetworkRate": 1764320 }, { "Resource": "tuner1" } ]

Copy link
Collaborator

@shamoon shamoon left a comment

Choose a reason for hiding this comment

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

Thanks for the changes, almost there =)

src/widgets/hdhomerun/component.jsx Outdated Show resolved Hide resolved
docs/widgets/services/hdhomerun.md Outdated Show resolved Hide resolved
src/widgets/hdhomerun/component.jsx Outdated Show resolved Hide resolved
src/widgets/hdhomerun/component.jsx Outdated Show resolved Hide resolved
src/widgets/hdhomerun/component.jsx Outdated Show resolved Hide resolved
src/widgets/hdhomerun/component.jsx Outdated Show resolved Hide resolved
@shamoon shamoon enabled auto-merge (squash) January 26, 2024 18:54
Copy link
Collaborator

@shamoon shamoon left a comment

Choose a reason for hiding this comment

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

Great thanks for making all the changes

Screenshot 2024-01-26 at 10 54 27 AM

@shamoon shamoon merged commit 05a7c0f into gethomepage:main Jan 26, 2024
5 checks passed
discretizer pushed a commit to discretizer/homepage that referenced this pull request Jan 28, 2024
truecharts-admin added a commit to truecharts/charts that referenced this pull request Jan 29, 2024
…0.8.7@06203a4 by renovate (#17670)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
|
[ghcr.io/gethomepage/homepage](https://togithub.com/gethomepage/homepage)
| patch | `v0.8.6` -> `v0.8.7` |

---

> [!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.8.7`](https://togithub.com/gethomepage/homepage/releases/tag/v0.8.7)

[Compare
Source](https://togithub.com/gethomepage/homepage/compare/v0.8.6...v0.8.7)

#### What's Changed

- Feature: Added gatus uptime widget by
[@&#8203;Nedra1998](https://togithub.com/Nedra1998) in
[gethomepage/homepage#2729
- Fix: iCal integration use event data hash instead of unstable uid by
[@&#8203;shamoon](https://togithub.com/shamoon) in
[gethomepage/homepage#2758
- Enhancement: extend hdhomerun widget by
[@&#8203;JeffRandall](https://togithub.com/JeffRandall) and
[@&#8203;shamoon](https://togithub.com/shamoon) in
[gethomepage/homepage#2757

#### New Contributors

- [@&#8203;Nedra1998](https://togithub.com/Nedra1998) made their first
contribution in
[gethomepage/homepage#2729
- [@&#8203;JeffRandall](https://togithub.com/JeffRandall) made their
first contribution in
[gethomepage/homepage#2757

**Full Changelog**:
gethomepage/homepage@v0.8.6...v0.8.7

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "before 10pm on monday" in timezone
Europe/Amsterdam, 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:eyJjcmVhdGVkSW5WZXIiOiIzNy4xNTMuMiIsInVwZGF0ZWRJblZlciI6IjM3LjE1My4yIiwidGFyZ2V0QnJhbmNoIjoibWFzdGVyIn0=-->
discretizer pushed a commit to discretizer/charts that referenced this pull request Jan 29, 2024
…0.8.7@06203a4 by renovate (truecharts#17670)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
|
[ghcr.io/gethomepage/homepage](https://togithub.com/gethomepage/homepage)
| patch | `v0.8.6` -> `v0.8.7` |

---

> [!WARNING]
> Some dependencies could not be looked up. Check the Dependency
Dashboard for more information.

---

<details>
<summary>gethomepage/homepage (ghcr.io/gethomepage/homepage)</summary>

[`v0.8.7`](https://togithub.com/gethomepage/homepage/releases/tag/v0.8.7)

[Compare
Source](https://togithub.com/gethomepage/homepage/compare/v0.8.6...v0.8.7)

- Feature: Added gatus uptime widget by
[@&truecharts#8203;Nedra1998](https://togithub.com/Nedra1998) in
[gethomepage/homepage#2729
- Fix: iCal integration use event data hash instead of unstable uid by
[@&truecharts#8203;shamoon](https://togithub.com/shamoon) in
[gethomepage/homepage#2758
- Enhancement: extend hdhomerun widget by
[@&truecharts#8203;JeffRandall](https://togithub.com/JeffRandall) and
[@&truecharts#8203;shamoon](https://togithub.com/shamoon) in
[gethomepage/homepage#2757

- [@&truecharts#8203;Nedra1998](https://togithub.com/Nedra1998) made their first
contribution in
[gethomepage/homepage#2729
- [@&truecharts#8203;JeffRandall](https://togithub.com/JeffRandall) made their
first contribution in
[gethomepage/homepage#2757

**Full Changelog**:
gethomepage/homepage@v0.8.6...v0.8.7

</details>

---

📅 **Schedule**: Branch creation - "before 10pm on monday" in timezone
Europe/Amsterdam, 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:eyJjcmVhdGVkSW5WZXIiOiIzNy4xNTMuMiIsInVwZGF0ZWRJblZlciI6IjM3LjE1My4yIiwidGFyZ2V0QnJhbmNoIjoibWFzdGVyIn0=-->
GabrielBarzen pushed a commit to GabrielBarzen/charts that referenced this pull request Feb 2, 2024
…0.8.7@06203a4 by renovate (truecharts#17670)

This PR contains the following updates:

| Package | Update | Change |
|---|---|---|
|
[ghcr.io/gethomepage/homepage](https://togithub.com/gethomepage/homepage)
| patch | `v0.8.6` -> `v0.8.7` |

---

> [!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.8.7`](https://togithub.com/gethomepage/homepage/releases/tag/v0.8.7)

[Compare
Source](https://togithub.com/gethomepage/homepage/compare/v0.8.6...v0.8.7)

#### What's Changed

- Feature: Added gatus uptime widget by
[@&truecharts#8203;Nedra1998](https://togithub.com/Nedra1998) in
[gethomepage/homepage#2729
- Fix: iCal integration use event data hash instead of unstable uid by
[@&truecharts#8203;shamoon](https://togithub.com/shamoon) in
[gethomepage/homepage#2758
- Enhancement: extend hdhomerun widget by
[@&truecharts#8203;JeffRandall](https://togithub.com/JeffRandall) and
[@&truecharts#8203;shamoon](https://togithub.com/shamoon) in
[gethomepage/homepage#2757

#### New Contributors

- [@&truecharts#8203;Nedra1998](https://togithub.com/Nedra1998) made their first
contribution in
[gethomepage/homepage#2729
- [@&truecharts#8203;JeffRandall](https://togithub.com/JeffRandall) made their
first contribution in
[gethomepage/homepage#2757

**Full Changelog**:
gethomepage/homepage@v0.8.6...v0.8.7

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "before 10pm on monday" in timezone
Europe/Amsterdam, 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:eyJjcmVhdGVkSW5WZXIiOiIzNy4xNTMuMiIsInVwZGF0ZWRJblZlciI6IjM3LjE1My4yIiwidGFyZ2V0QnJhbmNoIjoibWFzdGVyIn0=-->
discretizer pushed a commit to discretizer/homepage that referenced this pull request Feb 6, 2024
discretizer pushed a commit to discretizer/homepage that referenced this pull request Feb 6, 2024
Plancke pushed a commit to Plancke/homepage that referenced this pull request Feb 23, 2024
Plancke pushed a commit to Plancke/homepage that referenced this pull request Feb 23, 2024
Copy link
Contributor

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants