frontpress
frontpress copied to clipboard
Tornar o ngDisqus opcional
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
useDiqusfortrue.
@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?
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:
- Mantive a dependência
- Tornei a dependência opcional via frontpress.json e $FrontPressProvider
- Para fechar a issue vou arrumar uma forma de importar os arquivos da dependência apenas se ela for importada
O que acha de seguir algo nesse sentido nessa issue?
Observação, agora o projeto roda por default em localhost:4000/#/ @alisonmonteiro
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.