Am 05.06.23 um 19:06 schrieb Thomas Lamprecht:
> Am 05/06/2023 um 17:43 schrieb Fiona Ebner:
>> This will avoid breaking older UI when extending the backend.
>>
>> Signed-off-by: Fiona Ebner <f.eb...@proxmox.com>
>> ---
>>  src/node/APTRepositories.js | 2 --
>>  1 file changed, 2 deletions(-)
>>
>> diff --git a/src/node/APTRepositories.js b/src/node/APTRepositories.js
>> index cb08bb6..ed58e5b 100644
>> --- a/src/node/APTRepositories.js
>> +++ b/src/node/APTRepositories.js
>> @@ -660,8 +660,6 @@ Ext.define('Proxmox.node.APTRepositories', {
>>                      if (!repoGrid.majorUpgradeAllowed) {
>>                          infos[path][idx].warnings.push(info);
>>                      }
>> -                } else {
>> -                    throw 'unknown info';
> 
> otherwise we could use some Ext.Msg or at least log via console.warn (maybe 
> it helps detecting
> a missing thing someday), would both not break UI.
> 

Yes, console.warn might be the best, because the message is indicating
an internal error and not very relevant to users.


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

Reply via email to