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

Backend implementation of isAnswered feature, including isAnswered field in posts, storing answered status, adding logic/property to edit post data, and API endpoints #27

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

Conversation

eunseokk
Copy link

@eunseokk eunseokk commented Sep 25, 2024

Issue #15
Backend implementation including:

  • Adding isAnswered field to posts
  • Storing answered status for post
  • Adding isAnswered property into get Edit Post
  • Adding logic to handle isAnswered in posts.edit
  • Adding controller and routes (INCOMPLETE AND NOT PASSING)

Current Status:

  • There is a error in creating the API endpoint, the tests are not running and the errors are in areas I can not seem to figure out how to debug. Tests will be added, but this does not solve the inherent issue.

Potential Next Steps:

  • Debugging the API endpoint and integration into frontend
  • Adding additional testing for the backend

Would like feedback for the commit up to fd84e7b

@VeronicaPim VeronicaPim force-pushed the f24 branch 2 times, most recently from bf0fcb8 to 29c75ea Compare October 9, 2024 21:45
Copy link

@VeronicaPim VeronicaPim left a comment

Choose a reason for hiding this comment

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

This part is good in fixing the linter errors.

VeronicaPim

This comment was marked as duplicate.

VeronicaPim

This comment was marked as resolved.

@eunseokk eunseokk changed the title IsAnswered Feature (Backend) Backend implementation of isAnswered feature, including isAnswered field in posts, storing answered status, adding logic/property to edit post data, and API endpoints Oct 9, 2024
Copy link

@VeronicaPim VeronicaPim left a comment

Choose a reason for hiding this comment

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

actually you need to make sure to remove trailing spaces in this file - for example in line 2533:1 here.

@ericlin2 ericlin2 added this to the Sprint 2 milestone Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants