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

#6 #7 remove all dependencies - fix console errors caused by dependencies (ember-cli-babel) #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Krzyrok
Copy link

@Krzyrok Krzyrok commented Aug 8, 2019

I've removed all unnecessary files:

for CORS is used only middleware defined in index.js. Rest is unnecessary especially that local server, tests and build don't work.

Should fix #6 and #7

@Krzyrok Krzyrok force-pushed the remove-dependencies-fix-warnings branch from 624d54f to 48695ce Compare August 8, 2019 11:08
@xcambar
Copy link

xcambar commented Dec 19, 2019

I'm commenting just to get this back into the owners' notifications.

Thanks for the PR!

@xcambar
Copy link

xcambar commented Dec 20, 2019

FWIW, the project has been forked and published to npm as ember-cli-cors2, with this PR included. Thanks @Krzyrok

See https://github.com/peopledoc/ember-cli-cors2

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.

Deprecation Notice
2 participants