Follow-up from "Package icon"
The following discussion from !847 should be addressed:
-
@bochecha started a discussion: (+2 comments) Note: This is meant for the previous line, but it seems I can't add multiple comments on the same line of the diff
I guess the
name
is used for the file name on the disk? What if the package id contains illegal characters, like/
? (I realize that this is probably a much bigger problem, which could impact us in other parts of the code, I'd be fine with punting to another merge request if you prefer)