Skip to content

Commit 224f39d

Browse files
authored
Merge pull request #425 from alimy/pr-fixed-follow-page
fixed follow page incorrect results
2 parents 6a66a29 + a93cc01 commit 224f39d

26 files changed

+37
-25
lines changed

Diff for: CHANGELOG.md

+3
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,9 @@ TODO;
88
### Change
99
- frontend: optimize to use fold/unfold action for post item display.
1010

11+
### Fixed
12+
- fixed follow page incorrect results.
13+
1114
## 0.5.1
1215
### Added
1316
- User/Profile page add setting option when page's user is self.

Diff for: internal/servants/web/followship.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,7 @@ func (s *followshipSrv) ListFollowings(r *web.ListFollowingsReq) (*web.ListFollo
3535
logrus.Errorf("Ds.GetUserByUsername err: %s", err)
3636
return nil, web.ErrNoExistUsername
3737
}
38-
res, err := s.Ds.ListFollowings(he.ID, r.PageSize, r.Page-1)
38+
res, err := s.Ds.ListFollowings(he.ID, r.PageSize, (r.Page-1)*r.PageSize)
3939
if err != nil {
4040
logrus.Errorf("Ds.ListFollowings err: %s", err)
4141
return nil, web.ErrListFollowingsFailed
@@ -55,7 +55,7 @@ func (s *followshipSrv) ListFollows(r *web.ListFollowsReq) (*web.ListFollowsResp
5555
logrus.Errorf("Ds.GetUserByUsername err: %s", err)
5656
return nil, web.ErrNoExistUsername
5757
}
58-
res, err := s.Ds.ListFollows(he.ID, r.PageSize, r.Page-1)
58+
res, err := s.Ds.ListFollows(he.ID, r.PageSize, (r.Page-1)*r.PageSize)
5959
if err != nil {
6060
logrus.Errorf("Ds.ListFollows err: %s", err)
6161
return nil, web.ErrListFollowsFailed

Diff for: web/dist/assets/404-ee20e4ba.js renamed to web/dist/assets/404-3870dbb4.js

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Diff for: web/dist/assets/Anouncement-0447e517.js renamed to web/dist/assets/Anouncement-ffc6370f.js

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Diff for: web/dist/assets/Collection-105946d5.js renamed to web/dist/assets/Collection-c204e163.js

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)