-
Notifications
You must be signed in to change notification settings - Fork 202
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
Framework support for Ember.JS #354
Comments
This is a great idea- we should definitely add a link to a starter for Ember. @NullVoxPopuli I tried reproing the 413 issue by forking the base Ember CLI project you linked but can't seem reproduce it. Do you know if this is an error that's happened recently at all? |
Yup. Every time I use stackblitz. Repro steps:
|
Oh I see, the start command is optimistic in opening the devserver. Definitely seems like the writes to the |
I'd think stackblitz should respect .gitignore |
@EricSimons ember now has dedicated repo / branch for Stackblitz: https://github.com/ember-cli/editor-output/tree/stackblitz-app-output |
@jrvidal / @EricSimons - is there a place I can PR to add the link? or is the website private? |
@NullVoxPopuli ah yeah our website isn't public, but I'll circle with the team tomorrow morning and get this updated on the homepage/dashboard shortly 🙏 |
what's the verdict? <3 |
@EricSimons how'd the circling go? |
Hi @NullVoxPopuli! I'd be happy to make this happen, and we should be able to add the starter this week. One concern I have atm is, since our free tier doesn't include binary files, and the starter includes the Do you think you could change it to an svg or a hotlinked image? Either should resolve the issue. |
tldr: I think it's fine if we leave things as is -- the options for REPLs are a "best of no good options" sort of deal, and I think this can "be put up with" We can't really swap out the default blueprint, as it's nearly the same as what folks use on their local machines. And if they have dependencies installed locally (due to already making an ember app somewhere else), it's nice that they don't have to have internet to have the default welcome experience. Questions though:
|
That's too bad, it's not the best UX for a starter we'd want to put as an official one. I can't see any different options, but I'll discuss it with the team. Regarding the questions:
Where does the image come from btw? I wasn't able to locate it in the repository. |
thanks for the answers!
(it's a common pattern in the ember-ecosystem to have libraries that may provide static assets (super handy for work projects, too))
can we opt out of that? I don't think we need that behavior?
I suppose if we mustn't have the image, I'd have to remove the welcome page entirely by removing the addon and adjusting the blueprint files. (in this auto-updating script that we use to update online-editor outputs): https://github.com/ember-cli/ember-cli/blob/master/dev/update-output-repos.js#L67 |
Oh my, I've been digging deeper and it looks like that I've just forked the repo and set the I imagine this doesn't solve the issue as this would make the setup non-default thought, right? |
Ok, I will try to push this paths-ignoring configuration to the team – can't promise any ETA, but I realize it would be super useful |
taking care of that would also fix: #294
as far as output goes, I think that's less important. But, anything against ignoring what's specified in .gitignore?
Nice! I didn't actually know about that option in the
Thanks for investigating!! <3 |
Good news! looks like we are going with |
yay!!! 🎉 🌮 🥳 |
success? @sulco <3 |
WIP! :) |
success? @sulco <3 |
Right now the URL from the issue description (https://stackblitz.com/github/ember-cli/editor-output/tree/stackblitz-app-output?file=package.json) gives me: ~/projects/mklizrgop.github
❯ npm install && npm start
warn preInstall No repository field
┌ [1/3] 🔍 Resolving dependencies
└ Completed in 1.221s
┌ [2/3] 🚚 Fetching dependencies
│ info pruneDeps Excluding 2 dependencies. For more information use `--verbose`.
└ Completed in 3.674s
┌ [3/3] 🔗 Linking dependencies
└ Completed in 1.982s
info security We found `install` scripts which turbo skips for security reasons. For more information see
https://turbo.sh/install-scripts.
└─ [email protected]
success Saved lockfile "package-lock.json"
success Updated "package.json"
success Install finished in 7.626s
$ ember serve
TypeError: Failed to fetch
at _0x530128.performHTTPCall (https://mklizrgop--github.w.staticblitz.com/blitz.05388ae171e3a0a12539f1efbda0fa1189d4f549.js:15:282948)
~/projects/mklizrgop.github 13s
❯ |
@sulco any updates? |
Today, I get this error:
|
Now I have:
|
@EricSimons and @sulco I've made some progress with patch-package. https://stackblitz.com/github/NullVoxPopuli/ember-cli-editor-output/tree/stackblitz-app-patched-broccoli-and-no-ember-data?file=README.md
Atm, I have two situations depending on.. maybe luck? maybe browser? In firefox, I get this error:
and in chrome I get this error:
|
@EricSimons / @sulco any debugging tips? |
Things seem to be looking smooth now, actually: https://stackblitz.com/github/ember-cli/editor-output/tree/stackblitz-app-output?file=README.md can we get Ember added to the home page? <3 https://stackblitz.com/ |
Is there some kind of tutorial on how to setup an ember project with webcontainers? I would like to try it very much! |
if you have a node environment in your web container, you'd do: npx ember-cli new my-app
cd my-app
npm start |
Is there a way to make livereload work? I see the console rebuilding, but not the demo. |
I wonder if you need to fork it first? I have live reload working here: https://stackblitz.com/edit/github-qivg2a?file=app%2Fcomponents%2Fnav.hbs,app%2Frouter.js |
@NullVoxPopuli uff, it worked! I had to sign in for it to work. NICE. i'll refactor ASAP codesandbox from ember-eui demos for stackblitz |
you can put them all on a github repo and use specially crafted URLs to load all your demos from github 🎉
This way, if something goofy is going on, you have multiple options for loading your example code 🎉 (stackblitz, github.dev, local, etc) |
do you have an example of this @NullVoxPopuli ? |
Yeah, the automatically updated URL: https://stackblitz.com/github/ember-cli/editor-output/tree/stackblitz-app-output Updates every release |
We now have both JS and TS generated templates: JS: https://stackblitz.com/github/ember-cli/editor-output/tree/stackblitz-app-output New automation to keep them in sync: ember-cli/ember-cli#10211 |
@Nemikolh any updates? <3 |
Hey @NullVoxPopuli ! Sorry for the late response. Do you think you could give me an update on what you think is broken / missing? From what I could gather, it seems the main blocker is support for the |
No worries, @Nemikolh , lemme catch you up:
I think that was only for old copies of the ember generated app blueprint -- the solution was just to avoid Ember updates its app templates every 6 weeks, so things can progressively get fixed / updated on these links over time: Using these links |
Hey @NullVoxPopuli, just a heads up. We are currently making improvements to our starter list UI and I'll make sure your templates will be included. I don't have an ETA for now, but be reassured that we haven't forgotten about Ember! (and thank you so much for your message, it was super useful 😍 ) |
@NullVoxPopuli I think we still need to investigate why exactly the |
Can you remind of the ticket that was filed for the |
Yeah, totally - and super appreciate that mentality! Here is them Reproduction blitz: https://stackblitz.com/edit/github-8qeupk?file=package.json Looks like both GH issues have stable/non-updating blitz examples |
@NullVoxPopuli we're adding several new starters to the platform this week – Ember is going to be among them! ❤️ |
oooo very exciting!!! 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🎉 🥳 |
@NullVoxPopuli That's fantastic news! So it doesn't use the |
@d3lm kinda -- some legacy code paths still use it
but (I haven't looked at how |
Cool! Thanks for the info. |
Describe the solution you'd like:
I'd like stackblitz framework/template links to link to the official "output repo" for online editors (there is now an official branch just for stackblitz-specific customizations)
Provide a link to a github repo containing an example of the framework:
https://stackblitz.com/github/ember-cli/ember-new-output/tree/stable?file=README.mdhttps://stackblitz.com/github/ember-cli/editor-output/tree/stackblitz-app-output?file=README.md
This link is updated automatically with every release.
Additional context:
We have an influx of a need to create reproductions for situations for helping people out / having folks create small reproductions of their problems so we can help them better analyze their situation. Having "official" ember support would be beneficial so we don't get comments like this in Discord:
Issues that affect projects that have as many files as the default ember boilerplate need to be resolved:
The text was updated successfully, but these errors were encountered: