[pve-devel] [PATCH manager] ui: dc: subscription: fix subscription check

Dominik Csapak d.csapak at proxmox.com
Wed Oct 19 12:15:54 CEST 2022


On 10/19/22 12:00, Aaron Lauterer wrote:
> With the change to the rust backend for the subscription check, the
> return value changed as well.
> 
> Signed-off-by: Aaron Lauterer <a.lauterer at proxmox.com>
> ---
> This is the simple fix. We could also change it to lower case first, if
> we expect that this might change again.
> 
> I don't think that PMG is affected by this, am not 100% sure.
> 
>   www/manager6/dc/Support.js | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/www/manager6/dc/Support.js b/www/manager6/dc/Support.js
> index 1b51e5ba..b8e9b2ff 100644
> --- a/www/manager6/dc/Support.js
> +++ b/www/manager6/dc/Support.js
> @@ -61,7 +61,7 @@ Ext.define('PVE.dc.Support', {
>   		},
>   		success: function(response, opts) {
>   		    let data = response.result.data;
> -		    if (data.status === 'Active') {
> +		    if (data.status === 'active') {
>   			if (data.level === 'c') {
>   			    me.updateCommunity(data);
>   			} else {

i'd rather fix it like we did the other places:

data.status.toLowerCase() === 'active'

to check both variants





More information about the pve-devel mailing list