Follow / unfollow thread actions should bump its last activity date & time + minor general fixes#1931

Open

Oaphi wants to merge 24 commits intodevelopfrom

0valt/1930/comments-last-activity

Commits on Dec 21, 2025

switched comments last_activity_at to a column
Oaphi
committed

Commits on Dec 22, 2025

switched comment threads last_activity_at to a column
Oaphi
committed
added last_activity wrapper method to Comment (max of created, updated, or last activity)
Oaphi
committed
renamed last_activity_at CommentThread method to last_activity & ensured it accounts for the last_activity_at column
Oaphi
committed
made bump_last_activity_at methods public to be able to bump by unrelated actions
Oaphi
committed
switched 'thread_content' freshness check to last_activity & added the same to 'thread' to match
Oaphi
committed
creating/destroying ThreadFollowers should bump their threads last activity timestamp
Oaphi
committed
renamed all *_last_activity_at methods to *_last_activity
Oaphi
committed
[unrelated] fixed test description for the pings method
Oaphi
committed
[unrelated] fixed User#anonymize confusing inverted persist_changes param behavior due to rename from 'dirty'
Oaphi
committed
added tests for Comment#last_activity method
Oaphi
committed
reworked CommentThread#last_activity_at test into the new CommentThread#last_activity test
Oaphi
committed
added CommentThread#add_follower QoL method & expanded last_activity test accordingly
Oaphi
committed
added CommentThread#remove_follower QoL method & expanded last_activity test accordingly
Oaphi
committed
CommentThread#bump_last_activity should have the option to persist changes
Oaphi
committed
Comment#bump_last_activity should have the option to persist changes
Oaphi
committed
CommentThread#remove_follower should remove all followers
Oaphi
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

Follow / unfollow thread actions should bump its last activity date & time + minor general fixes#1931

Follow / unfollow thread actions should bump its last activity date & time + minor general fixes#1931
Oaphi wants to merge 24 commits intodevelopfrom
0valt/1930/comments-last-activity

Commits on Dec 21, 2025

switched comments last_activity_at to a column

Commits on Dec 22, 2025

switched comment threads last_activity_at to a column

added last_activity wrapper method to Comment (max of created, updated, or last activity)

renamed last_activity_at CommentThread method to last_activity & ensured it accounts for the last_activity_at column

made bump_last_activity_at methods public to be able to bump by unrelated actions

switched 'thread_content' freshness check to last_activity & added the same to 'thread' to match

creating/destroying ThreadFollowers should bump their threads last activity timestamp

renamed all _last_activity_at methods to _last_activity

[unrelated] fixed test description for the pings method

[unrelated] fixed User#anonymize confusing inverted persist_changes param behavior due to rename from 'dirty'

added tests for Comment#last_activity method

reworked CommentThread#last_activity_at test into the new CommentThread#last_activity test

added CommentThread#add_follower QoL method & expanded last_activity test accordingly

added CommentThread#remove_follower QoL method & expanded last_activity test accordingly

CommentThread#bump_last_activity should have the option to persist changes

Comment#bump_last_activity should have the option to persist changes

CommentThread#remove_follower should remove all followers

Commits on Dec 23, 2025

add_follower should prevent duplicate thread follower records

moved unfollowing coment threads from unrestrict_thread to its own unfollow_thread action

let's not fail if add_follower / remove_follower is a noop

added job for cleaning up duplicate thread followers

even if there's nothing to do, bump thread's last activity to improve user experience

added tests covering CommentThread#remove_follower last activity bumping

thread wrapper can be absent at the time of handling follow/unfollow success when clicking too fast

Uh oh!

Follow / unfollow thread actions should bump its last activity date & time + minor general fixes#1931Oaphi wants to merge 24 commits intodevelopcodidact/qpixel:developfrom 0valt/1930/comments-last-activitycodidact/qpixel:0valt/1930/comments-last-activityCopy head branch name to clipboard

Commits

Commits on Dec 21, 2025

Commits on Dec 22, 2025

Commits on Dec 23, 2025

Follow / unfollow thread actions should bump its last activity date & time + minor general fixes#1931
Oaphi wants to merge 24 commits intodevelopfrom
0valt/1930/comments-last-activity