Consistent logs - Part 2 using logger (@W-15832306@)

[adamraya]

Description

Follow-up PR from https://github.com/SalesforceCommerceCloud/pwa-kit/pull/1822 Replacing all the console statements in PWA Kit packages with the equivalent logger.

Types of Changes

• [ ] Bug fix (non-breaking change that fixes an issue)
• [ ] New feature (non-breaking change that adds functionality)
• [ ] Documentation update
• [ ] Breaking change (could cause existing functionality to not work as expected)
• [ ] Other changes (non-breaking changes that does not fit any of the above)

Breaking changes include:

• Removing a public function or component or prop
• Adding a required argument to a function
• Changing the data type of a function parameter or return value
• Adding a new peer dependency to package.json

Changes

• Replacing all the console statements in PWA Kit packages printing logs on the server/client with the equivalent logger.

How to Test-Drive This PR

• (step1)

Checklists

General

• [ ] Changes are covered by test cases
• [ ] CHANGELOG.md updated with a short description of changes (not required for documentation updates)

Accessibility Compliance

You must check off all items in one of the follow two lists:

• [ ] There are no changes to UI

or...

• [ ] Changes were tested with a Screen Reader (iOS VoiceOver or Android Talkback) and had no issues
• [ ] Changes comply with WCAG 2.0 guidelines levels A and AA
• [ ] Changes to common UI patterns and interactions comply with WAI-ARIA best practices

Localization

• [ ] Changes include a UI text update in the Retail React App (which requires translation)

[vmarta]

@adamraya how about the console warnings from commerce-sdk-react? The package has no dependency on pwa-kit-dev, so we probably don't want to import its logger in this case. https://github.com/SalesforceCommerceCloud/pwa-kit/blob/c8aa0c0525a9e6e3292dc01fdd36d8c80dd26387/packages/commerce-sdk-react/src/auth/index.ts#L405-L409

But perhaps we can make CommerceApiProvider to receive an optional logger instance? If the logger is provided, then commerce-sdk-react would use it. Otherwise, fall back to console.warn.