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

Feat: Add anonymous post and reply functionality #45

Merged
merged 2 commits into from
Feb 27, 2025

Conversation

nysagarg04
Copy link
Contributor

@nysagarg04 nysagarg04 commented Feb 27, 2025

This PR implements the frontend UI for the anonymous post and reply feature. Users can now see two new buttons:
• Anonymous Post button, which opens a post dialogue.
Screenshot 2025-02-27 at 1 14 09 PM

•	Anonymous Reply button, which opens a reply dialogue.
Screenshot 2025-02-27 at 1 14 36 PM

Although the backend logic to handle anonymous posting isn’t fully functional yet (the post still shows the name of the user), the UI successfully persists after a page refresh, ensuring a smoother user experience.

Changes Introduced
• Added buttons for anonymous posting and replying.
• Implemented modal dialogues for submitting anonymous posts and replies.
• Ensured that the buttons remain visible after a page refresh (handled state persistence).

Challenges & Limitations
• Currently, posts do not appear as anonymous due to a complex backend issue.
• I sought assistance from TAs, but the issue requires further debugging.
• The next step is to resolve backend integration to ensure anonymity works correctly.

Next Steps
• Investigate and fix the backend issue preventing anonymous posts from appearing correctly.
• Test and validate the entire flow from UI to backend once the issue is resolved.

@ethan1696
Copy link
Contributor

Looks great! Feel free to merge!

@coveralls
Copy link

Pull Request Test Coverage Report for Build 13572891965

Details

  • 6 of 10 (60.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.002%) to 82.674%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/topics/posts.js 3 4 75.0%
src/controllers/composer.js 1 4 25.0%
Totals Coverage Status
Change from base Build 13571779325: -0.002%
Covered Lines: 22355
Relevant Lines: 25619

💛 - Coveralls

@nysagarg04 nysagarg04 changed the title Anonymous post reply Feat: Add anonymous post and reply functionality Feb 27, 2025
@nysagarg04 nysagarg04 merged commit 9c77b1e into main Feb 27, 2025
2 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants