-
Notifications
You must be signed in to change notification settings - Fork 644
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
Symbols UI Fixes #6576
Symbols UI Fixes #6576
Conversation
@diverdan92 - please review the UI and strings. |
@shishirx34 -
Delete:
Otherwise, LGTM. |
public static bool IsLatestSymbolPackageAvailable(this Package package) | ||
{ | ||
var latestSymbolPackage = package.LatestSymbolPackage(); | ||
return latestSymbolPackage != null |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why it needs to be the latest and not getting the available package if any?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right it could be any available package, but given that we do not show Download symbols
link when an replacement snupkg fails validation(or if its validating) it kind of gets confusing when uploading the symbols. I agree the overall UI needs to be better around symbols view. I will create a tracking issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Created a tracking issue: https://github.com/NuGet/Engineering/issues/1797
Addresses #6559
UI changes:
.nupkg vs .snupkg
warning and metadata:
delete default to dropdown: