Skip to content

Chore/cleanup postcard todos#322

Open
SarthakDudhe wants to merge 8 commits into
QuoteVote:mainfrom
SarthakDudhe:chore/cleanup-postcard-todos
Open

Chore/cleanup postcard todos#322
SarthakDudhe wants to merge 8 commits into
QuoteVote:mainfrom
SarthakDudhe:chore/cleanup-postcard-todos

Conversation

@SarthakDudhe

Copy link
Copy Markdown

This PR cleans up the PostCard.jsx file.

Changes:

Removed the legacy // TODO: show quote up/down comment as the feature is now fully implemented and active in the UI.

…ied HTML

Modified getPost resolver to support urlId and hardened edge function regexes. Fixes QuoteVote#230
Implements the UI for showing quote-specific votes calculated from the interactions array. Added smaller icons and tooltips for clarity.
Added href properties to Link components and removed ESLint suppressions. Fixed a typo on the title link.
Implemented a point system based on posts, quotes, and votes. Fixed a database query bug for user votes.
…ation hook

Created useTokenValidation hook and refactored Scoreboard and PasswordResetPage to use it. This improves code reuse and maintainability.
Updated resolver to check the success flag from sendGridEmail utility and removed legacy TODO.
Updated FollowButton to calculate follow status internally from global state. Removed legacy TODO.
Removed the outdated TODO comment since the quote up/down vote display is already implemented.
@netlify

netlify Bot commented May 2, 2026

Copy link
Copy Markdown

Deploy Preview for quotevote ready!

Name Link
🔨 Latest commit be5ed96
🔍 Latest deploy log https://app.netlify.com/projects/quotevote/deploys/69f5f37c8cafa800090672f7
😎 Deploy Preview https://deploy-preview-322--quotevote.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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.

1 participant