sy-records commented on code in PR #1275:
URL: https://github.com/apache/answer/pull/1275#discussion_r1980605923
##########
internal/service/question_common/question.go:
##########
@@ -409,8 +409,8 @@ func (qs *QuestionCommon) FormatQuestionsPage(
}
}
- // if order condition is newest or nobody edited or nobody
answered, only show question author
- if orderCond == schema.QuestionOrderCondNewest || (!haveEdited
&& !haveAnswered) {
+ // If the order condition is not active or no one has
edited/answered, display only the question author
+ if orderCond != schema.QuestionOrderCondActive || (!haveEdited
&& !haveAnswered) {
Review Comment:
Removing it causes a logical problem: when `orderCond` is `active` and no
one has edited or answered it, nothing is displayed.

--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]