Skip to content

fix: probably the laziest fix to the search bug you've ever seen#34

Closed
cirex-web wants to merge 2 commits into
mainfrom
fix/search
Closed

fix: probably the laziest fix to the search bug you've ever seen#34
cirex-web wants to merge 2 commits into
mainfrom
fix/search

Conversation

@cirex-web

Copy link
Copy Markdown

@yuriiShmal try to break this :P

@railway-app railway-app Bot temporarily deployed to Clean Code Team (nodebb) / nodebb-spring-26-clean-cod-pr-34 February 9, 2026 03:40 Destroyed
@railway-app

railway-app Bot commented Feb 9, 2026

Copy link
Copy Markdown

🚅 Deployed to the nodebb-spring-26-clean-cod-pr-34 environment in Clean Code Team (nodebb)

Service Status Web Updated (UTC)
nodebb-spring-26-clean-code ✅ Success (View Logs) Web Feb 9, 2026 at 3:43 am

@railway-app railway-app Bot temporarily deployed to Clean Code Team (nodebb) / nodebb-spring-26-clean-cod-pr-34 February 9, 2026 03:40 Destroyed
@yuriiShmal

yuriiShmal commented Feb 9, 2026

Copy link
Copy Markdown

It works but fails the linter.

@yuriiShmal

Copy link
Copy Markdown

Closing this line. Test fails due to undesired behavior: assuming that low level search cares about categories (it is possible it doesn't, as in the case of the failing testcase).

@yuriiShmal yuriiShmal closed this Feb 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants