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

Added empty Component-preload.js to avoid 404 in console #4

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

Added empty Component-preload.js to avoid 404 in console #4

wants to merge 1 commit into from

Conversation

toxvaerd
Copy link

@toxvaerd toxvaerd commented Mar 4, 2014

In openUI 1.18.8 I'm experiencing a 404 error in the console using the templates:
GET <project_url>/Component-preload.js 404 (Not Found) sap-ui-core.js:25

This error is discussed here: http://scn.sap.com/thread/3486179

I've added an empty Component-preload.js file to each project, and that seems to stop the errors from showing. I'm not sure if this is the right approach though.

@js1972
Copy link
Contributor

js1972 commented Mar 4, 2014

Just thought I'd add my 2cents worth since you mentioned my old scn thread. ;-)
I wouldn't worry about the component preload get request. Just ignore the error for now as putting an empty file in there still causes a wasted get request so doesn't help at all.
As the component model is still being worked on I'm sure it will be sorted out soon... @qmacro probably has some more insights.

@toxvaerd
Copy link
Author

toxvaerd commented Mar 4, 2014

Ok, thanks 😃

It wasn't obvious from the thread wether or not the error would be fixed in an upcoming version, but I would argue that either way the empty Component-preload file should be in the template, to avoid confusion for the user(developer).

If the error is fixed in an upcoming version of openUI5 the empty file can be removed then.

@js1972
Copy link
Contributor

js1972 commented Mar 4, 2014

Have you tried the preload=auto query string parameter as mentioned in that scn thread?

@toxvaerd
Copy link
Author

toxvaerd commented Mar 4, 2014

I have, but that doesn't fix the problem for me 😞

@js1972
Copy link
Contributor

js1972 commented Mar 4, 2014

Must be version dependant. Personally I'm just waiting for the openui5 sources to be released. ;-)

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.

2 participants