Skip to content

148 documentacion#149

Merged
JonayKB merged 6 commits into
mainfrom
148-documentacion
May 16, 2025
Merged

148 documentacion#149
JonayKB merged 6 commits into
mainfrom
148-documentacion

Conversation

@JonayKB
Copy link
Copy Markdown
Owner

@JonayKB JonayKB commented May 16, 2025

No description provided.

@JonayKB JonayKB requested a review from Copilot May 16, 2025 17:47
@JonayKB JonayKB linked an issue May 16, 2025 that may be closed by this pull request
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the project documentation and improves the comment repository query for the application. Key changes include:

  • A comprehensive update to README.md with added sections for usage, Docker setup, and endpoints.
  • An updated custom Neo4j query in the comment repository to retrieve comments by post ID.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
README.md Expanded documentation with improved navigation and detailed usage instructions, including Docker and endpoint usage.
API/src/main/java/es/iespuertodelacruz/routinefights/comment/infrastructure/adapters/secondary/repositories/ICommentEntityRepository.java Updated repository method to include a custom query for fetching comments by post ID using Neo4j.
Comments suppressed due to low confidence (1)

API/src/main/java/es/iespuertodelacruz/routinefights/comment/infrastructure/adapters/secondary/repositories/ICommentEntityRepository.java:14

  • Ensure that using elementId(p) for matching is optimal for your dataset; if this query is executed frequently, consider verifying that appropriate indexes or constraints are in place to maintain performance.
@Query("MATCH (u:User)-[r:Commented]->(c:Comment)-[r2:On]->(p:Post) WHERE elementId(p) = $postID RETURN c,r2,p,u,r")

Comment thread README.md

#### Setup your Neo4j

Setup your Neo4J or by the client.
Copy link

Copilot AI May 16, 2025

Choose a reason for hiding this comment

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

[nitpick] Consider clarifying this instruction to ensure readers understand whether they should set up a local Neo4j instance or use an external client; a more descriptive phrasing would improve usability.

Copilot uses AI. Check for mistakes.
@JonayKB JonayKB merged commit 79479cd into main May 16, 2025
1 check passed
JonayKB added a commit that referenced this pull request Jan 24, 2026
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.

Documentacion:

2 participants