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

Anonymous posting backend #22

Closed
wants to merge 3 commits into from

Conversation

Nalseaf
Copy link

@Nalseaf Nalseaf commented Sep 25, 2024

Modified src/posts/create.js file to initiate an anonymous value
then Modified to check for anonymity in the src/topics/posts.js file to then set the username and displayname to "Anonymous"

closes #6

@Nalseaf Nalseaf added this to the Sprint 1 milestone Sep 25, 2024
@Nalseaf Nalseaf self-assigned this Sep 25, 2024
@mthani2 mthani2 added the bug Something isn't working label Sep 25, 2024
Copy link

@mthani2 mthani2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like there is an issue with the db schema cause a failure in the node test. We need to update our schema definition in the test to include "anonymous".
Screenshot 2024-09-25 at 10 32 56 PM
This connects to issue #4

Copy link

@sarrakhelifi-cmu sarrakhelifi-cmu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have seen the issues passed by in terms of Node, this definitly connects to what I've been working on with issue #4. I will be working on this now.

@mthani2
Copy link

mthani2 commented Sep 30, 2024

Since this issue has the "help wanted" label, I will now be taking on this issue moving into sprint 2 making all the changes necessary to resolve issue #4 and issue #6, to progress towards closing the user story #2.

These changes will be made on the new branch 'anonymous-post-full-feature'.

@mthani2 mthani2 changed the title Anonymous posting backend anonymity check Anonymous posting backend Sep 30, 2024
@mthani2 mthani2 mentioned this pull request Sep 30, 2024
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backend bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

User Story 1: Implement Backend Logic for Handling Anonymous Posts.
3 participants