Skip to content

Conversation

@jonaharagon
Copy link
Contributor

So I can have multiple top contributor blocks (with different periods, order parameters, whatever)

@pmusaraj
Copy link
Contributor

Sorry @jonaharagon, closing this due to inactivity.

@pmusaraj pmusaraj closed this Feb 26, 2025
@jonaharagon
Copy link
Contributor Author

Okay, I don't know what activity you were hoping for.

@pmusaraj
Copy link
Contributor

Oh, I meant lack of activity on our side, the maintainers. Are you using this code successfully in production currently?

@jonaharagon
Copy link
Contributor Author

Yes, so I can have both a "Most liked this month" and an "All-time top contributors" block on https://discuss.privacyguides.net/

@pmusaraj pmusaraj reopened this Feb 26, 2025
@pmusaraj
Copy link
Contributor

Cool, reopened. Any chance you can address the merge conflicts? If not, no problem, I can have a look, just let me know. Thanks @jonaharagon

@pmusaraj pmusaraj self-requested a review February 26, 2025 22:52
@jonaharagon
Copy link
Contributor Author

Sure, I should update my fork anyways. I'm running this latest commit in production now 👍

Copy link
Contributor

@pmusaraj pmusaraj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, merging. Better extremely late than never, eh?

@pmusaraj pmusaraj merged commit 5843d50 into discourse:main Feb 27, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants