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

Update ember-welcome-page usage to angle brackets #8383

Merged
merged 1 commit into from Feb 2, 2019
Merged

Conversation

locks
Copy link
Contributor

@locks locks commented Jan 24, 2019

No description provided.

@rwjblue
Copy link
Member

rwjblue commented Jan 24, 2019

Need to update the test fixtures here:

@stefanpenner
Copy link
Contributor

fancy

@ppcano
Copy link
Contributor

ppcano commented Jan 27, 2019

screen shot 2019-01-27 at 6 50 56 am

The ember-welcome-page project should also change its copy:

remove the <WelcomePage /> component from your application.hbs

Note: MU apps defines the template at ui/routes/application/template.hbs.

@rwjblue
Copy link
Member

rwjblue commented Jan 27, 2019

Great point @ppcano! Mind tweaking the addon to change the copy? With a stretch goal of possibly keeping {{welcome-page}} displayed for Ember < 3.4 and detecting MU to display the correct location...

@ppcano
Copy link
Contributor

ppcano commented Jan 31, 2019

ember-cli/ember-welcome-page#124 is a PR for the mentioned addon changes

@rwjblue
Copy link
Member

rwjblue commented Jan 31, 2019

ping @locks - mind updating the fixtures I linked to?

@locks
Copy link
Contributor Author

locks commented Jan 31, 2019

On it!

@locks
Copy link
Contributor Author

locks commented Feb 2, 2019

Failing test on node 6:

not ok 1329 Acceptance: smoke-test ember test wasm
      Running: node_modules/ember-cli/bin/ember build in: /home/travis/build/ember-cli/ember-cli/tmp/some_cool_app_clone-lah7ZT3o.tmp

🤔

@rwjblue rwjblue merged commit 5718a59 into master Feb 2, 2019
@delete-merged-branch delete-merged-branch bot deleted the locks-patch-1 branch February 2, 2019 21:05
@rwjblue
Copy link
Member

rwjblue commented Feb 11, 2019

ember-cli/ember-welcome-page#124 updates the messaging to use angle bracket invocation as well, it is released in v4.0.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants