Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

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 join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants