Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Adiciona badge [Dados libertos pelo Brasil.io] na pág. de dataset #240

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from
Open

Adiciona badge [Dados libertos pelo Brasil.io] na pág. de dataset #240

wants to merge 8 commits into from

Conversation

luisberns
Copy link
Contributor

@luisberns luisberns commented Apr 23, 2020 •
edited by berinhard

Fixes #202

Foi adicionado o badge do Brasil.io no template dataset-detail.html, o resultado visual está representado abaixo.

Representação visual da implementação do badge na página

@luisberns luisberns mentioned this pull request Apr 23, 2020
Open
berinhard
berinhard approved these changes Apr 23, 2020
View reviewed changes
Copy link
Collaborator

@berinhard berinhard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pra mim parece ok, @turicas rola de dar uma conferida também?

@turicas
Copy link
Owner

turicas commented Apr 23, 2020

Testar se com minio a URL da imagem fica fixa.

@berinhard
Copy link
Collaborator

@turicas já podemos mergear esse PR, certo? Depois do deploy da migração dos arquivos estáticos pro minio, está garantido que a URL fica única.

@turicas
Copy link
Owner

turicas commented May 4, 2020

@turicas já podemos mergear esse PR, certo? Depois do deploy da migração dos arquivos estáticos pro minio, está garantido que a URL fica única.

Precisa corrigir o <a href="">citar a fonte</a>.

@berinhard
Copy link
Collaborator

@luisberns o link deve apontar para https://github.com/turicas/covid19-br#licen%C3%A7a-e-cita%C3%A7%C3%A3o de acordo com os comentários da issue #202

@luisberns
Copy link
Contributor Author

@berinhard Acabei de adicionar o link que você indicou acima!

@turicas
Copy link
Owner

turicas commented May 5, 2020

Pessoal, acho que teve alguma confusão: esse template é para a página genérica de dataset, não pode ficar com o link só pro covid19.

@luisberns
Copy link
Contributor Author

Ah verdade @turicas, estamos adicionando no template geral, qual seria o link para adicionarmos?

@turicas turicas force-pushed the develop branch 2 times, most recently from 09cb6a7 to 1a5b0a7 Compare September 5, 2020 01:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@berinhard berinhard berinhard approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Adicionar badges de compartilhamento de datasets
3 participants
@luisberns @turicas @berinhard