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

Implementare autocomplete la căutare #28

Closed
okvivi opened this issue May 11, 2012 · 6 comments
Closed

Implementare autocomplete la căutare #28

okvivi opened this issue May 11, 2012 · 6 comments

Comments

@okvivi
Copy link
Member

okvivi commented May 11, 2012

Pentru că deja API-ul de search returnează JSON, ar fi destul de interesant de adăugat autocomplete la căutare. Mi-ar plăcea dacă am aveau un autocomplete fancy care să afișeze și poza mică a politicianului și câteva informații foarte pe scurt despre el direct în autocomplete.

http://jqueryui.com/demos/autocomplete/

Vivi.

@okvivi
Copy link
Member Author

okvivi commented May 26, 2012

Acest issue ar fi destul de ușor de adăugat și suficient de izolat de cod în general. Autocomplete-ul ar trebui să folosească API-ul de căutare http://hartapoliticii.ro/api/search.php?q=traian+basescu&api_key=hackathon

@vld666
Copy link
Contributor

vld666 commented Sep 29, 2012

am comis un autocomplete :P

vld666 added a commit to vld666/hartapoliticii that referenced this issue Sep 29, 2012
@okvivi
Copy link
Member Author

okvivi commented Sep 29, 2012

Trimite-mi un pull request.

@tct
Copy link
Member

tct commented Sep 29, 2012

Vlad a plecat acasă, deci depindem de tine :-)

git remote add vlad git://github.com/vld666/hartapoliticii.git
git pull vlad master
git push origin master # la @pistruiatul  

@ghost ghost assigned vld666 and okvivi Oct 1, 2012
okvivi added a commit that referenced this issue Oct 10, 2012
#28: Implementare autocomplete la căutare
@okvivi
Copy link
Member Author

okvivi commented Oct 11, 2012

Hey, am făcut pull și la mine local nu merge.
Poți verifica că merge autocomplet-ul înainte să încep să debugez?

@ghost ghost assigned vld666 Oct 12, 2012
@okvivi
Copy link
Member Author

okvivi commented Oct 12, 2012

M-am prins de ce nu mergea, din cauza cheii locale pentru API. Am actualizat cheia și am făcut și niște schimbări la cum arată autocomplete-ul. ca504de

Am împins codul în producție, acum avem autocomplete! Yay!

@okvivi okvivi closed this as completed Oct 12, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants