Sobre as definições de imagens

klawdyo commented 4 months ago

1) Possibilidade de múltiplas imagens no produto

Como a propriedade image é opcional e nenhum sistema depende dela, eu sugiro que possa existir a propriedade images que contém um array de imagens. Se o sistema não suportar múltiplas imagens, e o image estiver vazio, ele tratará como um produto sem imagem. Pode existir uma recomendação para sempre enviar o image com a primeira imagem quando utilizar o array de imagens, garantindo retrocompatibilidade.

2) Tag na imagem

Permitir colocar uma tag na imagem, para dizer se ela é thumb ou a main. Ou um outro enum diferente como small, medium e large.

Ex.:

{
     url: "https://.......",
     "CRC-32": "21o3iu12oi3u123",
     type: "main"
}

Ou, outra possibilidade, adicionar a propriedade thumb para mostrar uma url alternativa com uma versão menor da imagem.

{
     url: "https://.......",
     "CRC-32": "21o3iu12oi3u123",
     thumb: "https://.......",
}

3) Incluir `webp` nas imagens aceitas.

Atualmente somente jpeg, gif e png estão suportadas.

OpenDelivery-Tech commented 4 months ago

Olá @klawdyo,

Vamos levar essas questões para discutir no próximo comitê técnico que acontecerá dia 18/07 as 9:30. Você poderia nos enviar seu e-mail de contato para o endereço lider.opendelivery@abrasel.com.br, assim te colocamos no invite para que você nos ajude com a solução.

klawdyo commented 4 months ago

Eu estarei nos próximos invites também?

Não participei desse pois o link foi enviado meia hora antes e eu só vi o email a noite.

OpenDelivery-Tech commented 3 months ago

Olá @klawdyo

Discutimos a issue nos dois últimos comitês, e ela foi acatada. Ela será implementada na próxima versão (v1.5.0) que sai em janeiro. Enquanto isso voce pode acompanhar as modificações na documentação da versão de release candidate: https://abrasel-nacional.github.io/docs/versions/rc/

A issue permanecerá aberta até o merge com a versão release.

Fique a vontade para colocar novos comentários.

klawdyo commented 3 months ago

Eu acho que a sugestão de adicionar uma propriedade thumb na imagem já existente seria melhor. Motivo: a propriedade images poderia vir a representar no futuro o suporte a um produto com várias imagens. Eu sei que não é comum no ramo, mas caso haja a mudança de ideia nas próximas versões, a forma aprovada poderá vir a se tornar uma break change.

OpenDelivery-Tech commented 3 months ago

Olá @klawdyo

Vamos retomar a discussão no próximo comite que acontece dia 05/09. Por favor verifique se voce possui o invite.

OpenDelivery-Tech commented 2 months ago

Olá @klawdyo

Levamos seus pontos para discussão no comite, mas entendemos que caso o padrãõ evolua para um versão 2.0, onde haveria break changes de qualquer maneira seria mais correto apenas manter o novo campo images, eliminando o campo imagejá existente. Iremos manter a solução adotada acima, mas a issue continuará aberta, até o lançamento da próxima versão que ocorre em Janeiro/25. Fica a vontade para colocar mais comentários e participar dos próximos comitês.

ferhrosa commented 2 months ago

Caso essa alteração seja implementada, tenho uma sugestão de mudança em relação ao que está na versão v1.5.0-rc.1. Eu acho que os itens do novo enum do campo type deveriam estar em caixa alta, assim como os outros enums já definidos. Exemplo: MAIN e THUMB

Abrasel-Nacional / docs