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.

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

AO3-3866 update revised_at when a work is revealed #4664

Conversation

eliahhecht
Copy link
Contributor

Issue

https://otwarchive.atlassian.net/browse/AO3-3866 (Please fill in issue number and remove this comment.)

Purpose

When a work changes from unrevealed to revealed, update its revised_at date, so that it sorts according to when it was revealed, rather than when it was originally posted.

Testing Instructions

Testing instructions in the issue should confirm the fix.

Credit

Eliah Hecht, he/him

app/models/work.rb Outdated Show resolved Hide resolved
@EchoEkhi
Copy link
Contributor

I thought the current plan is to re-do the whole thing instead?

@eliahhecht
Copy link
Contributor Author

eliahhecht commented Nov 18, 2023

I thought the current plan is to re-do the whole thing instead?

Yes, I'm aware of that. However, this issue was bothering my girlfriend (who has a long-running challenge that is affected by it) and I thought this might be a good quick fix; I'm not sure how long AO3-4546 might take to be implemented. But if this looks like adding technical debt and the maintainers don't want to do that, I would understand and we could cancel the PR.

@brianjaustin
Copy link
Member

@eliahhecht I think we'd prefer to collapse the timestamp fields as described in the epic @EchoEkhi linked. I'm going to close this PR for now so it doesn't take 1 of your 5 slots, but feel free to continue discussing here!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants