fatihtas / martmara-backend-ugur

0 stars 0 forks source link

Sorting Venues Before Answering Client Search Calls #6

Closed borakrc closed 9 years ago

borakrc commented 9 years ago

in services/SearchService.js, line 560 we have:

var query = Venue .find({'keywords.keyword' : new RegExp('^' + prefix + '\w*')}) .where('status').equals(Venue.STATUS_APPROVED) .sort('-_id') .limit(25) .populate('categories') .populate('ownerInfo.owner' , '-password') .populate('creator' , '-password');

burada donen tum venue'lari id'ye gore sortlayip size'i 25'e mi dusurmusuz, yoksa donen ilk 25 tanesini mi sortlamisiz? eger ilk dedigimse performans acisindan sikinti olacaktir ilerde, ayni zamanda adam mesela "yildirim" arattiginda adinda yildirim gecen ilk 25 magaza doner ama ona yakin olanlar degil bize daha once kaydolanlar doner. illa sortlayacaksak da mesafeye gore sortlasak, ne dersiniz?

not: biz bunu client tarafinda tekrar sortluyoruz zaten distance'a ya da rating'e gore, bu client'a dondugumuz venue seti.

fatihtas commented 9 years ago

Burada bahsedilen kod, app'in kodu degil, admin sayfasina dusen claimed business'larin kodu. Orada bi siralama yapiyor.. iOS'te ki search baska..

Eger admin tool'u gormek istersen: http://www.martmara.com/admin/login.php username: test@gmail.com password: 1234567890

ilgili sayfa: http://www.martmara.com/admin/list_venue.php burda claim edilmis mekanlari secersen..

borakrc commented 9 years ago

hmm, searchVenue ios tarafindan hic cagirilmiyor yani? tamam oyleyse, pardon.

fatihtas commented 9 years ago

Cagiriliyor, sadece kodun bu kisminda degil.. ama urun araniyor once, sonra company type, sonra category. Sonrada cevap olarak venue donuyor. bu sirkette var bu aradigin diye. ya urunlerinde, ya kategori'de yada isminde var.

Buna biz simdi urun yorumlari ile urun aciklamalarini da ekleyecegiz.

ugurdonmez commented 9 years ago

ios da bu method cagrilmiyor bora