CMU-313 / nodebb-fall-2025-level-up / 18264614226

78%

push

github

Feature: Hide private posts from non-staff users across category listings (#25)

* Feature: Hide private posts from non-staff users across category listings (#23)

* feat(topics): add support for persisting private flag on topic creation

Extended backend topics API to persist private field in Redis.

* feat(categories): hide private topics from non-staff in category views

Added filtering logic in getCategoryTopics to exclude private topics
for guests and non-staff users. Admins and moderators can still view
these topics. This ensures that private posts are not visible in
category listings such as Announcements.

* fix(categories): adjust category topic/post counts for private visibility

Pass uid through categoriesController.list -> getCategories so that
topic and post counts respect private flag visibility. Non-staff
users no longer see inflated totals when private topics exist.

* fix(categories): hide private topic teasers from category previews

Ensure that private topics do not appear in home page category teasers
for non-staff users. Teasers are now filtered in recentreplies.js
based on the topic private flag, while still showing for admins
and moderators.

* fix(lint): resolve linter issues in categories and topics modules

- Refactor categories/index.js to avoid await inside loop by batching promises
- Fix operator-linebreak errors in categories/topics.js and topics/index.js
- Add missing privileges import in topics/recent.js to resolve no-undef

* fix(lint): resolved indentation error

* Fix: Resolve npm test schema errors

* Fix: Edited response body to match schema definitions in additional files

* feature/fix(ui): Updated instructor tagline UI (#19)

fixed failing npm run test by removing isAdmin flag
    from user object so it doesn't fail schema validation.
    Instead injected html as part of already existing
    profile info into template for seamless integration.

* test(controllers): fix category pagination test by making user adm... (continued)

11190 of 16092 branches covered (69.54%)

Branch coverage included in aggregate %.

83 of 92 new or added lines in 9 files covered. (90.22%)

24764 of 29683 relevant lines covered (83.43%)

1490.93 hits per line