Skip to content

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

Merged
rwjblue merged 1 commit intomasterfrom
locks-patch-1
Feb 2, 2019
Merged

Update ember-welcome-page usage to angle brackets#8383
rwjblue merged 1 commit intomasterfrom
locks-patch-1

Conversation

@locks
Copy link
Copy Markdown
Contributor

@locks locks commented Jan 24, 2019

No description provided.

@rwjblue
Copy link
Copy Markdown
Member

rwjblue commented Jan 24, 2019

Need to update the test fixtures here:

@stefanpenner
Copy link
Copy Markdown
Contributor

fancy

@ppcano
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor

ppcano commented Jan 31, 2019

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

@rwjblue
Copy link
Copy Markdown
Member

rwjblue commented Jan 31, 2019

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

@locks
Copy link
Copy Markdown
Contributor Author

locks commented Jan 31, 2019

On it!

@locks
Copy link
Copy Markdown
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
Copy Markdown
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.

5 participants