Afrux Top Posters Widget

SychO

ctml On that front I wish there were some metrics extension that kept all this info and other extensions could pull from.

That sounds like an amazing idea to be honest.

Justoverclock

SychO ctml On that front I wish there were some metrics extension that kept all this info and other extensions could pull from.

Totally agree

Ralkage

ctml Top Posters - 30 Days

What about those months that have 31 days? 😉

ctml

Ralkage I believe the current implementation is calculating post counts over the last 30 days, regardless of how many days are in the current month or last month. That is the reason I think it doesn't make sense to use "this month" and "30 days" might be more appropriate unless of course calculation were changed to only use posts from the current month as we discussed above 😉

m4v3rick

@Justoverclock

Is there something wrong with the order?

SychO

m4v3rick woah, that's odd, I wonder what's happening there, my guess is that the function that transforms the count to a nice format is doing something funky, but can't be certain. Did you try clearing the cache?

Justoverclock

m4v3rick i'm not the developer of this extension, but you're in the right thread, maybe Sycho will reply soon.

edit: lol, he's faster than me

m4v3rick

SychO Did you try clearing the cache?

Just did and didn't work. 😉

m4v3rick

Justoverclock

Sorry...too many extensions 🤪 - I got a bit confused I guess. 😃

SychO

m4v3rick well, I logged the issue afrux/top-posters-widget1 but I'll need to be able to reproduce it first.

m4v3rick

SychO I'll need to be able to reproduce it first

You need to generate 1k posts I guess 😆 .

If you need any help, input or console logs, let me know.

m4v3rick

SychO Don't want to bother you, but any idea when this will be fixed?

m4v3rick

SychO You could aswell sign up in our forum. 🙂

m4v3rick

Hmm works until 1k posts.
Higher than 1k it is still broken:

So it is still the same error:

m4v3rick You need to generate 1k posts I guess 😆 .

SychO

m4v3rick visually seeing it won't help me, that's why I need to try to reproduce it locally, so that I can fiddle in the backend. I have not had time to try reproducing it/looking into it yet.

Fealist

SychO when will it be clickable?

m4v3rick

SychO

Hey, did you reproduce the error already? 🙂

ash3T

Thanks for developing this extension. Will this extension add links to each top poster in the future? So each avatar or name of the user is clickable, and links to his/her profile page. I checked codes and found in top-posters-widget/js/dist/forum.js from line 106

m("div", {
                        className: "Afrux-TopPostersWidget-users-item-avatar",
                    },

Found the following code in other forum.js

href: e ? je.route.user(e) : "#"

But I don't know how to make it work.

VidhunNagarajan

ash3T Please implement this!

joanaranda

@SychO, is there the chance that you consider adding links to each poster profile in the future, as suggested at ash3T time ago?

It looks like linking to the profile is a quite standard practice in the core of Flarum and most extensions. In my particular use case, additionally, your extension is just above the Last Registered Users extension which indeed has those links, making the overall experience a little bit stange. It is hard to explain to the regular user why some usernames can be clicked while the other ones cannot.

murdocklawless

it doesn't show up after 1.50 update?

m4v3rick

murdocklawless

Looks good atm, true.

VidhunNagarajan

m4v3rick Yes it works on my small forum too.

« Previous Page Next Page »