Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIX: error when CategoryList tried to find relevant topics #22339

Merged
merged 2 commits into from Jun 29, 2023

Conversation

lis2
Copy link
Contributor

@lis2 lis2 commented Jun 29, 2023

Recently, we added the option for watched tag/categories to take precedence over muted tag/categories. Therefore, remove_muted_tags is using category_users to check if categories are not watched. There was missing join in CategoryList which was causing an error.

Recently, we added the option for watched tag/categories to take precedence over muted tag/categories. Therefore, `remove_muted_tags` is using `category_users` to check if categories are not watched. There was missing join in CategoryList which was causing an error.
Copy link
Contributor

@martin-brennan martin-brennan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a minor comment :)

app/models/category_list.rb Outdated Show resolved Hide resolved
@lis2 lis2 merged commit de2febc into discourse:main Jun 29, 2023
13 checks passed
@lis2 lis2 deleted the broken-categories branch June 29, 2023 01:26
@discoursebot
Copy link

This pull request has been mentioned on Discourse Meta. There might be relevant details there:

https://meta.discourse.org/t/getting-a-500-error-on-the-mobile-bookmark/269928/20

@discoursebot
Copy link

This pull request has been mentioned on Discourse Meta. There might be relevant details there:

https://meta.discourse.org/t/postalerter-job-hangs-ooms-as-of-late/270183/1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
3 participants