Skip to content

Show name of the update site in the statbox, listed or unlisted - #429

Merged
hinerm merged 2 commits into
mainfrom
update-site-in-statbox
Oct 22, 2026
Merged

hinerm merged 2 commits into
mainfrom
update-site-in-statbox

Conversation

@gletort

@gletort gletort commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

If update-site or update-site-unlisted field are filled in the header, will head a component in the statbox to directly see the name (listed) or the url (unlisted) of the update site

@gletort gletort assigned tinevez and unassigned tinevez Jul 29, 2026
@hinerm

hinerm commented Oct 21, 2026

Copy link
Copy Markdown
Member

@gletort I tested this and when update-site is specified with a name, you get an incorrect relative url just to that name.

To me, the unlisted url is potentially helpful. Although, if someone is contributing a wiki page I'm not sure why they wouldn't also contribute their update site to the list.

For listed update sites, I'm less sure a link is useful, since a user wouldn't actually use that url when enabling the site in Fiji.

What do you think? What goals did you want to achieve with this change?

@hinerm
hinerm force-pushed the update-site-in-statbox branch from 7d7c27a to d6c2658 Compare October 21, 2026 21:34
@gletort

gletort commented Oct 22, 2026

Copy link
Copy Markdown
Collaborator Author

Thanks @hinerm for looking into this. My goal was only to get a direct information on how to install the plugin, so that people can directly see which update site to add, if it's listed or unlisted, without having to go through all the documentation. So I totally agree that there is no need for a link in the case of a listed site, just to have the name of this update site is enough!

Listed update sites show name only, while unlisted are urls with links.

Co-authored-by: Copilot <copilot@github.com>
@hinerm
hinerm merged commit e90abd3 into main Oct 22, 2026
2 checks passed
@hinerm
hinerm deleted the update-site-in-statbox branch October 22, 2026 18:57
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.

3 participants