Verify package with language, devdep & deps

[robertmuehsig]

New PR with just the VerifyPackage enhancements (instead of #2242) - just in read only mode.

The Verify Package Page now shows also:

• Language
• Minimum Client Version
• Dev Dependeny
• FrameworkAssemblies
• Other Dependencies

Sample 1:

Sample 2 - with DevDependency:

Sample 3:

Problems:
The "PackageMetadata" doesn't read the MinimumClientVersion at all and a real DevelopmentDependency property is missing on the class. I could get the DevelopmentDependency Value via the key, but maybe a real property would be cleaner.
If the DevelopmentDependency property is not found the default "not specified" appears, but this would be easy to change.

I noticed that the frameworkassembly/dependency stuff has changed - I just saw the visualization of the dependencies on the details page, maybe I should adapt this kind of style, right?

What do you think?

[robertmuehsig]

uh - wow. That was fast 👍
Is the "packageMetadata.MinClientVersion" still a thing? What about the missing devDependency property (like packageMetadata.DevelopmentDependency)?

[maartenba]

Not currently on the server radar (but something we may get on the radar at one point)

Thanks for the PR! Merged with the suggested UI edits, will go live on next deploy.

[robertmuehsig]

Good to hear!
If MinClientVersion is always null then there is no need to display it - it might even confuse people if the value is set in the nuspec but not displayed correctly? Or is there a way I could help to fix this "broken" behavior?

[maartenba]

Just pushed a fix for that on dev.

[304NotModified]

woohoo! nice this PR! \0/