You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The core app.js file, which is the entrypoint for the whole frontend application, is almost 400 lines. There's a lot of logic in there that could be named and encapsulated in a function for readability. It also includes some anonymous functions that should be named.
We should break as much code as we can out of that module and name any discrete pieces of logic, purely for readability. We shouldn't do this until we've written tests for the existing functionality in #42
The text was updated successfully, but these errors were encountered:
The core app.js file, which is the entrypoint for the whole frontend application, is almost 400 lines. There's a lot of logic in there that could be named and encapsulated in a function for readability. It also includes some anonymous functions that should be named.
We should break as much code as we can out of that module and name any discrete pieces of logic, purely for readability. We shouldn't do this until we've written tests for the existing functionality in #42
The text was updated successfully, but these errors were encountered: