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

change ForgetUserSince to just ForgetUser #42

Open
zephyrtronium opened this issue Mar 13, 2024 · 0 comments
Open

change ForgetUserSince to just ForgetUser #42

zephyrtronium opened this issue Mar 13, 2024 · 0 comments
Assignees
@zephyrtronium
Milestone
Rewrite

Comments

@zephyrtronium
Copy link
Owner

The caller is the one who needs to arrange the "since" part because message times are encoded in user hashes. The time parameter doesn't add anything except possibly limiting result sets on the SQL brain, which we almost certainly aren't going to use anyway.
@zephyrtronium zephyrtronium added this to the Rewrite milestone Mar 13, 2024
@zephyrtronium zephyrtronium self-assigned this Mar 13, 2024
zephyrtronium added a commit that referenced this issue Mar 27, 2024
@zephyrtronium
brain/*: change ForgetUserSince to ForgetUser
d66156a 
Fixes #42.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zephyrtronium zephyrtronium

Labels
None yet
Projects
None yet
Milestone
Rewrite
Development

No branches or pull requests
1 participant
@zephyrtronium