github/Bose-SoundTouch
1
0
Fork 0
mirror of https://github.com/gesellix/Bose-SoundTouch.git synced 2026-08-24 14:47:23 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix/beautify the account overview

Browse Source
This commit is contained in:
Tobias Gesellchen
2026-04-23 22:08:54 +02:00
parent 522492177d
commit 066e381737
4 changed files with 28 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
pkg/models/models.go
+1
View File
@@ -755,6 +755,7 @@ type FullResponseSource struct {
Username string `json:"username" xml:"username"`
Account string `json:"account,omitempty" xml:"account,attr,omitempty"`
SourceLabel string `json:"source_label" xml:"-"`
ProviderLabel string `json:"provider_label,omitempty" xml:"-"`
}
// FullResponsePreset represents a preset specifically for the /full response.
pkg/service/constants/constants.go
+16
View File
@@ -265,6 +265,22 @@ func GetProviderName(providerID string) string {
return providerID
}
// GetProviderLabel returns the user-friendly label for a provider ID (e.g. "TuneIn Radio", "Spotify").
func GetProviderLabel(providerID string) string {
id, err := strconv.Atoi(providerID)
if err != nil {
return ""
}
for _, p := range StaticProviders {
if p.ID == id {
return p.Label
}
}
return ""
}
// GetProviders returns a list of known source provider names.
func GetProviders() []string {
var providers []string
pkg/service/handlers/handlers_account_mgmt.go
+1
View File
@@ -269,6 +269,7 @@ func mapToFullResponseSource(src *models.ConfiguredSource) models.FullResponseSo
UpdatedOn: src.UpdatedOn,
Account: src.SourceKey.Account,
SourceLabel: constants.GetSourceLabel(src.Type),
ProviderLabel: constants.GetProviderLabel(src.SourceProviderID),
SourceSettings: src.SourceSettings,
}
fs.Credential.Value = src.Secret
pkg/service/handlers/web/js/script.js
+10 -7
View File
@@ -707,8 +707,8 @@ async function fetchAccountDetails(accountId) {
itemName = p.name || (p.source ? (p.source.source_label || p.source.name || p.source.type) : "Unknown");
if (p.source) {
const s = p.source;
const name = s.source_label || s.source_name || s.name || s.type;
const account = (s.account && s.account !== s.username) ? ` [${s.account}]` : "";
const name = s.provider_label || s.display_name || s.source_name || s.name || s.type;
const account = (s.account && s.account !== s.username && s.account !== name) ? ` [${s.account}]` : "";
const finalName = name || s.type || "Unknown Source";
if (finalName) {
sourceLabel = `<br><small style="color: #666; font-size: 0.85em;">via ${finalName}${account}</small>`;
@@ -732,14 +732,17 @@ async function fetchAccountDetails(accountId) {
let sourceLabel = "";
if (r.source) {
const s = r.source;
const sName = s.source_label || s.source_name || s.name || s.type;
const account = (s.account && s.account !== s.username) ? ` [${s.account}]` : "";
const sName = s.provider_label || s.display_name || s.source_name || s.name || s.type;
const account = (s.account && s.account !== s.username && s.account !== sName) ? ` [${s.account}]` : "";
const finalSName = sName || s.type || "Unknown Source";
if (finalSName) {
sourceLabel = `<br><small style="color: #666; font-size: 0.9em;">via ${finalSName}${account}</small>`;
}
}
return `<li>${name}${sourceLabel} <br><small style="color:#888">${r.created_on ? new Date(r.created_on * 1000).toLocaleString() : 'N/A'}</small></li>`;
const dateRaw = r.last_played_at || r.created_on;
const dateObj = dateRaw ? (isNaN(Number(dateRaw)) ? new Date(dateRaw) : new Date(Number(dateRaw) * 1000)) : null;
const dateStr = dateObj ? dateObj.toLocaleString('sv-SE') : 'N/A'; // sv-SE produces YYYY-MM-DD HH:MM:SS with 24h time
return `<li>${name}${sourceLabel} <br><small style="color:#888">${dateStr}</small></li>`;
}).join("") : "<li>No recents</li>"}
</ul>
</div>
@@ -750,9 +753,9 @@ async function fetchAccountDetails(accountId) {
<h5 style="margin: 0 0 5px 0">Configured Sources</h5>
<div style="display: flex; flex-wrap: wrap; gap: 5px">
${device.sources ? device.sources.filter(s => (s.source_label || s.source_name || s.name || s.type)).map(s => {
const sourceName = s.source_label || s.source_name || s.name || s.type;
const sourceName = s.provider_label || s.display_name || s.source_name || s.name || s.type;
const usernameSuffix = (s.username && s.username !== "Local") ? ` (${s.username})` : "";
const accountSuffix = (s.account && s.account !== s.username) ? ` [${s.account}]` : "";
const accountSuffix = (s.account && s.account !== s.username && s.account !== sourceName) ? ` [${s.account}]` : "";
return `
<span style="background: #eefbff; color: #0056b3; border: 1px solid #b8daff; padding: 2px 8px; border-radius: 12px; font-size: 0.75em" title="Source Type: ${s.type}">
${sourceName}${usernameSuffix}${accountSuffix}