Skip to content
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

[BUGFIX release] ease core-object upgrade for addons which lack .project #5972

Merged
merged 2 commits into from Jun 11, 2016

Conversation

stefanpenner
Copy link
Contributor

No description provided.

@Turbo87
Copy link
Member

Turbo87 commented Jun 10, 2016

👍

@@ -31,6 +31,17 @@ describe('models/addon.js', function() {
});
});

describe('old core object compat', function() {
it.only('treeGenerator works without .project', function() {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

oops

@stefanpenner
Copy link
Contributor Author

stefanpenner commented Jun 10, 2016

this will need to be (once green) cherry-picked to all the active branches

  • beta
  • release

Most likely need new releases for:

  • 1.7.0-beta.1 (optional)
  • 1.6.0 (required)

@stefanpenner
Copy link
Contributor Author

Confirmed this fixes the issue in question. I'll be able to cut a new release around noon PST

@nathanhammond
Copy link
Contributor

@homu r-

@nathanhammond
Copy link
Contributor

Blocking for a second: we need to provide an error message or we'll never migrate the community off of broken addons.

@stefanpenner stefanpenner added this to the v2.6.1 milestone Jun 10, 2016
@stefanpenner
Copy link
Contributor Author

Blocking for a second: we need to provide an error message or we'll never migrate the community off of broken addons.

I can add a warning

@stefanpenner
Copy link
Contributor Author

stefanpenner commented Jun 10, 2016

screen shot 2016-06-10 at 11 42 25 am

Color is wrong but if an addon doesn't have project, it likely also doesn't have UI. So I think its ok.

@stefanpenner
Copy link
Contributor Author

Ah my OCD wont allow that, fixed:

screen shot 2016-06-10 at 11 47 49 am

Some addons may not have been instantiated correctly (for latest CoreObject)
This detects a common failure mode. Hopefully this helps us catch and
upgrade these addons quickly.
@nathanhammond
Copy link
Contributor

@homu r+

@homu
Copy link
Contributor

homu commented Jun 11, 2016

📌 Commit 6cb0713 has been approved by nathanhammond

@homu
Copy link
Contributor

homu commented Jun 11, 2016

⌛ Testing commit 6cb0713 with merge a72decc...

homu added a commit that referenced this pull request Jun 11, 2016
…mond

[BUGFIX release] ease core-object upgrade for addons which lack .project
@homu
Copy link
Contributor

homu commented Jun 11, 2016

☀️ Test successful - status

@homu homu merged commit 6cb0713 into master Jun 11, 2016
@stefanpenner stefanpenner deleted the ease-core-object-upgrade branch June 11, 2016 03:38
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.

None yet

4 participants