CMU-313 / spring24-nodebb-apple

spring24-nodebb-apple created by GitHub Classroom
https://spring24-nodebb-apple-ibtybkwnba-uc.a.run.app/
GNU General Public License v3.0
0 stars 1 forks source link

endorsement feature #27

Closed takuel37 closed 6 months ago

takuel37 commented 7 months ago

fixed the files to allow anonymity handling as well as fixing npm run lint

Added features to show endorsement Added instructor role for special permissions Allowed instructors to endorse posts

I implemented the endorsement feature's front-end. Users are now able to endorse posts and it will have a tag when the page is refreshed. I could not implement the back-end so only instructors could do it. I struggled with the test cases.

Summary

Implemented the endorsement button feature for post frontend, allowing posts to be endorsed. Added endorsement tag visibility to posts upon endorsement. Established front-end event handling for endorsement button clicks. Added controller functions to manage endorsement status updates with proper permissions. Motivation The endorsement feature is inspired by the effectiveness of similar functionalities in educational platforms:

As a student, it allows me to highlight posts that are insightful or particularly helpful. As an instructor, it enables me to easily identify and endorse valuable contributions, enhancing the learning experience for everyone. Implementing this feature supports a more interactive and engaging forum environment, encouraging quality contributions and facilitating easier navigation through endorsed content. Changes Made

Introduced new endorsement functionality to enhance post interaction. Testing Done Visual Testing

Tested endorsing a post, verifying that the endorsement tag is correctly displayed upon page refresh. Tested with an unlogged user, confirming that the endorsement feature is not visible or usable without authentication. Logged in as an instructor, ensured that endorsement functionality is available and operates as expected. Automated Testing

Failed to do automated testing Next Steps

A valuable enhancement would be to introduce a filter mechanism in the forum to allow users to sort or view posts based on their endorsement status, further improving user experience and content discoverabili

coveralls commented 6 months ago

Pull Request Test Coverage Report for Build 8033933179

Details


Files with Coverage Reduction New Missed Lines %
src/user/approval.js 3 87.39%
src/user/jobs.js 4 68.63%
<!-- Total: 7 -->
Totals Coverage Status
Change from base Build 7875174970: -0.02%
Covered Lines: 22634
Relevant Lines: 25370

💛 - Coveralls