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

Removing view blueprint #5607

Merged
merged 1 commit into from
Mar 14, 2016
Merged

Conversation

jasonmit
Copy link
Contributor

Since ember-cli now targets 2.x, which doesn't expose Ember.View, we should likely remove it and instead move the blueprint to the ember-legacy-views addon.

Related: http://discuss.emberjs.com/t/can-not-generate-a-view-for-the-index-route/10131

@rwjblue
Copy link
Member

rwjblue commented Mar 13, 2016

Agreed.

@stefanpenner
Copy link
Contributor

@homu r+

@homu
Copy link
Contributor

homu commented Mar 13, 2016

📌 Commit c4c053d has been approved by stefanpenner

@homu
Copy link
Contributor

homu commented Mar 13, 2016

⌛ Testing commit c4c053d with merge 65bbd93...

homu added a commit that referenced this pull request Mar 13, 2016
Removing view blueprint

Since ember-cli now targets 2.x, which doesn't expose `Ember.View`, we should likely remove it and instead move the blueprint to the `ember-legacy-views` addon.

Related: http://discuss.emberjs.com/t/can-not-generate-a-view-for-the-index-route/10131
@homu
Copy link
Contributor

homu commented Mar 14, 2016

💔 Test failed - status

@jasonmit
Copy link
Contributor Author

@stefanpenner tests should now be passing, two required trailing line breaks trolled me..

@kellyselden
Copy link
Member

@homu r+

@homu
Copy link
Contributor

homu commented Mar 14, 2016

📌 Commit 3984f7f has been approved by kellyselden

@homu
Copy link
Contributor

homu commented Mar 14, 2016

⌛ Testing commit 3984f7f with merge c0dda14...

homu added a commit that referenced this pull request Mar 14, 2016
Removing view blueprint

Since ember-cli now targets 2.x, which doesn't expose `Ember.View`, we should likely remove it and instead move the blueprint to the `ember-legacy-views` addon.

Related: http://discuss.emberjs.com/t/can-not-generate-a-view-for-the-index-route/10131
@homu
Copy link
Contributor

homu commented Mar 14, 2016

☀️ Test successful - status

@homu homu merged commit 3984f7f into ember-cli:master Mar 14, 2016
@jasonmit jasonmit deleted the remove-view-blueprint branch March 14, 2016 04:58
@mike-north
Copy link
Member

In the future we may wish to at least leave the de-generator intact for this kind of thing. ember d view-test my-view continues to be of value as people are upgrading to ember 2.5.

@jasonmit
Copy link
Contributor Author

The blueprints were moved into the legacy view addon but it looks like that was never versioned.

/cc @rwjblue @stefanpenner

@nathanhammond
Copy link
Contributor

If we do a new release of ember-legacy-views then a new npm install would give you the ability to destroy. It's not too late. Can we bump that version to 0.2.1?

@toranb
Copy link
Contributor

toranb commented May 10, 2016

@stefanpenner @jasonmit ...so is ember-cli 2.4.x not ember 1.13 friendly anymore ? What is the last version ember 1.13 devs can use w/out conflict ?

@jasonmit
Copy link
Contributor Author

In lock step with Ember, but I see no reason why it wouldn't work with 1.13. If you need the blueprints you can continue to pull them in through the legacy view addon (assuming that was versioned).

@rwjblue
Copy link
Member

rwjblue commented May 10, 2016

Or just make the files by hand...

@toranb
Copy link
Contributor

toranb commented May 11, 2016

@rwjblue @jasonmit awesome! just wanted to make sure this was (overall) ember 1.13/2.x friendly :)

mriska added a commit to mriska/ember-cli-coffeescript that referenced this pull request Jun 8, 2016
They are removed in ember-cli as of 2.4.3 ember-cli/ember-cli#5607
And having them in ember-cli-coffeescript gives errors if using a never version of ember-cli in the same project

```
Cannot find module 'ember-cli/blueprints/view'
Error: Cannot find module 'ember-cli/blueprints/view'
  at Function.Module._resolveFilename (module.js:339:15)
  at Function.Module._load (module.js:290:25)
  at Module.require (module.js:367:17)
  at require (internal/module.js:20:19)
  at Object.<anonymous> (/Users/mriska/work/test3/node_modules/ember-cli-coffeescript/blueprints/view/index.js:1:80)
```
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

8 participants