initial Klevu integration#1854
initial Klevu integration#1854jerrypena1 wants to merge 1 commit intobigcommerce:integrations/klevufrom
Conversation
|
|
@jerrypena1 is attempting to deploy a commit to the BigCommerce Platform Team on Vercel. A member of the Team first needs to authorize it. |
matthewvolk
left a comment
There was a problem hiding this comment.
Just one! Thanks Juan 😄
There was a problem hiding this comment.
My only review comment is to do with this file - and I think you probably already saw it coming from what you mentioned in your README 🙂
This file contains the KlevuInit component which needs two attributes specific to your Klevu account `apiKey` & `url`.
Is there any chance we can replace the hard coded apiKey and url with a NEXT_PUBLIC_ environment variable? This will help with developers wanting to swap out keys, but will also help with the OCC deployment - we can programmatically add these variables potentially during deployment.
This is my only change requested, this PR looks good for approval after this!
What/Why?
This is the initial integration of Klevu webcomponents into Catalys