Skip to content
This repository has been archived by the owner on Oct 24, 2024. It is now read-only.

Upgrade Rails to 7 #728

Closed
wants to merge 18 commits into from
Closed

Upgrade Rails to 7 #728

wants to merge 18 commits into from

Conversation

mogul
Copy link
Contributor

@mogul mogul commented Jul 12, 2024

I got 24 failing tests without changing any defaults yet.

I'm attempting to replace UJS with turbo-rails but have not had any luck with that yet.

@mogul mogul force-pushed the rails-upgrade-70 branch 2 times, most recently from ac45e27 to f0094d6 Compare July 15, 2024 02:59
@mogul
Copy link
Contributor Author

mogul commented Jul 15, 2024

I'm stuck on this problem where importmaps may not work with certain extensions installed.

In particular I see in the browser console:

An import map is added after module script load was triggered.localhost/:1 
Uncaught TypeError: Failed to resolve module specifier "application". Relative references must start with either "/", "./", or "../". localhost/:1 
Uncaught TypeError: Failed to resolve module specifier "list.js". Relative references must start with either "/", "./", or "../".

If I load the page in an Incognito window, I don't see the error. However, I can't find any extensions installed that might interfere.

@mogul
Copy link
Contributor Author

mogul commented Jul 16, 2024

When submitting (directly, or via the tests) the result is often a 406 with ActionController::UnknownFormat. This appears to be because the search_controller.rb is not set up to respond with HTML.

@rahearn
Copy link
Contributor

rahearn commented Jul 23, 2024

Closing in favor of #737

@rahearn rahearn closed this Jul 23, 2024
@rahearn rahearn deleted the rails-upgrade-70 branch July 24, 2024 12:18
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants