Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated Umbraco marketplace listing #167

Conversation

AndyButland
Copy link
Contributor

I noted you'd listed this package on the Umbraco marketplace, so firstly, thanks for taking the time to do that. I noticed though that we only have the details collected from NuGet for the package... which is fine, but to help it to be found more easily it's possible to provide additional information, as is discussed here.

To get started with this I've created this PR, which will at least ensure your package is listed under what looks to be the appropriate category. I've also made use of the package nesting feature we have, so the "sub-packages" can be listed on the page of the main package, and only the main one appears in the package listing from the category links or search. Up to you if you prefer that, but I think it's a cleaner listing in this way.

You can of course add more information, like tags, links, developer information, or extended descriptions and readme information. I hope that's useful for you.

@patrickdemooij9
Copy link
Owner

Thank you for the PR! Definitely have not had the time to add it myself yet, so this is very appreciated. I do wonder about the sub-package part. All the packages themselves are able to be run independently so having them hidden would be a bit of a shame. But dunno if they are any guidelines or rules on whether they would be considered sub-packages or not.

@AndyButland
Copy link
Contributor Author

It's your call - if you prefer to list them separately you can. Was just thinking from the perspective of someone searching in the category, is there value in seeing 5 entries or 1. But if you think they make sense separately, that's fine, and feel free to revert that amend.

@patrickdemooij9
Copy link
Owner

Hmm, difficult decision.. I will just leave it like this for now. I can always change it later on

@patrickdemooij9 patrickdemooij9 merged commit f2ca903 into patrickdemooij9:dev/Umbraco11 Mar 6, 2023
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.

2 participants