bencodeorg

Results 27 comments of bencodeorg

> > For my learning, could you describe what "the project file" means in this context? > > Yeah! By the 'project file', I meant the `main.json` for the project....

Not blocking, but count me a vote to keep this disabled. Keeping underscores is more readable to me, which I think is more valuable than skipping an unnecessary variable assignment.

> @bencodeorg - would you give a second look at this? I added DCDO flags just in case and it's my first attempt at those. Thanks! Looks right to me!...

> @bencodeorg - Yup! I want to default to using this change and have a back-up plan just in case. I tested locally. In the screenshots, I edited the local...

> Nice! Could other labs also simply add `project.registerSaveOnUnload();` to get this behavior? That, and making sure the editor has a change handler set up that marks a project as...

@fisher-alice @sanchitmalhotra126 had to move some stuff around here for testing purposes (and add a couple new tests) since y'all last looked, lmk what you think! Sorry to have not...

I don't even know if I agree with my past self here, but thoughts on keeping the clearing logic together with what we have in `AichatView`? https://github.com/code-dot-org/code-dot-org/pull/58103#discussion_r1575181925

@sureshc I'm hitting issues running `dashboard-server` after the MySQL 8 upgrade -- they are resolved if I do the gem upgrade in this PR -- thoughts on merging? ``` /Users/benjaminbrooks/.rbenv/versions/3.0.5/lib/ruby/gems/3.0.0/gems/activesupport-6.1.7.7/lib/active_support/dependencies.rb:332:in...

Uninstalling 4.2.5 did it, thanks!

@wilkie would you be open to rebasing this onto the `adds-docker-compose-basics` branch rather than the Selenium one? I think we have a clearer path/motivation for merging this one, and I'm...