LinkinStars commented on code in PR #1275:
URL: https://github.com/apache/answer/pull/1275#discussion_r1980587122
##########
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:
> Only the "Active" filter shows the last person who performed the action,
e.g., answered, modified.
https://github.com/apache/answer/issues/1270#issuecomment-2696093377
That means the other conditions always show the operator who asked the
question.
So, the `orderCond != schema.QuestionOrderCondActive` condition is enough.
Remove the `|| (!haveEdited && !haveAnswered)`
--
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]
