Skip to content

feat: track data from preview app on preview and run commands #4287

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
Jan 30, 2019

Conversation

Fatme
Copy link
Contributor

@Fatme Fatme commented Jan 16, 2019

{N} CLI checks if there is a file on device written from preview app, gets its content if such file exists and sends an event to google analytics. This will allow us to know how much users are using the Playground site and {N} CLI.

PR Checklist

What is the current behavior?

What is the new behavior?

Fixes/Implements/Closes #[Issue Number].

@Fatme Fatme added this to the 5.2.0 milestone Jan 16, 2019
@Fatme Fatme self-assigned this Jan 16, 2019
@dtopuzov
Copy link
Contributor

run ci

@@ -63,6 +90,10 @@ process.on("message", async (data: ITrackingInformation) => {
return;
}

if (data.type === TrackingTypes.PreviewAppData) {
await trackPreviewAppData(<IPreviewAppTrackingInformation>data);
Copy link
Contributor

Choose a reason for hiding this comment

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

add return

@Fatme
Copy link
Contributor Author

Fatme commented Jan 16, 2019

Update nativescript-preview-sdk version in package.json before merging

@Fatme
Copy link
Contributor Author

Fatme commented Jan 21, 2019

run ci

@Fatme Fatme force-pushed the fatme/track-data-from-preview-app branch 3 times, most recently from c9e7d51 to c21eb24 Compare January 22, 2019 15:29
@Fatme
Copy link
Contributor Author

Fatme commented Jan 23, 2019

run ci

@Fatme Fatme force-pushed the fatme/track-data-from-preview-app branch from c21eb24 to 56ec5fc Compare January 24, 2019 06:51
@Fatme
Copy link
Contributor Author

Fatme commented Jan 25, 2019

run ci

@rosen-vladimirov
Copy link
Contributor

Can you please rebase before merging, as we have a lot of commits in master and it will be great in case these commits are at the top of the tree

@Fatme Fatme force-pushed the fatme/track-data-from-preview-app branch from d45cfa3 to 973662f Compare January 30, 2019 08:54
@Fatme Fatme merged commit 4b9f1a3 into master Jan 30, 2019
@Fatme Fatme deleted the fatme/track-data-from-preview-app branch January 30, 2019 09:36
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.

4 participants