VisitorIdentification component now uses sitecore context hook

[art-alexeyenko]

VisitorIdentification component now uses sitecore context hook instead of withSitecoreContext helper

Description / Motivation

Replacing withSitecoreContext usage in VI component to the context hook - in order to avoid nextjs production issues.

Testing Details

Manual testing

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

[thaiphan]

Thanks! We were literally running into an issue caused by this.

[illiakovalenko]

In case if it's not working using withSitecoreContext it means that the problem is there? It can be fixed and we don't need to replace HOC by the hook. What is the difference? Why it's working using a hook?

[art-alexeyenko]

@illiakovalenko the underlying issue is React context not working with Provider-Consumer approach in production mode (hook is fine) which requires some more work and investigation. Perhaps we can push this proposed change and then take a closer look at the Provider-Consumer problem.

[pzi]

@illiakovalenko @ambrauer are we able to get a hotfix for v20.0.1 with this and the updated types for nextjs pkg, please?

[illiakovalenko]

@pzi Right, the hotfix will be released, at least for this fix

[pzi]

@illiakovalenko thank you