Skip to content

Commit

Permalink
Add promise handling to isIdentified() call (#284)
Browse files Browse the repository at this point in the history
* updates isIdentified to correctly handle promise

* updates changelog
  • Loading branch information
cykolln authored Jan 18, 2024
1 parent d660b14 commit 8b3ed62
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 7 deletions.
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
<!-- BEGIN RELEASE NOTES -->
### [Unreleased]

#### Fixed
- Updated customer.js to correctly handle promise returned from isIdentified

### [4.1.1] - 2023-12-12

#### Added
Expand Down
17 changes: 10 additions & 7 deletions view/frontend/web/js/customer.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,13 +12,16 @@ define([
customerData.getInitCustomerData().done(function () {
var customer = customerData.get('customer')();

if(_.has(customer, 'email') && customer.email && !klaviyo.isIdentified()) {
klaviyo.identify({
'$email': customer.email,
'$first_name': _.has(customer, 'firstname') ? customer.firstname : '',
'$last_name': _.has(customer, 'lastname') ? customer.lastname : ''
});
}
klaviyo.isIdentified().then((identified)=> {
if(_.has(customer, 'email') && customer.email && !identified) {
klaviyo.identify({
'$email': customer.email,
'$first_name': _.has(customer, 'firstname') ? customer.firstname : '',
'$last_name': _.has(customer, 'lastname') ? customer.lastname : ''
});
}
});

});

});

0 comments on commit 8b3ed62

Please sign in to comment.