feat(Home Page): Admin news on home

[Aaron-Detre]

Changes

• When Discourse is not set in application properties, the home page should show news items from the admin tools.
• Moved content of DiscourseLatestNewsComponent to LatestNewsComponent which is shared between Discourse and Admin news.
• Admin news items link to pre-existing /news page.
• NewsComponent was using [innerHTML]="sanitizer.bypassSecurityTrustHtml(newsItem.news)" instead of just displaying the text. I didn't see any reason it shouldn't just display the text, so I changed it, but I'm not sure if that was something important.

Test

• When Discourse url and news category are set, the home page should still show Discourse news, otherwise it should show admin news.
• Admin news should show the 3 latest public news items.
• The More News button should link to the separate admin news page.
• Each of the three news items should link to that same page and should automatically scroll to that news item.
• If signed out, the news page should only show public news items. If signed in, it should show all news items.

[qltysh]

Coverage Impact

Unable to calculate total coverage change because base branch coverage was not found.

Modified Files with Diff Coverage (6)

Rating	File	% Diff	Uncovered Line #s
	.../home/discourse-latest-news/discourse-latest-news.component.ts	100.0%
	src/app/home/latest-news/latest-news.component.ts	100.0%
	src/app/discourse-feed/discourse-feed.component.ts	100.0%
	src/app/home/admin-latest-news/admin-latest-news.component.ts	100.0%
	src/app/home/home.component.ts	100.0%
	src/app/news/news.component.ts	75.0%	57, 66-71
	Total	92.2%

🤖 Increase coverage with AI coding...

In the `admin-news-on-home` branch, add test coverage for this new code:

- `src/app/news/news.component.ts` -- Lines 57 and 66-71

🚦 See full report on Qlty Cloud »

🛟 Help

• Diff Coverage: Coverage for added or modified lines of code (excludes deleted files). Learn more.

• Total Coverage: Coverage for the whole repository, calculated as the sum of all File Coverage. Learn more.

• File Coverage: Covered Lines divided by Covered Lines plus Missed Lines. (Excludes non-executable lines including blank lines and comments.)

  • Indirect Changes: Changes to File Coverage for files that were not modified in this PR. Learn more.

[hirokiterashima]

Looking great! 👍 Features work as described.

Let's add back the bypassSecurityTrustHtml() bit. This allows the admins to use HTML syntax in their news posts, like adding a link or making text bold. Without it, it will show those html blocks, which is not ideal.

Also, see code improvement suggestions inline. I may not have caught all the places to clean up unused import/code.

[hirokiterashima]

LGTM

[hirokiterashima]

🎉 This PR is included in version 5.234.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀