Skip to content

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

Open
luisberns wants to merge 8 commits into
turicas:developfrom
luisberns:badge-dados
Open

Adiciona badge [Dados libertos pelo Brasil.io] na pág. de dataset#240
luisberns wants to merge 8 commits into
turicas:developfrom
luisberns:badge-dados

Conversation

@luisberns

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

Copy link
Copy Markdown
Contributor

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

@berinhard berinhard left a comment

Copy link
Copy Markdown
Contributor

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

turicas commented Apr 23, 2020

Copy link
Copy Markdown
Owner

Testar se com minio a URL da imagem fica fixa.

@berinhard

berinhard commented May 4, 2020

Copy link
Copy Markdown
Contributor

@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

turicas commented May 4, 2020

Copy link
Copy Markdown
Owner

@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

berinhard commented May 4, 2020

Copy link
Copy Markdown
Contributor

@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

luisberns commented May 5, 2020

Copy link
Copy Markdown
Contributor Author

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

@turicas

turicas commented May 5, 2020

Copy link
Copy Markdown
Owner

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

luisberns commented May 5, 2020

Copy link
Copy Markdown
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

1 more reviewer

@berinhard berinhard berinhard approved these changes

Reviewers whose approvals may not affect merge requirements

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