Skip to content

chore: Merge release(3.4.3) in master #3458

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 10 commits into from
Mar 14, 2018

Conversation

rosen-vladimirov
Copy link
Contributor

No description provided.

Natalia-Hristova and others added 10 commits February 1, 2018 13:31
We need to track users that export their projects from playground and opens them in CLI or Sidekick. To support that we introduce playground key in nativescript key in package.json file. For example:
```
{
	"nativescript": {
		"playground": {
			"id": "some user quid",
			"usedTutorial": false // is not obligatory. In case it is present, can be true or false
		}
	}
}
```

If  case when package.json file contains playground key, {N} CLI reads playground data and saves them in userSettings file. If usedTutorial=true is already saved in userSettings file, {N} CLI does not overwrite it. After that {N} CLI deletes playground key from package.json file.

In case when package.json file does not contain playground key, {N} CLI checks if playground key is already saved in userSettings file. It this is the case, {N} CLI reads playground data from userSettings file.
In case project template has a preconfigured plugins data (for example API keys), it will be stored in the template's package.json.
After creating the project, CLI removes all keys except the one defined in `PackageJsonKeysToKeep` array from `app/package.json` file.
So it removes the predefined configuration of the pluginData.
Fix this by adding the entry "pluginsData" in the array, so we'll keep it as well.
feat(Analytics): Respect playground key from package.json file
feat(create): Keep pluginsData key from template's package.json
Update version.
Update Changelog.
chore(release): Preapre for 3.4.3 release
Conflicts:
	PublicAPI.md
	lib/bootstrap.ts
	lib/common
	npm-shrinkwrap.json
	package.json
	test/platform-commands.ts
	test/plugins-service.ts
@rosen-vladimirov rosen-vladimirov merged commit 2d96010 into master Mar 14, 2018
@rosen-vladimirov rosen-vladimirov deleted the vladimirov/merge-rel-master branch March 14, 2018 19:28
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