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

Remove deprecated features for ember-cli-fastboot v3 release #820

Merged
merged 3 commits into from
Apr 7, 2021

Conversation

xg-wang
Copy link
Member

@xg-wang xg-wang commented Apr 4, 2021

  • @service('fastboot')
    • cookies -> this.fastboot.request.cookies
    • headers -> this.fastboot.request.headers
    • host -> this.fastboot.request.host
  • ember-addon.fastBootDependencies -> ember-addon.fastbootDependencies (the lowercased 'b')
  • app/initializers|instance-initialzers/fastboot -> fastboot/initializers|instance-initializers/

Copy link
Member

@kiwiupover kiwiupover left a comment

Choose a reason for hiding this comment

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

@xg-wang test-packages/example-addon should not be a submodule. I think it has a .git folder that needs to be deleted.

@xg-wang
Copy link
Member Author

xg-wang commented Apr 5, 2021

@kiwiupover yea, turns out if I rm -rf .git after adding it, it is still being tracked as submodule 🤦🏼

@kiwiupover
Copy link
Member

@xg-wang this is better

@kiwiupover
Copy link
Member

@xg-wang Thanks.

@xg-wang xg-wang added this to the ember-cli-fastboot v3 milestone Apr 6, 2021
* `@service('fastboot')`
  * `cookies` -> `this.fastboot.request.cookies`
  * `headers` -> `this.fastboot.request.headers`
  * `host` -> `this.fastboot.request.host`
* `ember-addon.fastBootDependencies` -> `ember-addon.fastbootDependencies` (the lowercased 'b')
* `app/initializers|instance-initialzers/fastboot` -> `fastboot/initializers|instance-initializers/`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants