Skip to content

Revisao post 2#18

Open
Marcelo-Jose wants to merge 10 commits intomasterfrom
revisao_post_2
Open

Revisao post 2#18
Marcelo-Jose wants to merge 10 commits intomasterfrom
revisao_post_2

Conversation

@Marcelo-Jose
Copy link
Contributor

@Marcelo-Jose Marcelo-Jose commented Dec 18, 2020

Propósito deste Pull Request

  • Novo post no blog
  • Alterações em uma página estática
  • Alteração de configuração geral do site
  • Nova página estática

Descrição breve das alterações

Feitas as alterações solicitadas pelo Jeová e Jayme, apenas questões textuais.

Responsáveis

Davi e Marcelo

Endereço para pré-visualização

https://revisao_post_2--pet-est.netlify.app/

Copy link
Member

Choose a reason for hiding this comment

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

Tirar os acentos do slug

Copy link
Member

@fernandomayer fernandomayer Mar 3, 2021

Choose a reason for hiding this comment

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

Os gráficos gerados pelo plotly não estão aparecendo na página. Acho que precisa fazer alguma coisa tipo o que está aqui https://harborisland.github.io/plotly/how-to-use-plot.ly-and-ggplotly-with-R-Markdown para funcionar.

Os gráficos parecem ter sido salvos em PNG no static/blog/... mas não estão aqui (ou não vieram no commmit).

Copy link
Member

Choose a reason for hiding this comment

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

Não aparece

Copy link
Member

Choose a reason for hiding this comment

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

Não aparece

Copy link
Member

Choose a reason for hiding this comment

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

Não aparece

Copy link
Member

@fernandomayer fernandomayer left a comment

Choose a reason for hiding this comment

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

Vejam os comentários que fiz nos gráficos, eles não estão aparecendo.

Não precisa abrir um novo pull request depois que arrumarem, basta arrumar e comitar nesse mesmo branch.

Copy link

Choose a reason for hiding this comment

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

No trecho "O estado do Paraná adotou no mês de junho" acredito que seja importante contextualizar o ano.

@fernandomayer
Copy link
Member

Eu arrumei os nomes dos arquivos e agora consigo renderizar a página com os gráficos localmente. O problema é que no servidor ainda não aparece.

De qualquer maneira, precisamos pensar em uma outra forma de adicionar os gráficos do plotly, que não seja dessa forma direta. Isso porque toda vez que alguém renderizar o site, ele vai ficar compilando esse plotly e vai começara pesar para todo mundo.

Pelo que vi, uma saída é salvar o gráfico em json ou html e depois incluir de alguma outra forma (assim ele não vai ficar gerando todas as vezes.

Links:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants