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.

Sign up for GitHub

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

Jump to bottom

V2 Backend Methods for Pinning A Post #15

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

V2 Backend Methods for Pinning A Post #15

wants to merge 7 commits into from

Conversation

tinchil
Copy link

@tinchil tinchil commented Oct 11, 2024 •
edited
Loading

Address Issue #14

  • Added a backend method Pin & Unpin methods to /src/posts/tools.js
  • Added isPinned and TogglePin methods in /src/posts/tools.js to update the toggle
  • Added backend API routes to call the pin and unpin methods through editing src/routes/api.js
  • Added event_handlers for front-end integration and merging in /public/src/client/topic/postTools.js
  • Passed Linters but unable to pass automated testing and test suites

I would like any feedback or assistance on passing automated testing and working through tested features to fully get this feature working and connected with the front_end.

@tinchil tinchil added the enhancement New feature or request label Oct 11, 2024
@tinchil tinchil self-assigned this Oct 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@tinchil tinchil

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@tinchil