#199 coreos-download.js: Added dropdown to display aarch64 artifacts and images
Merged by dustymabe. Opened by gursewak.
Unknown source master

Download 199.patch

Linked issue: https://github.com/coreos/fedora-coreos-tracker/issues/959

Added a dropdown to display aarch64 artifacts and images on coreos downloads page. The dropdown allows the user to select from either of the architecture and it updates the page with links to the appropriate images/artifacts. It takes and displays all the architectures under various streams(stable,testing and next).
Eg. https://builds.coreos.fedoraproject.org/streams/stable.json

Could you deploy a preview or take a couple of screenshots to help review the change? Thanks!

Let me know if this helps @siosm : https://www.youtube.com/watch?v=OOF8pj3TUlU

Looks great so far. Thanks @gursewak.

I have 1 suggestion and 1 separate change that I'm thinking is probably a requirement.

  • suggestion: maybe we should make it not right justified and just to the right of the text on the left hand side where is says "Currently displayed stream".
  • requirement: it would be AMAZING if we can link people directly to the page they need. For example we'd want to add a &architecture=aarch64 to the URL and have it show the page directly. Notice when you click around on the other options the URL changes to add these options. Then you can copy/paste to share with people the exact page you're on.

rebased onto cc5a32da22808b5941576c934c5a4242be388aad

If we add ppc64le architecture we'll need to update this code. Is there any way to make this code generic so we wouldn't be required to do that?

Minor/optional: maybe just arch for the URL parameter name?

Maybe by just checking if the value is in the list of arches in the stream metadata and if not default to x86_64?

I had suggested to him &architecture=, but I support changing it to &arch= if that's preferred.

rebased onto ee0ae5ce9fafad41a4dbcd9389b7689b9dace89e

Nice work @gursewak! LGTM

rebased onto 203cf5282a4ac743bcef4e0edd1c53ba556b8d7b

Pull-Request has been merged by dustymabe

Metadata