Skip to content

Hub endpoint refactor#167

Open
Nefarious46 wants to merge 11 commits into
teragrep:mainfrom
Nefarious46:HubEndpointRefactor
Open

Hub endpoint refactor#167
Nefarious46 wants to merge 11 commits into
teragrep:mainfrom
Nefarious46:HubEndpointRefactor

Conversation

@Nefarious46

Copy link
Copy Markdown
Contributor
  • Void on delete
  • Procedures named accordingly
  • Controllers name accordingly
  • Tests renewed so that testData is inserted before tests.

More info on PR #155

@Nefarious46 Nefarious46 requested a review from kortemik May 20, 2025 13:46
@Nefarious46 Nefarious46 self-assigned this May 20, 2025
@Nefarious46 Nefarious46 added the enhancement New feature or request label May 20, 2025
@Nefarious46 Nefarious46 force-pushed the HubEndpointRefactor branch from e7e7697 to e515277 Compare June 3, 2025 14:57
Comment thread src/main/java/com/teragrep/cfe18/handlers/HubController.java
Comment thread src/main/java/com/teragrep/cfe18/handlers/HubController.java
Comment thread src/test/java/com/teragrep/cfe18/controllerTests/HubControllerTest.java Outdated
@eemhu

eemhu commented Jun 4, 2025

Copy link
Copy Markdown
Collaborator

Otherwise looks ok

@Nefarious46 Nefarious46 requested a review from eemhu June 4, 2025 09:06
eemhu
eemhu previously approved these changes Jun 4, 2025
@q22u q22u removed the review label Aug 22, 2025
@Nefarious46

Nefarious46 commented Feb 18, 2026

Copy link
Copy Markdown
Contributor Author

rebase 4f6c299
Spotless still fails.

@Nefarious46

Copy link
Copy Markdown
Contributor Author

Rebased

@Nefarious46 Nefarious46 force-pushed the HubEndpointRefactor branch from 7517a9d to af0eec5 Compare April 10, 2026 07:55
@Nefarious46 Nefarious46 requested a review from eemhu April 10, 2026 07:57
@Nefarious46 Nefarious46 force-pushed the HubEndpointRefactor branch from af0eec5 to 35094ad Compare May 29, 2026 12:16
@Nefarious46

Copy link
Copy Markdown
Contributor Author

Rebased. Going to rework TestApiClient still.

@Nefarious46 Nefarious46 removed request for eemhu and kortemik May 29, 2026 12:17
@eemhu eemhu dismissed their stale review June 1, 2026 09:35

Changes after review.

@Nefarious46 Nefarious46 force-pushed the HubEndpointRefactor branch from 65bf6ee to 2e49c82 Compare June 1, 2026 09:40
@Nefarious46

Copy link
Copy Markdown
Contributor Author

Rebased utility endpoint changes. Ready for a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants