Skip to content

Problema das coleções paginadas#2

Merged
cvscarlos merged 3 commits intomasterfrom
unknown repository
Sep 11, 2013
Merged

Problema das coleções paginadas#2
cvscarlos merged 3 commits intomasterfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Mar 21, 2013

Fiz um ajuste pra consertar o plugin no caso de coleções paginadas. Não sei se foi dos melhores, mas deve funcionar. Mexi só no método getSearchUrl. Dá uma olhada e, se estiver OK, aceita o pull aí.

Valeu!

@ghost
Copy link
Copy Markdown
Author

ghost commented Mar 21, 2013

Ei, ainda não está funcionando bem. Por favor, não aceite o pull request por enquanto.

Versão minificada também foi atualizada.
@ghost
Copy link
Copy Markdown
Author

ghost commented Mar 22, 2013

Agora está funcionado. Dá uma olhada no commit mais recente pra ver a modificação que eu fiz. Mexi também só método getSearchUrl. Se quiser ver um exemplo do plugin rodando com coleções paginadas, olha aqui: http://develop-coop.vtexcommerce.com.br/eletronicos

Valeu!

@cvscarlos
Copy link
Copy Markdown
Owner

Darei uma olhada, obrigado pela contribuição :)

  • Carlos Vinicius*
    *
    *
    +55 11 9-9718-6277 (vivo)
    +55 11 9-8065-0860 (oi)

Em 21 de março de 2013 21:32, Pedro Freires notifications@github.comescreveu:

Agora está funcionado. Dá uma olhada no commit mais recente pra ver a
modificação que eu fiz. Mexi também só método getSearchUrl. Se quiser ver
um exemplo do plugin rodando com coleções paginadas, olha aqui:
http://develop-coop.vtexcommerce.com.br/eletronicos

Valeu!


Reply to this email directly or view it on GitHubhttps://github.com//pull/2#issuecomment-15274960
.

@cvscarlos
Copy link
Copy Markdown
Owner

Não consegui ver funcionando na URL, imagino que seja por conta de ser um pullrequest antigo. De qualquer modo irei aceitar o pullrequest e ajusto algo caso precise.

Obrigado

cvscarlos added a commit that referenced this pull request Sep 11, 2013
Problema das coleções paginadas
@cvscarlos cvscarlos merged commit ddfebca into cvscarlos:master Sep 11, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant