"Uma string vazia não é uma string JSON válida" ao atualizar o componente de tema

Bug
,
1

Ao atualizar um componente de tema, ocorre o seguinte erro.

Falha ao atualizar o Componente de Tema Brand Header
Uma string vazia não é uma string JSON válida.
/var/www/discourse/lib/theme_settings_manager/objects.rb:13:in `parse'
/var/www/discourse/lib/theme_settings_manager/objects.rb:13:in `value='
/var/www/discourse/app/models/theme.rb:760:in `update_setting'
/var/www/discourse/app/models/theme.rb:881:in `block (2 levels) in migrate_settings'
/var/www/discourse/app/models/theme.rb:880:in `each'
/var/www/discourse/app/models/theme.rb:880:in `block in migrate_settings'
/var/www/discourse/app/models/theme.rb:919:in `migrate_settings'
/var/www/discourse/app/models/remote_theme.rb:377:in `block in update_from_remote'
/var/www/discourse/vendor/bundle/ruby/3.3.0/gems/activerecord-7.1.3.4/lib/active_record/connection_adapters/abstract/transaction.rb:535:in `block in within_new_transaction'
/var/www/discourse/vendor/bundle/ruby/3.3.0/gems/activesupport-7.1.3.4/lib/active_support/concurrency/null_lock.rb:9:in `synchronize'
/var/www/discourse/vendor/bundle/ruby/3.3.0/gems/activerecord-7.1.3.4/lib/active_record/connection_adapters/abstract/transaction.rb:532:in `within_new_transaction'
/var/www/discourse/vendor/bundle/ruby/3.3.0/gems/activerecord-7.1.3.4/lib/active_record/connection_adapters/abstract/database_statements.rb:344:in `transaction'
/var/www/discourse/vendor/bundle/ruby/3.3.0/gems/activerecord-7.1.3.4/lib/active_record/transactions.rb:212:in `transaction'
/var/www/discourse/vendor/bundle/ruby/3.3.0/gems/activerecord-7.1.3.4/lib/active_record/transactions.rb:301:in `transaction'
/var/www/discourse/app/models/remote_theme.rb:383:in `update_from_remote'

Isso é sobre GitHub - discourse/discourse-brand-header: Brand header theme component for Discourse que está sendo atualizado de 5b0c2da para f458dc9 (porque o Discourse está sendo atualizado de 3.2.4 para 3.3.0).
O motivo pelo qual isso está acontecendo é que o campo “links” está vazio e a lógica de atualização aparentemente não consegue lidar com isso @tgxworld

Solução alternativa:

  • insira foo,https://example.com,bar no campo “links”
  • atualize o componente de tema
  • abra o editor de links e remova a entrada

No entanto, essa solução alternativa é difícil de automatizar… :sob:

4 curtidas
3

Vou dar uma olhada nisso hoje.

3 curtidas
4

Isso foi corrigido em

Provavelmente será mesclado e lançado nas próximas 24 horas.

4 curtidas
6

Este tópico foi automaticamente fechado após 3 dias. Novas respostas não são mais permitidas.