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

Worked on the backend for anonymous posts, its fully functional and a… #39

Merged
merged 3 commits into from
Oct 9, 2024

Conversation

khalifat3
Copy link
Collaborator

Implemented backend for Anonymous posts, it is fully functional. User can now make a post without appearing as 'Guest'.

  • Modified src/topics/teasear.js, src/topics/posts.js, src/topics/topics.js.
  • Src/topics/create.js - Changing UID to 0, when user declares post to be posted as anonymous.
  • Customizing so it displays a specific name, specific avatar color, and etc. Gives user a sense of freedom.

This enhancement allows the user to create a post anonymously which makes the user comfortable.

Resolves Issue #6

@khalifat3 khalifat3 added enhancement New feature or request backEnd Issues with backend labels Oct 9, 2024
@khalifat3 khalifat3 self-assigned this Oct 9, 2024
@coveralls
Copy link

coveralls commented Oct 9, 2024

Pull Request Test Coverage Report for Build 11256291330

Details

  • 24 of 24 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.02%) to 82.681%

Totals Coverage Status
Change from base Build 11252958939: 0.02%
Covered Lines: 22350
Relevant Lines: 25612

💛 - Coveralls

@Abdallah-Abdaljalil Abdallah-Abdaljalil linked an issue Oct 9, 2024 that may be closed by this pull request
Copy link
Collaborator

@Abdallah-Abdaljalil Abdallah-Abdaljalil left a comment

Choose a reason for hiding this comment

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

  • Added to the idea of an anonymous user posting as a guest
  • Logic seems great, as you are simply styling different attributes of a user object such as the username
  • Code is clean
  • Works fine and passes tests
  • Nice Job

@Abdallah-Abdaljalil Abdallah-Abdaljalil merged commit ae532b7 into f24 Oct 9, 2024
1 check passed
@Abdallah-Abdaljalil Abdallah-Abdaljalil added this to the Sprint 2, Week 2 milestone Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backEnd Issues with backend enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Anonymous Posts Backend Fully Functional
3 participants