Skip to content

2019 enhancement backend endpoint for hash heatmap#2068

Open
jessevz wants to merge 6 commits into
devfrom
2019-enhancement-backend-endpoint-for-hash-heatmap
Open

2019 enhancement backend endpoint for hash heatmap#2068
jessevz wants to merge 6 commits into
devfrom
2019-enhancement-backend-endpoint-for-hash-heatmap

Conversation

@jessevz
Copy link
Copy Markdown
Contributor

@jessevz jessevz commented May 12, 2026

closes #2019

@jessevz
Copy link
Copy Markdown
Contributor Author

jessevz commented May 13, 2026

hashtopolis/python-hashtopolis#4 Should be merged to pass the tests

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

Adds a new APIv2 helper endpoint intended to move “cracks per day” aggregation (used for hash heatmaps) from the frontend to the backend (closes #2019).

Changes:

  • Added GET /api/v2/helper/getCracksPerDay which returns a date→count mapping of cracks for the current year.
  • Registered the new helper endpoint in the APIv2 router.
  • Added CI tests validating the new helper’s response shape and basic behavior.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File Description
src/inc/apiv2/helper/GetCracksPerDayHelperAPI.php Implements the new backend aggregation endpoint for cracked hashes per day.
src/api/v2/index.php Registers the new helper endpoint so it becomes reachable via APIv2.
ci/apiv2/test_cracks_per_day.py Adds CI coverage for the new helper endpoint.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/inc/apiv2/helper/GetCracksPerDayHelperAPI.php Outdated
Comment thread src/inc/apiv2/helper/GetCracksPerDayHelperAPI.php
Comment thread src/inc/apiv2/helper/GetCracksPerDayHelperAPI.php Outdated
Comment thread src/inc/apiv2/helper/GetCracksPerDayHelperAPI.php Outdated
Comment thread ci/apiv2/test_cracks_per_day.py Outdated
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

Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.

Comment thread src/inc/apiv2/helper/GetCracksPerDayHelperAPI.php
Comment thread src/inc/apiv2/helper/GetCracksPerDayHelperAPI.php Outdated
Comment thread src/dba/models/HashFactory.php Outdated
Comment thread src/dba/models/HashFactory.php Outdated
Comment thread src/dba/models/HashFactory.php Outdated
@jessevz jessevz marked this pull request as ready for review May 18, 2026 14:12
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.

2 participants