Tornar o ngDisqus opcional

frontpressorg / frontpress

:zap: A full front-end AngularJS template for WordPress Rest API.

MIT License

108 stars 17 forks source link

Tornar o ngDisqus opcional #91

Open teles opened 7 years ago

teles commented 7 years ago

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.

alisonmonteiro commented 7 years ago

@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?

teles commented 7 years ago

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?

teles commented 7 years ago

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

alisonmonteiro commented 7 years ago

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.