Skip to content

Implemented duplicate report detection and integration#104

Merged
jan-elia-24 merged 6 commits into
feature/duplicate-setupfrom
feature/duplicate-implementation
Dec 15, 2025
Merged

Implemented duplicate report detection and integration#104
jan-elia-24 merged 6 commits into
feature/duplicate-setupfrom
feature/duplicate-implementation

Conversation

@0x0xoscar
Copy link
Copy Markdown

This PR builds upon my previous PR #52 and implements the actual logic of duplicate detection.

Added:

  • DuplicateDetectionService to identify potential duplicate reports
  • Repository query to fetch recent reports of the same type, status, and time it was submitted
  • Distance based duplicate filtering using GeoUtils
  • Integration of duplicate detection into report creation
  • Unit tests covering both duplicate and non-duplicate scenarios

Notes:

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Dec 15, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/duplicate-implementation

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@jenkins-cd-for-zipp
Copy link
Copy Markdown

Jenkins Build #1 Summary (for PR #104)

  • Status: SUCCESS
  • Duration: 1 min 29 sec
  • Branch: PR-104
  • Commit: 5a7c2fe
  • Docker Image: 192.168.0.82:5000/zipp:5a7c2fe (pushed to registry)

Details:

  • Checkout: Successful
  • Build & Scan: Passed
  • Push: Successful

All stages passed—no issues detected.

@jenkins-cd-for-zipp
Copy link
Copy Markdown

Jenkins Build #2 Summary (for PR #104)

  • Status: SUCCESS
  • Duration: 1 min 31 sec
  • Branch: PR-104
  • Commit: 2753baa
  • Docker Image: 192.168.0.82:5000/zipp:2753baa (pushed to registry)

Details:

  • Checkout: Successful
  • Build & Scan: Passed
  • Push: Successful

All stages passed—no issues detected.

@jenkins-cd-for-zipp
Copy link
Copy Markdown

Jenkins Build #3 Summary (for PR #104)

  • Status: SUCCESS
  • Duration: 1 min 31 sec
  • Branch: PR-104
  • Commit: b163782
  • Docker Image: 192.168.0.82:5000/zipp:b163782 (pushed to registry)

Details:

  • Checkout: Successful
  • Build & Scan: Passed
  • Push: Successful

All stages passed—no issues detected.

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@kai-ericson kai-ericson left a comment

Choose a reason for hiding this comment

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

The code is clean and easy to follow, well done! Should it be merged directly into main instead of into duplicate-setup maybe?

Copy link
Copy Markdown
Contributor

@jan-elia-24 jan-elia-24 left a comment

Choose a reason for hiding this comment

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

Approved — solid work overall. The changes are easy to understand and don’t introduce any unnecessary complexity.

@jan-elia-24 jan-elia-24 merged commit 7f7ff31 into feature/duplicate-setup Dec 15, 2025
8 checks passed
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