wjs018

joined 1 month ago
[–] wjs018@piefed.wjs018.xyz 8 points 1 week ago* (last edited 1 week ago)

And PieFed also supports blurring photos using spoiler flair instead of just relying on NSFW tags as Lemmy does.

This and || inline spoilers || were some of my early contributions to Piefed because I was so frustrated that they weren't in lemmy.

[–] wjs018@piefed.wjs018.xyz 5 points 1 week ago (1 children)

They're different and have relative strengths and weaknesses. Despite being a contributor to piefed, I moderate a handful of communities on a lemmy instance and don't really have intentions of moving them. Overall, piefed is a lot less "mature" than lemmy, which makes sense because it is a lot newer of a project. It is getting better now about being more stable, but there are plenty of things that are still changing quite rapidly, especially on the api front.

As for python/pip/typing, I just don't see this as the major issue that some people seem to think it is. We aren't adding dependencies willy-nilly and the framework in which we are working (flask) is a very mature one that has stood the test of time. The fact that python is used for the project has tremendously helped the project in that it has allowed for a large number of contributions from many different people. Frankly, if piefed wasn't in python, it would not be nearly as feature complete as it is now thanks to the wide range of contributions we have received from folks.

[–] wjs018@piefed.wjs018.xyz 7 points 1 week ago (1 children)

the mobile implementation of the website is surprisingly good

This makes me feel good. I have done quite a bit of reworking the UI to make it more friendly at mobile screen sizes. Thanks!

[–] wjs018@piefed.wjs018.xyz 2 points 2 weeks ago (1 children)

Either in this thread or simply by pm works. The blahaj admins have been helpful in trying to debug this, so the most useful thing for me would be the number that is in that url that goes nowhere, 15179 in your example. That is the id of the notification that lets me get more info about it from them.

[–] wjs018@piefed.wjs018.xyz 3 points 2 weeks ago (3 children)

Alright...I don't have a solution for you, but I am going to try some stuff and see if I can replicate this. After talking with some of the involved admins, I don't really know the problem, but I have some leads to follow up on. I was able to confirm it is related to reports, but why you are the one getting a notification, I am not sure yet.

If this happens again, feel free to reach out.

[–] wjs018@piefed.wjs018.xyz 3 points 2 weeks ago (5 children)

Out of curiosity, what is one of the urls it is trying to link you to that ends up 404'ing?

Final question, have you reported any posts/communities/users recently?

I am wondering if you are getting a notification that your report was reviewed, but because the content was removed, the link is just 404'ing. I honestly don't know if those kinds of notifications are a thing, but notifications linking to deleted content only to 404 is a known issue that we are working on.

[–] wjs018@piefed.wjs018.xyz 3 points 2 weeks ago (7 children)

I'll have a go and see if I can replicate this. Would you be able to share a screenshot of what the notification looks like for you?

[–] wjs018@piefed.wjs018.xyz 2 points 2 weeks ago* (last edited 2 weeks ago) (9 children)

Are these new notifications you have gotten? Or are they old notifications that now have broken links? It's possible that older notifications that are hanging around in your inbox are lacking all the information that we now include with notifications, so there could be broken links from that.

It looks like blahaj is running a very recent build of the code, so if these are new notifications, it must be something else. It doesn't look like you are a moderator or admin, so you shouldn't be getting reports as notifications. It could be that somewhere in the code it is sending the wrong notification type. There isn't too much else I can help with without getting some lower level information.

@rimu@piefed.social - It would be really helpful tracking this down to figure out what type of notification these are saved in the db as and any other targets saved along with them. A quick search in the code and I don't see any type 20 or 40 notifications where they shouldn't be.