search

xfrocks / bdApi

[bd] Api add-on for XenForo
GNU General Public License v3.0
98 stars 52 forks source link

Update /users/followings & /users/followers APIs #125

Closed huynhtehoa closed 4 years ago

huynhtehoa commented 4 years ago
huynhtehoa commented 4 years ago

@datbth ơi. a trả thêm cái users_total để FE tính limit, page á. còn cái param total a vẫn để đó tại không biết có chỗ nào đang xài gì hông

huynhtehoa commented 4 years ago

giữ behavior order mặc định là user_id như cũ đi anh

Updated em

daohoangson commented 4 years ago

Cái order coi các method khác nó gom vô chung param không tách riêng param direction nha. Default bao giờ cũng là natural, còn nếu các order khác thì sẽ tương ứng theo key khi trả về, thêm hậu tố nếu là ngược chiều. Ví dụ thread_create_date, thread_create_date_reverse etc.

huynhtehoa commented 4 years ago

Cái order coi các method khác nó gom vô chung param không tách riêng param direction nha. Default bao giờ cũng là natural, còn nếu các order khác thì sẽ tương ứng theo key khi trả về, thêm hậu tố nếu là ngược chiều. Ví dụ thread_create_date, thread_create_date_reverse etc.

Dạ ukie anh. Em có update lại, anh review lại giúep em nha. Thanks anh Sơn

daohoangson commented 4 years ago

Có vẻ ổn á, mà coi lại 2 điểm:

  1. Chắc bỏ cái in request check cho total, giờ luôn trả lại field đó rồi.
  2. Confirm lại design xem screen này có cần full user data không (e.g. avatar?), nếu có thì chuyển qua xài search api để trả về user data luôn. Vẫn giữ pagination.
huynhtehoa commented 4 years ago

Có vẻ ổn á, mà coi lại 2 điểm:

  1. Chắc bỏ cái in request check cho total, giờ luôn trả lại field đó rồi.
  2. Confirm lại design xem screen này có cần full user data không (e.g. avatar?), nếu có thì chuyển qua xài search api để trả về user data luôn. Vẫn giữ pagination.

Em có bỏ in request check với update lại code để dùng search API á anh. Anh review lại giúp em nha

huynhtehoa commented 4 years ago

Giờ đang fetch all rồi mới slice lại theo page. Cái này chắc model fetch theo page ngay từ đầu thì an toàn hơn ha. Đề phòng mấy user có nhiều followers kiểu cuhiep / duyluan á.

Còn lại anh thấy okie rồi

Dạ ukie. Updated nha anh

huynhtehoa commented 4 years ago

Anh @daohoangson khi nào rãnh review lại giúp em nhé

daohoangson commented 4 years ago

Sau mấy commits gần đây cái PR này đã test lại chưa?

huynhtehoa commented 4 years ago

Sau mấy commits gần đây cái PR này đã test lại chưa?

Em vừa test lại thì thấy vẫn ổn á

huynhtehoa commented 4 years ago

Ổn rồi á, refactor cái bdApi_countUsersBeingFollowedByUserId qua model riêng luôn cho gọn ha.

Dạ updated nha anh