frontpress icon indicating copy to clipboard operation
frontpress copied to clipboard

Tornar o ngDisqus opcional

Open teles opened this issue 9 years ago • 4 comments

Seguindo a mesma ideia da issue #90.

Hoje o ngDisqus é uma dependência fixa do projeto. Para diminuir a quantidade de dependências a utilização do ngDisqus deve ser opcional.

Futuramente (fora dessa issue) pensei em deixar o ngDisqus como um plugin nativo do FrontPress. Mas para essa issue o critério de pronto é:

  • Só utilizar scroll infinito quando useDiqus for true.

teles avatar Dec 01 '16 00:12 teles

@teles

On it!

Para diminuir a quantidade de dependências a utilização do ngDisqus deve ser opcional.

Você pensa em remover ele do package.json e, caso useDiqus === true, damos um npm i angular-disqus? Ou a ideia inicial é outra?

alisonmonteiro avatar Dec 07 '16 21:12 alisonmonteiro

Opa, e aí @alisonmonteiro ! Então acho que na parte do

Para diminuir a quantidade de dependências a utilização do ngDisqus deve ser opcional.

Eu me equivoquei, queria fazer algo como o #90 onde:

O que acha de seguir algo nesse sentido nessa issue?

teles avatar Dec 07 '16 23:12 teles

Observação, agora o projeto roda por default em localhost:4000/#/ @alisonmonteiro

teles avatar Dec 07 '16 23:12 teles

Acho algo bem mais sensato mesmo. ;)

Sugeri este caso apenas para entender melhor. Tanto que no caso de rodar o npm install... como citei, pode dar algum problema de permissão (algumas pessoas precisam rodar com sudo ao rodar npm install), etc.

alisonmonteiro avatar Dec 08 '16 00:12 alisonmonteiro