Xeno852 commented 6 months ago

DO NOT MERGE THIS IS NOT WORKING

PR just to update repo

Pull Request: Implement Topic Locking Feature

Summary

This pull request introduces the ability to lock and unlock topics within the forum. It aims to give topic owners, administrators, and moderators the authority to halt new replies on topics when deemed necessary, crucial for maintaining forum discussions' relevance and orderliness.

Motivation

The addition of this feature is motivated by the need to:

For Moderators and Administrators: Provide enhanced control over forum content by allowing the locking of topics that have been satisfactorily resolved, are no longer relevant, or where discussions have become unconstructive.
For Users: Offer clear indications of which topics are closed to further discussion, thereby directing focus towards active and unresolved topics.

Changes Made

Backend Changes:
- Introduction of a locked boolean field in the Topic model to keep track of the lock status.
- Creation of new API endpoints: /api/v3/topics/:tid/lock and /api/v3/topics/:tid/unlock for managing topic lock status.
- Update of permissions to ensure only owners, admins, or moderators can change the lock status of topics.
Frontend Changes:
- Addition of lock/unlock buttons on topic pages, visible solely to users with appropriate permissions.
- Implementation of visual indicators for locked topics to alert users to their status.
Event Handling:
- Setup of event listeners for lock/unlock button clicks to trigger the relevant API calls and update the UI accordingly.
Manual Testing:
- Creation and locking/unlocking of topics as a standard user, verifying the functionality's availability and effectiveness.
- Examination of lock/unlock button visibility to unauthorized users, ensuring no access to these functionalities.

How to Test

Go to a topic you own or for which you have admin/moderator rights.
Click the "Lock" button to stop new replies; the topic should immediately reflect a locked status.
On a locked topic, click "Unlock" to reopen it for new discussions.

cathyxfeiyang commented 6 months ago

It seems that the unlocking feature is implemented but locking feature is not? I'm not quite sure how this will function -- I don't think this functionality is complete