feat(blog): add related posts #976
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Tagging OptionsShould a new tag be published when this PR is merged?
|
|
Have you a link to admin were this is working? I want to see how the selection of |
hello, Luis! it would be like this:
|
matheusgr
requested changes
Jan 6, 2025
blog/utils/handlePosts.ts
Outdated
| ) { | ||
| const filteredPosts = filterPosts(posts, slug, term); | ||
| const filteredPosts = filterPosts(posts, slug, term, excludePostSlug); |
There was a problem hiding this comment.
excludePostSlug is only being applied to filterRelatedPosts.
Filter this post slug from any result:
Suggested change
| const filteredPosts = filterPosts(posts, slug, term, excludePostSlug); | |
| const filteredPosts = filterPosts(posts, slug, term).filter( | |
| ({ slug: postSlug }) => postSlug !== excludePostSlug) | |
| ); |
blog/loaders/BlogRelatedPosts.ts
Outdated
Comment on lines
1
to
8
| /** | ||
| * Retrieves a list of blog related posts. | ||
| * | ||
| * @param props - The props for the blog related post list. | ||
| * @param req - The request object. | ||
| * @param ctx - The application context. | ||
| * @returns A promise that resolves to an array of blog related posts. | ||
| */ |
There was a problem hiding this comment.
JSDoc should be on the exported default function.
Comment on lines
+50
to
+58
| /** | ||
| * @title BlogRelatedPosts | ||
| * @description Retrieves a list of blog related posts. | ||
| * | ||
| * @param props - The props for the blog related post list. | ||
| * @param req - The request object. | ||
| * @param ctx - The application context. | ||
| * @returns A promise that resolves to an array of blog related posts. | ||
| */ |
There was a problem hiding this comment.
Suggested change
| /** | |
| * @title BlogRelatedPosts | |
| * @description Retrieves a list of blog related posts. | |
| * | |
| * @param props - The props for the blog related post list. | |
| * @param req - The request object. | |
| * @param ctx - The application context. | |
| * @returns A promise that resolves to an array of blog related posts. | |
| */ |
matheusgr
approved these changes
Jan 6, 2025
vitoUwu
pushed a commit
that referenced
this pull request
Jan 30, 2025
* feat(blog): add related posts * refactor(blog): props loader related posts * feat(blog): jsdoc and refactors * chore(deno): eslint
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What is this contribution about?
This PR was made to add the option to search related posts.
The search is carried out according to the category/categories of the post.
An optional "excludeSlug" prop was also added to exclude the current post from the return list.
Example: