Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Tornar o ngDisqus opcional #91

Open
teles opened this issue Dec 1, 2016 · 4 comments
Open

Tornar o ngDisqus opcional #91

teles opened this issue Dec 1, 2016 · 4 comments
Assignees

Comments

@teles
Copy link
Member

teles commented Dec 1, 2016

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

@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
Copy link
Member Author

teles commented Dec 7, 2016

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
Copy link
Member Author

teles commented Dec 7, 2016

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

@alisonmonteiro
Copy link
Member

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.

teles added a commit that referenced this issue Dec 13, 2016
@teles teles self-assigned this Dec 26, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants