Como adicionar links para as configurações do site na descrição das configurações do meu componente?

Na verdade, pensei que sabia porque traduzi algo como <a href="/admin/site_settings/category/all_results?filter=top_menu">configuração do site do menu superior</a> várias vezes, e funcionou :tada:.
Mas os testes falham porque links sem %{base_path} ou %{base_url} não são permitidos.

Então procurei um exemplo e encontrei

Eu copiei isso:

%{base_url}/admin/site_settings/category/all_results?filter=top_menu

Mas então o link não funcionou. Tornou-se https://example.discourse.group/%%7Bbase_url%7D/admin/site_settings/category/all_results?filter=top_menu

Quando instalei Homepage Feature, o link lá também não funcionou.

Então tentei adicionar uma barra final e usar %{base_url}, mas nenhum deles funcionou.

/%{base_url}/admin/site_settings/category/all_results?filter=top_menu leva a https://example.discourse.group/admin/customize/themes/%%7Bbase_url%7D/admin/site_settings/category/all_results?filter=top_menu

%{base_path}/admin/site_settings/category/all_results?filter=top_menu leva a https://example.discourse.group/admin/customize/themes/%%7Bbase_path%7Dadmin/site_settings/category/all_results?filter=top_menu

/%{base_path}/admin/site_settings/category/all_results?filter=top_menu leva a https://example.discourse.group/%%7Bbase_path%7D/admin/site_settings/category/all_results?filter=top_menu

Mas eu não consegui encontrar uma solução que funcionasse.

4 curtidas

Assim que este PR for mesclado, você poderá usar %{base_path} nas descrições das configurações do tema:

5 curtidas

A alteração principal foi mesclada e fiz um PR para corrigir o componente de tema “homepage feature”:

2 curtidas

Como os links relativos na descrição das configurações do meu componente estão funcionando atualmente, presumo que devo fixar a versão em compatibility para garantir que eles continuem funcionando para aqueles que não atualizam o Discourse ao mesmo tempo.
Imagino que você não fixou porque os links não funcionavam antes de qualquer forma — isso está correto?

Sim, isso está correto :+1:

1 curtida

Este tópico foi fechado automaticamente 30 dias após a última resposta. Novas respostas não são mais permitidas.