[Bug]: Advertisements no longer appear #9747

Closed
opened 3 months ago by daikei · 10 comments

What happened?

Advertisements, no matter what type, do not appear on the feed.

What did you expect to happen?

An Ad would be displayed on the feed.

Version

Docker/13.2.0-Beta2

Instance

kitsunes.gay

What browser are you using?

Firefox

Relevant log output

No response

Contribution Guidelines

  • I agree to follow this project's Contribution Guidelines
### What happened? Advertisements, no matter what type, do not appear on the feed. ### What did you expect to happen? An Ad would be displayed on the feed. ### Version Docker/13.2.0-Beta2 ### Instance kitsunes.gay ### What browser are you using? Firefox ### Relevant log output _No response_ ### Contribution Guidelines - [x] I agree to follow this project's Contribution Guidelines
panos commented 3 months ago
Owner

Indeed, and @yawhn has already prepared a PR that fixes this, but was getting a 500 error when trying to upload it. It was probably Codeberg acting up again, so I hope he'll be able to upload it soon so it will be merged in dev.

Indeed, and @yawhn has already prepared a PR that fixes this, but was getting a 500 error when trying to upload it. It was probably Codeberg acting up again, so I hope he'll be able to upload it soon so it will be merged in dev.
yawhn commented 3 months ago

The fix is in this commit/branch if you like to use it: 94c0ebf689

Unfortunately, I tried to create a PR into develop branch, but I receive the following error:

image

The fix is in this commit/branch if you like to use it: https://codeberg.org/yawhn/elreqkey/commit/94c0ebf6893181b8c58ef38ee643c4931d8f8472 Unfortunately, I tried to create a PR into develop branch, but I receive the following error: ![image](/attachments/b2ed9888-f78e-4845-9806-67f02c742649)
135 KiB
Poster

Unfortunately my systems are all dockerized. :'

Unfortunately my systems are all dockerized. :'
panos commented 3 months ago
Owner

Unfortunately my systems are all dockerized. :'

Alright, we might do a beta3 relatively soon, if we see that we need more time for the RC.

> Unfortunately my systems are all dockerized. :' > Alright, we might do a beta3 relatively soon, if we see that we need more time for the RC.
panos added the
🐛Bug
label 3 months ago

Cherry-picked: 25c72943c7

Cherry-picked: https://codeberg.org/calckey/calckey/commit/25c72943c7e07b11f1c8319a67fb1bb03bc3ca36
thatonecalculator closed this issue 3 months ago
Poster

Reopening, it still appears I am without ads on kitsunes.gay - which had ads previously, and with a check using an (up to this point) adless instance, kitsunes.club.

Edit for Context
This is now with all instances on beta3.1.
Note: this is explicitly for inline advertismenets. Widget advertisements are fine.

Reopening, it still appears I am without ads on kitsunes.gay - which had ads previously, and with a check using an (up to this point) adless instance, kitsunes.club. Edit for Context This is now with all instances on beta3.1. Note: this is explicitly for inline advertismenets. Widget advertisements are fine.
daikei reopened this issue 3 months ago

Please check that your ads haven't expired.

Please check that your ads haven't expired.
Poster

Nope, I set them for 2555.

Nope, I set them for 2555.
panos commented 3 months ago
Owner

Assigned to @yawhn

Assigned to @yawhn
yawhn commented 2 months ago

The fix is in this commit/branch if you like to use it: 94c0ebf689

Unfortunately, I tried to create a PR into develop branch, but I receive the following error:

image

I continue to experience the same error when creating a PR into develop branch. I don't know why.

Reopening, it still appears I am without ads on kitsunes.gay - which had ads previously, and with a check using an (up to this point) adless instance, kitsunes.club.

Edit for Context
This is now with all instances on beta3.1.
Note: this is explicitly for inline advertismenets. Widget advertisements are fine.

I have fixed the issue with inline ads. Until I find a way to overcome the PR error, I am sharing with you the commit which fixes it: c7ab9c44cb

> The fix is in this commit/branch if you like to use it: https://codeberg.org/yawhn/elreqkey/commit/94c0ebf6893181b8c58ef38ee643c4931d8f8472 > > Unfortunately, I tried to create a PR into develop branch, but I receive the following error: > > ![image](/attachments/b2ed9888-f78e-4845-9806-67f02c742649) I continue to experience the same error when creating a PR into develop branch. I don't know why. > Reopening, it still appears I am without ads on kitsunes.gay - which had ads previously, and with a check using an (up to this point) adless instance, kitsunes.club. > > Edit for Context > This is now with all instances on beta3.1. > Note: this is explicitly for inline advertismenets. Widget advertisements are fine. I have fixed the issue with inline ads. Until I find a way to overcome the PR error, I am sharing with you the commit which fixes it: https://codeberg.org/yawhn/elreqkey/commit/c7ab9c44cb0795d673590ab37492c7c1eb9370bc
thatonecalculator referenced this issue from a commit 1 month ago
daikei closed this issue 3 weeks ago
Sign in to join this conversation.
No Milestone
No project
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: calckey/calckey#9747
Loading…
There is no content yet.