rhandrade / tray-theme

CLI criado para ajudar desenvolvedores a criarem ótimos temas para Tray.
MIT License
20 stars 3 forks source link

Criar flag para mostrar temas excluídos no comando "tray themes" #18

Closed leolopes closed 2 years ago

leolopes commented 3 years ago

Descrição

Como desenvolvedor de temas, gostaria que o comando tray themes retornasse apenas os temas ainda disponíveis no painel.

Motivo

Atualmente o comando mostra todos os temas que já existiram em uma determinada loja, pois, como é descrito na documentação, os temas excluídos permanecem na plataforma. Como desenvolvedores, geramos muitas cópias de temas, e esse relatório fica poluído se mostrar temas antigos excluídos.

Detalhes

Acredito que esse comando deva ser alterado para mostrar apenas os temas não excluídos. Para mostrar os excluídos, aí podemos utilizar uma flag específica. Exemplo: tray themes --include-deleted. Pode ter um alias também, como --d

rhandrade commented 3 years ago

@leolopes Sugestão muito bacana, mas como faríamos isso hoje? 🤔 Atualmente a API do opencode da Tray é quem possui essa limitação, não o CLI em si. Teria alguma sugestão de como faríamos esse filtro após obter os temas?

leolopes commented 3 years ago

@rhandrade , então, eu não analisei a resposta da Tray sobre os temas, não sabendo então se eles possuem algum tipo de propriedade que indique se estão excluídos ou não. Se não tiver, aí não teria como fazer.

A única coisa que eu analisei foi a tabela que o CLI monta para mostrar os temas, mas lá pelo menos parece não mostrar esse dados.

rhandrade commented 3 years ago

Entendi @leolopes. É exatamente por isso que não coloquei nenhuma indicação lá na tabela no CLI. A resposta que a API da Tray me retorna é similar essa baixo (não alterei os nomes dos campos, somente retirei alguns resultados para todo o objeto ficar menor para ilustrar). Inclusive a API me retorna com problemas de acentuação mesmo 😕

{
    "themes": [
        {
            "id": "1",
            "name": "Tema Padrão",
            "published": "1"
        },
        {
            "id": "3",
            "name": "Layout",
            "published": "0"
        }
    ]
}

Vou abrir um chamado no suporte deles e verificar se tem a opção de incluir alguma opção nesse sentido, ou se eles modificam para trazer somente os temas que existem. Ai qualquer coisa eu coloca aqui. Vlw 👍

rhandrade commented 2 years ago

Recurso não suportado.