Skip to content

Commit

Permalink
Remove checks that only adds RSS feed link on home and post pages
Browse files Browse the repository at this point in the history
I actually don't really see a reason for this. Maybe you wouldn't want it on a documentation site, but I don't think that complexity is warranted now, and it's not within the 80% use case.
  • Loading branch information
caendesilva committed Jul 31, 2022
1 parent edaf49c commit a93e300
Showing 1 changed file with 1 addition and 15 deletions.
16 changes: 1 addition & 15 deletions packages/framework/src/Concerns/HasPageMetadata.php
Original file line number Diff line number Diff line change
Expand Up @@ -82,21 +82,7 @@ public function canUseCanonicalUrl(): bool

public function canUseRssFeedLink(): bool
{
if (RssFeedService::canGenerateFeed() && isset($this->slug)) {
if ($this instanceof MarkdownPost) {
return true;
}

if (str_starts_with($this->getCurrentPagePath(), 'post')) {
return true;
}

if ($this->getCurrentPagePath() === 'index') {
return true;
}
}

return false;
return RssFeedService::canGenerateFeed();
}

public function hasTwitterTitleInConfig(): bool
Expand Down

0 comments on commit a93e300

Please sign in to comment.