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

Validação de tipo de arquivo #10

Closed
rhandrade opened this issue Aug 22, 2021 · 9 comments
Closed

Validação de tipo de arquivo #10

rhandrade opened this issue Aug 22, 2021 · 9 comments
Labels
improvements New features, requests or improvements
Milestone

Comments

@rhandrade
Copy link
Owner

Descrição

Como usuário do CLI gostaria que a ferramenta verificasse o tipo do arquivo a ser enviado e comunique caso ele não seja compatível.

Motivo

Ao desenvolvermos um tema estamos constantemente trabalhando com vários tipos de arquivos e podemos não nos atentar ao tipo do arquivo no momento em que criamos. Sem uma validação disso podemos desprender tempo averiguando algo que não é um bug ou problema, e sim uma limitação.

@rhandrade rhandrade added the improvements New features, requests or improvements label Aug 22, 2021
@rhandrade rhandrade added this to the Versão 1.1.0 milestone Aug 22, 2021
@leolopes
Copy link
Contributor

Imagino que a ideia seria validar o tipo de arquivo (e depois talvez até a pasta) para identificar se ele pode ou não ser subido ao servidor, em vez de deixar o servidor testar isso, correto?

@rhandrade
Copy link
Owner Author

Exatamente @leolopes. As pastas é até que bem fácil mapear pois a Tray permite que seja adicionado novas pastas somente na css e js. Os tipos de arquivos ai iria validar de acordo com o atual do opencode_theme e verificar possíveis tipos que ocorrem erros, como o caso de arquivos .scss

@leolopes
Copy link
Contributor

leolopes commented Sep 3, 2021

Entendido, acho que vou tentar dar andamento a este.

@leolopes
Copy link
Contributor

leolopes commented Sep 3, 2021

Eu fiz um esboço inicial dessa melhoria, e estou aprimorando.

Ainda assim, me questiono: se evitarmos a todo custo receber respostas de erro da Tray, não vamos saber quando houver alguma mudança de regra na API.

Será que deveria haver um "modo desenvolvedor", para deixarmos o arquivo subir e ser rejeitado? Assim aprenderemos sempre sobre as mudanças de regra.

@rhandrade
Copy link
Owner Author

@leolopes Bom questionamento... Acho que podemos usar um modo debug, uma opção extra no arquivo de configurações que se habilitar o modo debug tudo quer é feito gera um log em um arquivo que sempre será ignorado pelo CLI, algo como .debug.log.

@leolopes
Copy link
Contributor

leolopes commented Sep 6, 2021

@rhandrade, interessante isso do arquivo de log, pois assim não polui os logs do prompt de comando.
Quem sabe colocamos num milestone mais para frente?

@rhandrade
Copy link
Owner Author

@leolopes Sim... Vou já gerar outra issue, atrelada a essa aqui para pensar nessa questão do arquivo de log.

@rhandrade
Copy link
Owner Author

rhandrade commented Oct 10, 2021

Issue fechado, pois feature foi implementada.
Versão 1.1.0 liberada! Vlw pessoal 🎉

@leolopes
Copy link
Contributor

Boaaa, já vou começar a usar :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
improvements New features, requests or improvements
Projects
None yet
Development

No branches or pull requests

2 participants