Skip to content

WD-17150 - aliases section in snap details#5356

Merged
alvaromateo merged 10 commits intomainfrom
WD-17150
Nov 12, 2025
Merged

WD-17150 - aliases section in snap details#5356
alvaromateo merged 10 commits intomainfrom
WD-17150

Conversation

@alvaromateo
Copy link
Copy Markdown
Contributor

@alvaromateo alvaromateo commented Sep 8, 2025

Done

Add aliases section to snap details.

Design

Who is this intended for?
These changes are meant to be for users of snap packages.

Why are we doing this?
We want to give users a first glance at what they are installing in their machine.
Snaps are self contained packages invoked on the CLI by their name; but sometimes the snap name is different than the command name, so the snap has some aliases defined by default. These are the commands that the user of the snap will have available on their system to invoke the snap after installing it and currently there's no visibility of it (probably it will be information available on the documentation page of the snap, but it's left as a responsibility of the publisher to specify it).

What is the benefit to the user, and/or experience? To the business?
This change will give more information to the user of snap packages, by listing all the commands that are added to their system when installing the snap they are looking at.

How will we know if it worked?
This is a Quality of Life addition. I'm not sure how can we measure success for this type of change.

How to QA

  • Example of Aliases section with 'Show more' button: nushell
  • Example of Aliases section without 'Show more' button: nitrate
  • Example of snap without Aliases (so no section): vault

Testing

  • This PR has tests
  • No testing required (explain why):

Issue / Card

Fixes WD-17150

@webteam-app
Copy link
Copy Markdown

@edisile edisile self-assigned this Sep 8, 2025
Copy link
Copy Markdown
Contributor

@edisile edisile left a comment

Choose a reason for hiding this comment

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

LGTM

@alvaromateo alvaromateo force-pushed the WD-17150 branch 2 times, most recently from bc2aae6 to edc0568 Compare September 15, 2025 12:23
@alvaromateo alvaromateo force-pushed the WD-17150 branch 3 times, most recently from 46f3eec to 97cd46a Compare October 13, 2025 10:17
@leiaru-c
Copy link
Copy Markdown

Would you be able to move the alias section between the source code and the report a bug section? In my mind this info is interesting, but maybe not as much as the things we already have, like the website and source code.
image

Other than that, I like the addition of the more descriptive title "command > alias", so looks good to me :)

@alvaromateo alvaromateo merged commit c3acdfc into main Nov 12, 2025
13 checks passed
@alvaromateo alvaromateo deleted the WD-17150 branch November 12, 2025 11:07
M7mdisk added a commit that referenced this pull request Nov 17, 2025
M7mdisk added a commit that referenced this pull request Nov 17, 2025
alvaromateo added a commit that referenced this pull request Nov 18, 2025
alvaromateo added a commit that referenced this pull request Nov 20, 2025
alvaromateo added a commit that referenced this pull request Nov 20, 2025
* Reapply "WD-17150 - aliases section in snap details (#5356)" (#5457)

This reverts commit 5775d22.

* WD-31253 - Add catch to avoid page failing if details errors
Add empty channel to avoid errors if 'latest/stable' has no revisions

* WD-31253 - Add tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants