github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2024-11-25 08:43:25 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

FEATURE: Hide Privacy Policy and TOS topics (#18533)

Browse Source 
* FEATURE: Hide Privacy Policy and TOS topics

As a way to simplify new sites this change will hide the privacy policy
and the TOS topics from the topic list. They can still be accessed and
edited though.

* add tests
This commit is contained in:
Blake Erickson 2022-10-10 17:19:18 -06:00 committed by GitHub
parent cab173d3b5
commit 49abcf965b
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 16 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/topic_query.rb
View File

@ -438,6 +438,9 @@ class TopicQuery
topics = topics.to_a
# These don't need to be shown in the topic list
topics = topics.reject { |t| [SiteSetting.privacy_topic_id, SiteSetting.tos_topic_id].include?(t[:id]) }
if options[:preload_posters]
user_ids = []
topics.each do |ft|

13
spec/requests/list_controller_spec.rb
View File

@ -79,6 +79,19 @@ RSpec.describe ListController do
expect(parsed["topic_list"]["topics"].length).to eq(1)
end
it 'filters out privacy policy and tos topics' do
tos_topic = create_topic
SiteSetting.tos_topic_id = tos_topic.id
pp_topic = create_topic
SiteSetting.privacy_topic_id = pp_topic.id
get "/latest.json"
expect(response.status).to eq(200)
parsed = response.parsed_body
expect(parsed["topic_list"]["topics"].length).to eq(1)
end
it "shows correct title if topic list is set for homepage" do
get "/latest"