[pve-devel] [PATCH V3 pve-manager 1/2] fix #2822: add lvm, lvmthin & zfs storage for all cluster nodes
Fiona Ebner
f.ebner at proxmox.com
Tue Aug 2 09:29:16 CEST 2022
Am 01.08.22 um 16:30 schrieb Stefan Hrdlicka:
> Hello :)
>
> On 7/27/22 12:19, Fiona Ebner wrote:
>> Am 19.07.22 um 13:57 schrieb Stefan Hrdlicka:
>>> @@ -42,7 +45,7 @@ Ext.define('PVE.storage.IScsiScan', {
>>> fields: ['target', 'portal'],
>>> proxy: {
>>> type: 'proxmox',
>>> - url: `/api2/json/nodes/${me.nodename}/scan/iscsi`,
>>> + url: me.apiBaseUrl + me.nodename + me.apiStoragePath,
>>
>> Style nit: please keep using template string syntax
>> Same for the other ones below
>
> Exactly this one:
> url: `/api2/json/nodes/${me.nodename}/scan/iscsi` or do you mean or
> would something like this be ok as well:
> url: `${me.apiBaseUrl}${me.nodename}${me.apiSuffix}`
>
The second one please ;)
The change itself is fine, but the template string style is preferred.
In this case, it's not that big of a deal, but when mixing with actual
string literals, it's just nicer to read. And in other rare cases it can
prevent accidental numerical addition.
More information about the pve-devel
mailing list