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

Feat/257 Argument votes #287

Draft
wants to merge 55 commits into
base: main
Choose a base branch
from
Draft

Feat/257 Argument votes #287

wants to merge 55 commits into from

Conversation

ndharmateja
Copy link
Collaborator

No description provided.

@ndharmateja
add view to upvote an argument
2634385
@ndharmateja
add path to upvote argument
9c54964
@ndharmateja
add todos and comments
505b7b5
@ndharmateja
modify comments in upvote argument view
ac08f02
@ndharmateja
add skeleton view for downvote argument
91cf10d
@ndharmateja
add url corresponding to upvote argument undo
5bb1a34
@ndharmateja
create a function to check existence of a post
9d7ee80
@ndharmateja
add edge cases to upvote argument view
a78daf6
@ndharmateja
implement upvote argument undo view
af9ed78
@ndharmateja
add doc string to get_post fn
24b929f
@ndharmateja
add functions to Vote class
cff92e7
@ndharmateja
create a vote object irrespective of its previous existence
722611c
@ndharmateja
update comment in upvote argument undo
61cd3d6
@ndharmateja
add edge cases for upvoting and downvoting in Vote model
3a7564d
@ndharmateja
fix error handling in get_post fn
2ab12ca
@ndharmateja
handle upvote edge case where the vote is already upvote
6cb998c
@ndharmateja
handle edge cases in upvote argument undo
c119f42 
1. if the vote doesn't exist
2. if the existing vote is a downvote
@ndharmateja
add downvote_argument and downvote_argument_undo fns
28d4a69
@ndharmateja
bind downvote urls to the view fns
70a4fe6
@vercel Vercel
Copy link

vercel bot commented Aug 4, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
rebutify ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 10, 2024 9:22pm

@ndharmateja ndharmateja linked an issue Aug 7, 2024 that may be closed by this pull request
Argument votes #257
Open
5 tasks
@ndharmateja ndharmateja self-assigned this Aug 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@ndharmateja ndharmateja

Labels
[ backend /
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Argument votes
1 participant
@ndharmateja