Skip to content

feat(payments-next): Replace console logging calls with LoggerService#20134

Open
david1alvarez wants to merge 1 commit intomainfrom
PAY-2131
Open

feat(payments-next): Replace console logging calls with LoggerService#20134
david1alvarez wants to merge 1 commit intomainfrom
PAY-2131

Conversation

@david1alvarez
Copy link
Contributor

Because:

  • LoggerService provides better metadata and consistent log levels across the application than the console logs permit

This commit:

  • Replaces console calls with LoggerService in NestJS services, Next.js route handlers, and server actions
  • Adds an optional logger parameter to type-cacheable cache strategies, defaulting to console to preserve existing behavior

Closes #PAY-2131

Checklist

Put an x in the boxes that apply

  • My commit is GPG signed.
  • If applicable, I have modified or added tests which pass locally.
  • I have added necessary documentation (if appropriate).
  • I have verified that my changes render correctly in RTL (if appropriate).

Screenshots (Optional)

Please attach the screenshots of the changes made in case of change in user interface.

Other information (Optional)

Any other information that is important to this pull request.

Because:
* LoggerService provides better metadata and consistent log levels across the application than the console logs permit

This commit:
* Replaces console calls with LoggerService in NestJS services, Next.js route handlers, and server actions
* Adds an optional logger parameter to type-cacheable cache strategies, defaulting to console to preserve existing behavior

Closes #PAY-2131
@david1alvarez david1alvarez requested a review from a team as a code owner March 3, 2026 19:48
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.

1 participant