Skip to content

Conversation

@dkhalife
Copy link
Owner

Summary

  • remove jwtAuth field from UsersAPIHandler
  • update UsersAPI constructor signature

Testing

  • go vet ./...
  • go test ./...

https://chatgpt.com/codex/tasks/task_e_6872cd896a74832a93fce39d16fdbffc

Copilot AI review requested due to automatic review settings July 12, 2025 21:12
Copy link
Contributor

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 removes the unused jwtAuth dependency from the Users API handler and its constructor signature.

  • Removed the jwtAuth field from UsersAPIHandler
  • Updated UsersAPI constructor to drop the jwtAuth parameter

@codecov
Copy link

codecov bot commented Jul 12, 2025

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/apis/user.go 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@dkhalife dkhalife merged commit a112153 into main Jul 12, 2025
4 of 5 checks passed
@dkhalife dkhalife deleted the codex/remove-jwtauth-from-usersapihandler branch July 12, 2025 21:13
dkhalife added a commit that referenced this pull request Oct 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants