igorkasyanchuk/active_storage_validations

Validations not working on Ckeditor gem models?

Fechada

#145 aberto em 24 de fev. de 2022

Ver no GitHub
 (5 comentários) (0 reação) (0 responsável)Ruby (148 forks)github user discovery
bughelp wanted

Métricas do repositório

Stars
 (1.179 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

Hello! I am unsure if this should mentioned here or under the ckeditor gem...

I have noticed that adding validations for file content types and sizes for Ckeditor::Picture, Ckeditor::Asset, and Ckeditor::AttachmentFile models do not seem to work.

Adding: validates :storage_data, size: { less_than: 5.megabytes }, content_type: /\Aimage\/.*\z/ or validates :storage_data, size: { less_than: 5.megabytes }, content_type: ['image/png', 'image/jpg', 'image/jpeg'] in any of the above listed models do not restrict a large image or a pdf from being uploaded. The types seem to be registered with Marcel::EXTENSIONS

ckeditor gem (5.1.1) active_storage_validations gem (0.9.6)

Guia do colaborador

Direção de pesquisa
Inspecione como as validações são aplicadas aos modelos Ckeditor e verifique se o atributo 'storage data' é um ActiveStorage::Attached::One ou similar. Verifique se a gem active storage validations suporta validações em modelos aninhados ou se há falta de inclusão do módulo de validação.
Pilha de tecnologia
ruby
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
RubyRails
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75