On 04.10.21 12:25, Aaron Lauterer wrote:
> This will hopefully help people to find the needed storage as not
> everyone will be aware of the fact that CIFS is also a name used for it
> (technically it was only SMB v1).

> 
> Signed-off-by: Aaron Lauterer <a.laute...@proxmox.com>
> ---
>  www/manager6/Utils.js | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
>

applied both patches, thanks!

As they do one thing, i.e., switching "CIFS" to "SMB/CIFS" in the ui it would
be fine, or rather even better, in one patch. But, as people rather tend to 
throw
everything together and squashing is easier than splitting on review take it not
as direct/big complaint ;-)


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to