Skip to content

Conversation

@parulin
Copy link
Contributor

@parulin parulin commented Nov 27, 2025

Update release notes and schedule links from PR QubesOS/qubes-doc#1504
to the latest branch.

Note: I don't understand the purpose of the "Upgrading to Qubes R4.3" link.

Fixes: QubesOS/qubes-issues#10428

@parulin
Copy link
Contributor Author

parulin commented Nov 27, 2025

I don't understand why the checks have failed.

@andrewdavidwong
Copy link
Member

Suggestion: If you intend for this pull request to resolve the associated issue and would like for it to be linked to the issue automatically, you can put Closes QubesOS/qubes-issues#XXXX in the body of the PR message (where XXXX is the issue number). Closes is one of several special keywords that GitHub will recognize. If and when the PR is merged, GitHub will automatically close the associated issue.

@ben-grande
Copy link
Contributor

I recommend using a commit title that survives on itself, without requiring to access external resources such as sites. Example:

Fix broken links on download page

Fixes: https://github.com/QubesOS/qubes-issues/issues/10428

Update release notes and schedule links from PR QubesOS/qubes-doc#1504
to the latest branch.

Fixes: QubesOS/qubes-issues#10428
@parulin parulin changed the title Fix QubesOS/qubes-issues#10428 links Fix R4.3 release links on download page Dec 3, 2025
@parulin
Copy link
Contributor Author

parulin commented Dec 3, 2025

Sorry for the mess.

@andrewdavidwong
Copy link
Member

Note: I don't understand the purpose of the "Upgrading to Qubes R4.3" link.

I believe it's for current 4.2 users who want to upgrade to 4.3.

Sorry for the mess.

Not at all! Thank you for contributing!

@parulin
Copy link
Contributor Author

parulin commented Dec 4, 2025

I believe it's for current 4.2 users who want to upgrade to 4.3.

I don't see that link in the list:

  • How to verify downloads
  • Installation guide
  • Release notes
  • Release schedule
  • All download mirrors

@marmarek
Copy link
Member

marmarek commented Dec 4, 2025

Interesting, while the yaml page has list of title + URL, the html version (_includes/downloads.html) expects specific titles (more or less making it pointless structure):

          {% for docdata in release.docs %}
            {% assign doc_name = docdata[0] %}
            {% assign doc = docdata[1] %}
            {% if doc_name == "Installation guide" %}
              <li><a href="{{ lang }}{{ doc.url }}" class="black-link" title="Installation guide">{{ doc_name }}</a></li>
            {% endif %}
            {% if doc_name == "Release notes" %}
              <li><a href="{{ lang }}{{ doc.url }}" class="black-link" title="Release notes">{{ doc_name }}</a></li>
            {% endif %}
            {% if doc_name == "Release schedule" %}
              <li><a href="{{ lang }}{{ doc.url }}" class="black-link" title="Release schedule">{{ doc_name }}</a></li>
            {% endif %}
          {% endfor %}

IMO it should just use doc_name, instead of this list of "if"

parulin added a commit to parulin/qubesos.github.io that referenced this pull request Dec 4, 2025
After another discussion (QubesOS#273), I did the
following:

* update some links from the download page, that were pointing to
  documentation URLs on www.qubes-os.org instead of doc.qubes-os.org
* remove the filtering of the download links based on their name,
  allowing "Upgrading to Qubes R[...]" links to be displayed
* remove duplicate title attribute on links, because I don't think it
  is good for accessibility
@parulin
Copy link
Contributor Author

parulin commented Dec 4, 2025

Thanks for the tip !

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.

qubes download page: broken links

4 participants