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

Remove the ad section #55

Merged
merged 1 commit into from
Jun 20, 2024
Merged

Conversation

Ilyeo
Copy link
Contributor

@Ilyeo Ilyeo commented Jun 18, 2024

Quick Info

We need to remove the ad section located underneath the post section
because the image appears broken.

I've noticed that the image source is from Dropbox, so it's likely that
we've lost access to that content or the image has been removed.

I need to confirm whether this removal is temporary or permanent.

Screenshots

Before After
image image

@magmaleads magmaleads temporarily deployed to today-i-lear-chore-remo-1rzbyr June 18, 2024 20:21 Inactive
@Ilyeo
Copy link
Contributor Author

Ilyeo commented Jun 18, 2024

@vicmaster

Could you please confirm if this change is permanent?

@Ilyeo Ilyeo requested a review from vicmaster June 18, 2024 20:28
We need to remove the ad section located underneath the post section
because the image appears broken.

I've noticed that the image source is from Dropbox, so it's likely that
we've lost access to that content or the image has been removed.

I need to confirm whether this removal is temporary or permanent.
@Ilyeo Ilyeo force-pushed the chore/remove-the-ad-section-from-the-post-view branch from fe3f8db to 3e7585a Compare June 20, 2024 04:32
@magmaleads magmaleads temporarily deployed to today-i-lear-chore-remo-s1yu4u June 20, 2024 04:33 Inactive
Copy link
Member

@vicmaster vicmaster left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@Ilyeo Ilyeo merged commit 464b81e into main Jun 20, 2024
1 check passed
@Ilyeo Ilyeo deleted the chore/remove-the-ad-section-from-the-post-view branch June 20, 2024 17:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants