Skip to content

Conversation

@backportbot-libresign
Copy link

Backport of PR #6552

- Mover lógica de loadVisibleElements para após loadSigners()
- Criar método loadVisibleElements() separado para melhor organização
- Remover duplicação de lógica que tentava procurar arquivo em array vazio
- Usar metadata do arquivo principal diretamente

Fixes #1234

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
- Incluir visibleElements no provideInitialState para que apareçam no frontend
- Dados retornados pelo FileService agora são passados corretamente ao cliente

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
- Adicionar visibleElements ao estado padrão do documento
- Carregar visibleElements do initialState no método initFromState()
- Garantir que elementos visíveis apareçam corretamente na página

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
- Alterar importação de OCA\Libresign\Service\WorkerHealthService para OCA\Libresign\Service\Worker\WorkerHealthService
- Classe estava localizada no subnamespace Worker

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
- Adicionar teste com dataProvider para verificar se visibleElements é incluído quando showVisibleElements() é chamado
- Verificar que visibleElements não é incluído quando flag não está set
- Testes cobrem o comportamento documentado da funcionalidade

Signed-off-by: Vitor Mattos <1079143+vitormattos@users.noreply.github.com>
@github-project-automation github-project-automation bot moved this to 0. Needs triage in Roadmap Jan 23, 2026
@backportbot-libresign backportbot-libresign bot added this to the Next Patch (32) milestone Jan 23, 2026
@vitormattos vitormattos merged commit dc2cac9 into stable32 Jan 23, 2026
33 of 37 checks passed
@github-project-automation github-project-automation bot moved this from 0. Needs triage to 4. to release in Roadmap Jan 23, 2026
@vitormattos vitormattos deleted the backport/6552/stable32 branch January 23, 2026 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: 4. to release

Development

Successfully merging this pull request may close these issues.

2 participants