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.
   
   
![image](https://github.com/user-attachments/assets/3ec97421-09dc-4907-a080-e1afcf0d04b1)
   



-- 
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]

Reply via email to