Skip to content
This repository was archived by the owner on Jul 28, 2022. It is now read-only.

Conversation

@abarisain
Copy link
Contributor

Hello,

Cette PR implémente:

  • trackEvent, avec l'objet BatchEventData introduit dans les SDK récents
  • trackTransaction
  • trackLocation

Le code viens principalement de notre plugin cordova, en version allegée au vu des possibilités offertes par RN.

J'ai aussi modifié legerement la configuration du projet pour pouvoir le build quand il est dans un sous dossier d'une test-app (sinon typescript remonte les dossiers, inclus les typings de l'app parente et plante completement)

Enfin, j'ai ajouté quelques helpers typescript (typage + logs), mais je n'y suis pas attaché.

Des changements au readme apportent:

  • Une précision permettant d'eviter à l'app de crash au démarrage sur une erreur de FirebaseApp
  • Une réécriture de la partie sur les small icon (Change small notification icon doc #14)
  • Des précisions sur le readme de developement: je l'ai resuivi de 0 et j'ai eu quelques problemes suite à la migration du repo NPM dans son nouveau prefixe

Fixes #14
Fixes #13

@abarisain abarisain requested a review from yleflour February 25, 2019 13:51
@abarisain
Copy link
Contributor Author

Hello,

Est-ce qu'il reste des retours blocants ? Je suis pret à faire les modifs nécessaires !

@yleflour yleflour merged commit 76a4c2d into bamlab:v4.0.0 Mar 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants