Skip to content

Conversation

jctrebalag
Copy link
Collaborator

@jctrebalag jctrebalag commented Oct 18, 2019

Pour tester: créer un nouveau bénéficiaire puis un nouvel aidant, et userpilot s'occupe du reste (enfin presque^^).

@sophiemoustard sophiemoustard changed the title Com 714 [DO NOT MERGE] Com 714 Oct 18, 2019
const hasSigned = await this.hasSignedDoc(mandate.everSignId);
if (hasSigned) {
this.$customers.saveSignedDoc({ _id: this.customer._id, mandateId: mandate._id });
window.userpilot.identify(this.$store.getters['main/user']._id, { signedMandate: 'yes' });
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pourquoi tu as juste unidentify ici alors que pour les souscriptions tu as track et identify ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est un oubli '-_-

}
}
} else {
identifyUser(store.getters['main/user'])
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tu ne reload pas ici ? pourquoi ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

C'est un oubli ''-_-

Comment on lines 301 to 302
if (isIban) window.userpilot.track('iban_ok');
window.userpilot.identify(this.$store.getters['main/user']._id, { iban: 'yes' });
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

on ne regarde pas le couple bic et iban ?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Si, c'est corrigé !

@cmolle
Copy link
Collaborator

cmolle commented Oct 18, 2019

Lorsque je charge la page j'obtient une erreur parce que mandate est undefined

@jctrebalag
Copy link
Collaborator Author

Corrigé !

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.

3 participants