Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Attempt to fix existing bugs in endorsement feature #34

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

Steve0012345
Copy link
Collaborator

This pull request entails most of the changes I made in the client files and controller files in the backend to get the endorsement feature to work. This feature was unfortunately unable to meet the set milestone as it did not live up to all the preset metrics for success. However, on using the endorsement feature, a fetch API is dispatched to the backend to update the 'endorsed_by_Instructor' field in the database. The PR also entails multiple changes to the front-end files in public/src/client/ to modify the click event and dispatch both a hook and a fetch API where the API returns a response indicating the failure or success of the task which is to update the endorsed field of that particular post in the database. While this feature only works to a certain limited point and the milestone for this feature has not been met in full, the overall code used in implementing this feature is tremendously versatile. It can easily be recycled and reused for other tasks in the future.

@Steve0012345 Steve0012345 added enhancement New feature or request Front-end Deals with front-end devlopment Back-end Deals with back-end devlopment labels Oct 5, 2023
@Steve0012345 Steve0012345 self-assigned this Oct 5, 2023
@mrahmatu mrahmatu added merged Completed and merged with main branch and removed merged Completed and merged with main branch labels Oct 5, 2023
GhalyaJ pushed a commit that referenced this pull request Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Back-end Deals with back-end devlopment enhancement New feature or request Front-end Deals with front-end devlopment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants