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

Correcting backend logic for the API and Controller files #13

Open
wants to merge 2 commits into
base: f24
Choose a base branch
from

Conversation

Nita242004
Copy link

@Nita242004 Nita242004 commented Sep 25, 2024

  • Corrected backend logic for API file. Instead of all replies being flagged as anonymous, only those where the checkbox is ticked are flagged
  • Corrected the backend logic for the controller file. Instead of the username being changed to anonymous for all replies, only ones where the checkbox is ticked are changed. The correct user fields are also changed so it can be used in the front end.

Effectively Resolved:
#12

Screenshot 2024-09-23 at 4 10 07 AM

The payload is changed correctly when the anonymous flag is true and the correct functions are called in the controller.

Screenshot 2024-09-23 at 4 09 49 AM

The postObj fields are also changed to anonymous.

@Nita242004 Nita242004 added the Anonymous feature This series of PRs/Issues are relating to the anonymous feature label Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Anonymous feature This series of PRs/Issues are relating to the anonymous feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants