cedaro / satispress

Expose installed WordPress plugins and themes as Composer packages.
500 stars 48 forks source link

Add package name to SatisPress list #191

Closed tyrann0us closed 1 year ago

tyrann0us commented 1 year ago

Please check if the PR fulfills these requirements

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...) Feature

What is the current behavior? (You can also link to an open issue here) The list of enabled packages ("Settings" -> "SatisPress" -> tab "Repository") includes the Composer name, description, author, package type, etc. However, it does not include the package name. This sometimes makes it hard to identify a package. Example: A plugin called "WPO365 | SYNC" has Composer name satispress/wpo365-login-premium, another one is called "WPO365 | LOGIN PLUS", Composer name wpo365-login-professional. Without looking at the code, you won't know which plugin is what.

What is the new behavior (if this is a feature change)? The package name is printed below the description, above the homepage.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?) No.

Other information: The name property was already there but unused. I haven't tested the change locally because I did it on the GitHub UI only.

tyrann0us commented 1 year ago

Hi @bradyvercher, sorry for being pushy, but is this something you'd consider adding? Thanks!

bradyvercher commented 1 year ago

@tyrann0us No problem, thanks for the PR and your patience!