-
Notifications
You must be signed in to change notification settings - Fork 397
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(advertisement): adds getRandom
function to retrieve random ads
#1745
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Show! Deixei alguns comentários, talvez o único importante seja o sobre $required.content_type
.
Vou trabalhar na UI da etapa 3a com base nessa branch (me avise caso já tenha começado a trabalhar nela).
Já vou atualizar a branch resolvendo o que foi apontado 🤝 Não cheguei a fazer nada sobre a UI dessa etapa. Precisa de alguma ajuda? |
5e8942f
to
faa5e5e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pronto para mesclar 👍
Não cheguei a fazer nada sobre a UI dessa etapa. Precisa de alguma ajuda?
Já está quase pronta, falta dar uma conferida final. Abrirei o PR em breve. Obrigado 🤝
Então vou pra etapa 3c 🤝 @Rafatcb, quer ficar com a 3d? Talvez dê para aproveitar algo do seu estudo em #1491 (comment) |
Ok. Abri o PR da UI da 3a agora. |
Tá tudo muito massa 😍 |
Essa é uma parte da etapa 3a citada em #1491 (comment).
Mudanças realizadas
Cria o model
advertisement
com uma funçãogetRandom
responsável por devolver uma quantidade escolhida de publicidades aleatórias.Cada publicidade devolvida contém apenas as propriedades básicas necessárias para a UI: id, title, slug, owner_username, source_url e type. Sendo que
type
aqui é oad_type
e não ocontent_type
, então por enquanto é sómarkdown
.Foi criado o
/sponsored-beta
provisoriamente para facilitar os testes do novo model.Só não estamos testando realmente a aleatoriedade, que foi deixada como responsabilidade para
ORDER BY RANDOM()
do PostgreSQL.Tipo de mudança
Checklist: