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

fix some pages not loading #2178

Merged
merged 1 commit into from
Oct 7, 2022
Merged

Conversation

patricklx
Copy link
Collaborator

@patricklx patricklx commented Oct 5, 2022

e.g. render performance page did not load and possibly other issues
because chunks of webpack where not loaded since url was wrong

embroider-build/ember-auto-import#540

@RobbieTheWagner
Copy link
Member

@patricklx can you please explain this some more? I do not understand when this is an issue. The render performance page works for me without this. Is this if apps are using embroider or something?

@patricklx
Copy link
Collaborator Author

patricklx commented Oct 5, 2022

@rwwagner90 sorry, I mean this is not working in the current main branch and can be seen with the nightly build on chrome: https://chrome.google.com/webstore/detail/ember-inspector-nightly/ibdbkmdgflhglikhjdbogjjflkialpfi
the problem also exists in the current release, when I inspect the ember-inspector network requests I see that it fails to get the chunks. I guess it was just luck that it does not seem to break anything.

@patricklx
Copy link
Collaborator Author

image

@patricklx
Copy link
Collaborator Author

also, if you look at the generated index.html script src:

<body class="ember-application theme--light">
    

    <script src="assets/vendor.js"></script>
<script src="/assets/chunk.131.94e9295903c351885e48.js"></script>
<script src="/assets/chunk.143.2f2438030067e3bf00af.js"></script>
    <script src="assets/ember-inspector.js"></script>
...
</body>

@RobbieTheWagner RobbieTheWagner merged commit 4ed17a0 into emberjs:main Oct 7, 2022
@patricklx patricklx deleted the patch-16 branch October 22, 2022 06:55
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.

2 participants