Skip to content
This repository has been archived by the owner on Feb 11, 2022. It is now read-only.

Comando "new" com "theme_base" não surtiu efeito #19

Closed
leolopes opened this issue Sep 9, 2021 · 6 comments
Closed

Comando "new" com "theme_base" não surtiu efeito #19

leolopes opened this issue Sep 9, 2021 · 6 comments
Labels
bug Something isn't working
Projects
Milestone

Comments

@leolopes
Copy link
Contributor

leolopes commented Sep 9, 2021

Descrição

Ao rodar o comando tray new incluindo o parâmetro theme_base não surtiu o efeito esperado. O novo tema, em vez de ser baseado no tema base, foi gerado a partir do tema padrão da Tray.

@rhandrade rhandrade added the bug Something isn't working label Sep 11, 2021
@rhandrade
Copy link
Owner

@leolopes Consegue verificar se esse comportamento se repete ao usar o opencode new da Tray? Nele a Tray passa o nome do tema a ser usado como base e caso não passe nenhum ele usa o default.

Repare que se o tema foi criado pela interface web, os nomes no opencode list são diferentes (até hoje não entendi o porque), e isso pode afetar o comando new com o parâmetro theme_base.

Se conseguir fazer esses testes e postar o resultado aqui agradeceria.

@leolopes
Copy link
Contributor Author

leolopes commented Sep 11, 2021

@rhandrade o parâmetro theme_base é o nome do tema então? Eu achei que seria o código.
Posso fazer os testes sim.

@Merieli
Copy link

Merieli commented Sep 12, 2021

@rhandrade e @leolopes eu testei de várias formas então acho que posso contribuir. O próprio comando opencode new da Tray não está funcionando e também gera o tema a partir do padrão tray.
A tentativa foi realizada com o tema base Car Parts

Como testei:

  • nome do tema base [usei o nome existente no opencode list]
  • código do tema base
  • nome do tema base entre aspas [devido ao espaço existente no nome pensei que poderia ser o problema]
  • nome do tema base substituindo o espaço por um "_"

Da mesma forma testei utilizando o comando tray new e ocorre o mesmo.

@leolopes
Copy link
Contributor Author

Obrigado @Merieli!
Imagino então que o problema seja com a API, que não está respeitando o comando.
Quem sabe tenhamos que deixar o comando no código mas logar algum aviso indicando que talvez não funcione.

@rhandrade
Copy link
Owner

Obrigado pelos testes @Merieli, ajudou muito a gente validar aqui 🙂

@leolopes Já abri um chamado na Tray sobre essa questão e eles verificaram que o problema é do lado deles mesmo. O comando está ignorando o parâmetro theme_base. Eles falaram que o pessoal iria verificar o porque isso acontece. Até lá vamos colocar um log informando que o parâmetro theme_base pode não ser refletido devido a problemas na API.

Já adicionei essa issue para a versão 1.0.0. Vlw pessoal!!! 👏

@rhandrade rhandrade added this to the Versão 1.0.0 milestone Sep 25, 2021
@rhandrade rhandrade added this to To do in Tray Theme Sep 25, 2021
@rhandrade rhandrade moved this from To do to In progress in Tray Theme Sep 25, 2021
@rhandrade rhandrade moved this from In progress to Done in Tray Theme Sep 25, 2021
@rhandrade
Copy link
Owner

Já fiz um PR que adiciona um log informando o problema com esse parâmetro. Vlw toda a ajuda pessoal, vocês são demais! 👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
Development

No branches or pull requests

3 participants